Merge pull request #1040 from eduolivares/OSPRH-20414

Override frr and ovn_bgp_agent directory owners recursively

Author: openshift-merge-bot[bot]

roles/edpm_bootstrap/tasks/bootstrap.yml

@@ -28,6 +28,26 @@
     setype: var_log_t
   become: true
 
+- name: Gather user fact
+  ansible.builtin.setup:
+    gather_subset:
+      - "!all"
+      - "!min"
+      - "user"
+  when:
+    - ansible_user is undefined
+
+- name: Ensure /var/lib/config-data/ansible-generated exists and fix subdirs owner and mode if it existed
+  become: true
+  ansible.builtin.file:
+    path: "/var/lib/config-data/ansible-generated"
+    state: directory
+    recurse: true
+    mode: "0755"
+    owner: "{{ ansible_user | default(ansible_user_id) }}"
+    group: "{{ ansible_user | default(ansible_user_id) }}"
+    setype: container_file_t
+
 - name: Gather services facts
   ansible.builtin.service_facts:
 

roles/edpm_frr/tasks/install.yml

@@ -54,9 +54,12 @@
       - "selinux"
 
 - name: Create directory {{ edpm_frr_config_basedir }}
+  become: true
   ansible.builtin.file:
     path: "{{ edpm_frr_config_basedir }}/etc/frr"
     recurse: true
     state: directory
     selevel: s0
     setype: container_file_t
+    owner: "{{ ansible_user | default(ansible_user_id) }}"
+    group: "{{ ansible_user | default(ansible_user_id) }}"

roles/edpm_ovn_bgp_agent/tasks/install.yml

@@ -35,6 +35,7 @@
     - {'path': "{{ edpm_ovn_bgp_agent_config_basedir }}", 'setype': container_file_t, 'mode': '0750'}
 
 - name: Create directory {{ edpm_ovn_bgp_agent_config_basedir }}
+  become: true
   ansible.builtin.file:
     path: "{{ edpm_ovn_bgp_agent_config_basedir }}/etc/ovn-bgp-agent"
     recurse: true