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>
(cherry picked from commit 22c18e8)
  • Loading branch information
guits committed Apr 29, 2021
1 parent 80cf5b7 commit 8b284d4
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions infrastructure-playbooks/cephadm-adopt.yml
Expand Up @@ -812,11 +812,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 8b284d4

Please sign in to comment.