ceph
/
ceph-ansible
mirror of https://github.com/ceph/ceph-ansible.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

handlers: move tmpdir removal 

This moves the tmpdir removal tasks to main playbook.
The handlers in Ansible are too restrictive and doesn't work
quite well for our use case.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit bab034024c)
pull/7536/head
Guillaume Abrioux 2024-03-14 21:39:45 +01:00
parent ecea562d1c
commit fecc615d02
3 changed files with 57 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
roles/ceph-handler/handlers/main.yml
View File

@ -69,26 +69,3 @@
or inventory_hostname in groups.get(mds_group_name, [])
or inventory_hostname in groups.get(rgw_group_name, [])
or inventory_hostname in groups.get(rbdmirror_group_name, [])
- name: Remove tempdir for scripts
ansible.builtin.file:
path: "{{ tmpdirpath.path }}"
state: absent
listen:
- "Restart ceph mons"
- "Restart ceph osds"
- "Restart ceph mdss"
- "Restart ceph rgws"
- "Restart ceph rbdmirrors"
- "Restart ceph mgrs"
register: tmpdirpath
when:
- tmpdirpath.path is defined
- not _exporter_handler_called | default(false) | bool
- not _crash_handler_called | default(false) | bool
- not _mds_handler_called | default(false) | bool
- not _mgr_handler_called | default(false) | bool
- not _mon_handler_called | default(false) | bool
- not _osd_handler_called | default(false) | bool
- not _rbdmirror_handler_called | default(false) | bool
- not _rgw_handler_called | default(false) | bool

29
site-container.yml.sample
View File

@ -469,6 +469,35 @@
status: "Complete"
end: "{{ lookup('pipe', 'date +%Y%m%d%H%M%SZ') }}"
- hosts:
- mons
- osds
- mdss
- rgws
- rbdmirrors
- clients
- mgrs
- monitoring
gather_facts: false
become: True
any_errors_fatal: true
tasks:
- name: Remove tempdir for scripts
ansible.builtin.file:
path: "{{ tmpdirpath.path }}"
state: absent
when:
- tmpdirpath.path is defined
- not _exporter_handler_called | default(false) | bool
- not _crash_handler_called | default(false) | bool
- not _mds_handler_called | default(false) | bool
- not _mgr_handler_called | default(false) | bool
- not _mon_handler_called | default(false) | bool
- not _osd_handler_called | default(false) | bool
- not _rbdmirror_handler_called | default(false) | bool
- not _rgw_handler_called | default(false) | bool
- hosts: mons[0]
gather_facts: false
become: True

28
site.yml.sample
View File

@ -451,6 +451,34 @@
status: "Complete"
end: "{{ lookup('pipe', 'date +%Y%m%d%H%M%SZ') }}"
- hosts:
- mons
- osds
- mdss
- rgws
- rbdmirrors
- clients
- mgrs
- monitoring
gather_facts: false
become: True
any_errors_fatal: true
tasks:
- name: Remove tempdir for scripts
ansible.builtin.file:
path: "{{ tmpdirpath.path }}"
state: absent
when:
- tmpdirpath.path is defined
- not _exporter_handler_called | default(false) | bool
- not _crash_handler_called | default(false) | bool
- not _mds_handler_called | default(false) | bool
- not _mgr_handler_called | default(false) | bool
- not _mon_handler_called | default(false) | bool
- not _osd_handler_called | default(false) | bool
- not _rbdmirror_handler_called | default(false) | bool
- not _rgw_handler_called | default(false) | bool
- hosts: mons
gather_facts: false
become: True