Skip to content

Commit

Permalink
cephadm_adopt: fix ceph-crash migration
Browse files Browse the repository at this point in the history
ceph-ansible leaves a ceph-crash container in containerized deployment.
It means we end up with 2 ceph-crash containers running after the
migration playbook is complete.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1954614

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
  • Loading branch information
guits committed Apr 28, 2021
1 parent 1f40c12 commit 22c18e8
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions infrastructure-playbooks/cephadm-adopt.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1003,11 +1003,10 @@

- name: stop and disable ceph-crash systemd service
service:
name: ceph-crash
name: "{{ 'ceph-crash@' + ansible_facts['hostname'] if containerized_deployment | bool else 'ceph-crash.service' }}"
state: stopped
enabled: false
failed_when: false
when: not containerized_deployment | bool

- name: update the placement of ceph-crash hosts
command: "{{ cephadm_cmd }} shell --fsid {{ fsid }} -- ceph --cluster {{ cluster }} orch apply crash --placement='label:ceph'"
Expand Down

0 comments on commit 22c18e8

Please sign in to comment.