From 571b05e5a4283df0d7fab54527a17fb5a4b41797 Mon Sep 17 00:00:00 2001 From: Zack Cerza Date: Fri, 22 Apr 2016 12:20:27 -0600 Subject: [PATCH] Correct /var/log/pcp permissions for collectors Some hosts were getting a /var/log/pcp/pmmgr that was owned by root:root for some reason. Fixes: http://tracker.ceph.com/issues/15575 Signed-off-by: Zack Cerza --- roles/pcp/tasks/collector.yml | 2 ++ roles/pcp/tasks/manager.yml | 7 +------ roles/pcp/tasks/permissons.yml | 7 +++++++ 3 files changed, 10 insertions(+), 6 deletions(-) create mode 100644 roles/pcp/tasks/permissons.yml diff --git a/roles/pcp/tasks/collector.yml b/roles/pcp/tasks/collector.yml index f5e97043..99605fbc 100644 --- a/roles/pcp/tasks/collector.yml +++ b/roles/pcp/tasks/collector.yml @@ -23,6 +23,8 @@ when: ansible_pkg_mgr == "dnf" +- include: permissons.yml + - name: Restart pcp service: name: "{{ pmcd_service }}" diff --git a/roles/pcp/tasks/manager.yml b/roles/pcp/tasks/manager.yml index 74060148..dd66e2ca 100644 --- a/roles/pcp/tasks/manager.yml +++ b/roles/pcp/tasks/manager.yml @@ -72,12 +72,7 @@ mode: 0644 register: target_discovery -- name: Ensure /var/log/pcp is owned by pcp - file: - path: /var/log/pcp - owner: "{{ pcp_user }}" - group: "{{ pcp_user }}" - recurse: yes +- include: permissons.yml # This greatly speeds up polling for hosts - name: Set PMCD_CONNECT_TIMEOUT in pmmgr.options diff --git a/roles/pcp/tasks/permissons.yml b/roles/pcp/tasks/permissons.yml new file mode 100644 index 00000000..742d6e6c --- /dev/null +++ b/roles/pcp/tasks/permissons.yml @@ -0,0 +1,7 @@ +--- +- name: Ensure /var/log/pcp is owned by pcp + file: + path: /var/log/pcp + owner: "{{ pcp_user }}" + group: "{{ pcp_user }}" + recurse: yes -- 2.47.3