[reproducer] Sync src dir when controller available and using ZIronic

Keep support for legacy workflow where controller host is created
and actions are done not on hypervisor directly, but on controller.

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>

Author: danpawlik

reproducer.yml

@@ -108,18 +108,22 @@
             name: firewalld
             state: restarted
 
+    # NOTE(dpawlik): Keep support for legacy workflow where
+    # controller host is created and actions are done not on
+    # hypervisor directly, but on controller.
     # NOTE(dpawlik): Synchronize current Zuul repositories stored in $HOME/src
     # to controller-0 to make sure all projects are in state as they should.
-    - name: Synchronize src dir with controller-0
+    - name: Set hypervisor name if not defined
+      when: hypervisor is not defined
+      ansible.builtin.set_fact:
+        hypervisor: "{{ ansible_fqdn }}"
+
+    - name: Synchronize src dir
       when:
         - not cifmw_deploy_reproducer_env | default(true) | bool
-      vars:
-        sync_dir: "{{ ansible_user_dir }}/src"
-      ansible.posix.synchronize:
-        src: "{{ sync_dir }}/"
-        dest: "zuul@controller-0:{{ sync_dir }}"
-        archive: true
-        recursive: true
+      ansible.builtin.include_role:
+        name: reproducer
+        tasks_from: sync_src_dir.yml
 
     # NOTE(dpawlik): After calling reproducer role using ZIronic tool,
     # when the bootstrap phase has been completed, it generates a file
@@ -133,6 +137,7 @@
       ansible.builtin.include_role:
         name: reproducer
         tasks_from: overwrite_zuul_vars.yml
+
     - name: Run deployment if instructed to
       when:
         - cifmw_deploy_architecture | default(false) | bool

roles/cifmw_setup/tasks/bootstrap.yml

@@ -59,6 +59,8 @@
     role: discover_latest_image
 
 - name: Create artifacts with custom params
+  when:
+    - cifmw_deploy_reproducer_env | default(true) | bool
   tags:
     - always
   ansible.builtin.copy:

roles/reproducer/tasks/overwrite_zuul_vars.yml

@@ -13,7 +13,7 @@
     path: "{{ ansible_user_dir }}/configs/zuul_vars.yaml"
   register: _current_zuul_vars
 
-- name: Overwrite reproducer-variables.yml when ZIronic used
+- name: Overwrite reproducer-variables.yml when PreMetal used
   when: _current_zuul_vars.stat.exists
   vars:
     temp_reproducer_var_path: /tmp/reproducer-variables.yml
@@ -62,3 +62,11 @@
         backup: true
       no_log: "{{ cifmw_nolog | default(true) | bool }}"
       delegate_to: controller-0
+
+    - name: Overwrite custom-params.yml
+      ansible.builtin.copy:
+        content: "{{ merged_reproducer_slurp.content | b64decode }}"
+        dest: "{{ cifmw_basedir }}/artifacts/parameters/custom-params.yml"
+        mode: "0664"
+      no_log: "{{ cifmw_nolog | default(true) | bool }}"
+      delegate_to: controller-0

roles/reproducer/tasks/sync_src_dir.yml

@@ -0,0 +1,45 @@
+---
+# NOTE(dpawlik): Synchronize data when PreMetal provision+bootstrap the
+# hypervisor AND Zuul job uses PreMetal flavor which is executing
+# job directly on hypervisor. When job starts, src and secrets dir are "recreated"
+# by zuul-executor with current job data. Sync the data with
+# controller-0 which is executing nested Ansible.
+- name: Synchronize hypervisor secrets dir with controller-0
+  vars:
+    sync_dir: "{{ ansible_user_dir }}/{{ subdir }}"
+  ansible.posix.synchronize:
+    src: "{{ sync_dir }}/"
+    dest: "zuul@controller-0:{{ sync_dir }}"
+    archive: true
+    recursive: true
+  no_log: "{{ cifmw_nolog | default(true) | bool }}"
+  loop:
+    - secrets
+    - src
+  loop_control:
+    loop_var: subdir
+
+# NOTE(dpawlik): Synchronize data when PreMetal provision+bootstrap the
+# hypervisor BUT legacy workflow is used (spawning controller VM
+# which is executing all other playbooks). When job starts, all
+# data like secrets, src dir etc. is stored on controller VM, but because
+# later tasks are using nested ansible execution, then all playbooks needs
+# to be up-to-date.
+# That case would be mostly used by developers. Synchronize module
+# would ignore if sync would happen to the same host. No need to synchronize
+# with controller-0 again, due tasks above would do that.
+- name: Synchronize controller VM configs, secrets and src dir with hypervisor
+  vars:
+    sync_dir: "{{ ansible_user_dir }}/{{ subdir }}"
+  ansible.posix.synchronize:
+    src: "{{ sync_dir }}/"
+    dest: "zuul@{{ hypervisor }}:{{ sync_dir }}"
+    archive: true
+    recursive: true
+  no_log: "{{ cifmw_nolog | default(true) | bool }}"
+  loop:
+    - configs
+    - secrets
+    - src
+  loop_control:
+    loop_var: subdir