Skip to content

Commit

Permalink
infra: only install logrotate on right nodes
Browse files Browse the repository at this point in the history
For intsance, there is no need to install logrotate on clients nodes.

This also ensure logrotate is installed only for containerized
deployments since the packaging has an explicit dependency to logrotate

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 8ed11ea)
  • Loading branch information
guits authored and dsavineau committed Aug 18, 2020
1 parent 8f18411 commit 3fad167
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion roles/ceph-infra/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,16 @@
state: present
register: result
until: result is succeeded
when: not is_atomic | bool
tags: with_pkg
when:
- not is_atomic | bool
- containerized_deployment | bool
- inventory_hostname in groups.get(mon_group_name, []) or
inventory_hostname in groups.get(osd_group_name, []) or
inventory_hostname in groups.get(mds_group_name, []) or
inventory_hostname in groups.get(rgw_group_name, []) or
inventory_hostname in groups.get(mgr_group_name, []) or
inventory_hostname in groups.get(rbdmirror_group_name, [])

- name: add logrotate configuration
template:
Expand Down

0 comments on commit 3fad167

Please sign in to comment.