Project

General

Profile

Bug #44673

cephadm: `orch apply` and `orch daemon add` use completely different code path

Added by Sebastian Wagner 8 months ago. Updated 2 months ago.

Status:
Fix Under Review
Priority:
Normal
Assignee:
-
Category:
cephadm
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature:

Description

... which is not obvious to users and they will use this interchangeably. Which is not really a good idea.

We should make it clear, that the CLI for doing unmanaged changes clearly states this.

Especially: cephadm will remove daemons added by daemon add!

History

#1 Updated by Sebastian Wagner 8 months ago

  • Assignee set to Sage Weil

#2 Updated by Sage Weil 7 months ago

  • Priority changed from High to Normal

#3 Updated by Sebastian Wagner 7 months ago

I'm already getting bug reports, like

I just deployed a v15.2.0 cluster and ceph orch ls tells me 3 out of 0 mons are running - is that a known bug? 
master:~ # ceph orch ls
NAME                  RUNNING  REFRESHED  AGE   PLACEMENT          IMAGE NAME                                                               IMAGE ID      
mgr                       3/3  26s ago    103m  node1,node2,node3  registry.suse.de/devel/storage/7.0/cr/containers/ses/7/ceph/ceph:latest  4947e2bbb809  
mon                       3/0  26s ago    -     <no spec>          registry.suse.de/devel/storage/7.0/cr/containers/ses/7/ceph/ceph:latest  4947e2bbb809  
...

ah, maybe because the MONs are deployed using ceph orch daemon add mon and not ceph orch apply

#5 Updated by Nathan Cutler 7 months ago

Sebastian Wagner wrote:

prerequisite: https://github.com/ceph/ceph/pull/34091

This one has been merged.

#6 Updated by Sebastian Wagner 7 months ago

  • Assignee deleted (Sage Weil)

#7 Updated by Sebastian Wagner 6 months ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 35064

#8 Updated by Sebastian Wagner 2 months ago

  • Description updated (diff)

Also available in: Atom PDF