[reproducer] Added dependencies urls and versions

This file in conjuction with:
* #3774
* #3772
* #3792

Allows ci-framework to clone those repos (architecture was already enabled)
when repositories are not already there due to zuul has not marked them as
required-projects or because we're running them outside the zuul ecosystem.

This is the almost the lowest in precedence, so if we still have these vars defined at zuul
level or defaults-vars (passed as extra-vars) or scenarios files, this file is not
taken into consideration.

 Precedence chain:
    * Zuul repos
    * cifmw_reproducer_repositories
    * cifmw_reproducer_default_repositories
    * Git pull logic in each of the roles

Signed-off-by: Enrique Vallespi Gil <evallesp@redhat.com>

Author: evallesp

group_vars/all.yml

@@ -30,12 +30,6 @@ cifmw_default_dns_servers:
   - "1.1.1.1"
   - "8.8.8.8"
 
-# group_vars related to github repo URLs
-cifmw_openstack_k8s_operators_org_url: "https://github.com/openstack-k8s-operators"
-cifmw_repo_url: "{{ cifmw_openstack_k8s_operators_org_url }}/ci-framework"
-cifmw_installyamls_repo_url: "{{ cifmw_openstack_k8s_operators_org_url }}/install_yamls"
-cifmw_architecture_repo_url: "{{ cifmw_openstack_k8s_operators_org_url }}/architecture"
-
 # set to false to enable logging of sensitive data
 cifmw_nolog: true
 
@@ -48,3 +42,31 @@ cifmw_default_container_image_tag: "current-podified"
 
 # Alternative namespace for master builds
 cifmw_master_container_image_namespace: "podified-master-centos9"
+cifmw_openstack_k8s_operators_org_url: "https://github.com/openstack-k8s-operators"
+
+#### Pinned external projects dependencies ####
+#
+# These variables define the source URL and version for
+# projects that ci-framework can either receive from Zuul (as a
+# required-project checkout) or clone on its own when running
+# outside Zuul or when the projects are not listed in required-projects.
+#
+# Roles reference these variables directly in their tasks.
+# Jobs can override any of them via vars or extra-vars.
+
+# ci-framework
+cifmw_repo_url: "{{ cifmw_openstack_k8s_operators_org_url }}/ci-framework"
+cifmw_repo_version_pin: "HEAD"
+
+# repo-setup
+cifmw_repo_setup_src: "{{ cifmw_openstack_k8s_operators_org_url }}/repo-setup"
+cifmw_repo_setup_version_pin: "HEAD"
+
+# install_yamls
+# TODO, eventually we should rename cifmw_installyamls* to cifmw_install_yamls* vars
+cifmw_installyamls_repo_url: "{{ cifmw_openstack_k8s_operators_org_url }}/install_yamls"
+cifmw_install_yamls_version_pin: "HEAD"
+
+# architecture
+cifmw_architecture_repo_url: "{{ cifmw_openstack_k8s_operators_org_url }}/architecture"
+cifmw_architecture_repo_version_pin: "HEAD"

roles/reproducer/tasks/default_repositories.yml

@@ -7,10 +7,12 @@
       ansible.builtin.stat:
         path: "{{ repository.dest }}"
 
+    # It seems override-checkout is not set anywhere. Not related with its value
+    # in zuul jobs
     - name: "Clone repository if needed: {{ repository.src | basename}}"
       when:
         - not _repo_clone.stat.exists
       ansible.builtin.git:  # noqa: latest[git]
         dest: "{{ repository.dest }}"
         repo: "{{ repository.src }}"
-        version: "{{ repository['override-checkout'] | default(omit) }}"
+        version: "{{ repository['override-checkout'] | default(repository.version) | default(omit) }}"

roles/reproducer/vars/main.yml

@@ -1,12 +1,26 @@
 ---
-# Default repositories we always want to have
+# Versions are set at group_vars/all.yml
+# TODO(evallesp): We're setting pull capabilities directly from roles that are required.
+# We might eventually think how to simplify reproducer and let each of the roles to clone
+# repositories. Might be also a good option to use an ansible role for pulling them
+# in the desired moment by cloning it.
+#
+# Precedence chain:
+#    * Zuul repos
+#    * cifmw_reproducer_repositories
+#    * cifmw_reproducer_default_repositories
+#    * Git pull logic in each of the roles
+
 cifmw_reproducer_default_repositories:
   - src: "{{ cifmw_repo_url }}"
     dest: "{{ cifmw_reproducer_controller_user_dir }}/{{ cifmw_repo_relative }}"
+    version: "{{ cifmw_repo_version_pin }}"
   - src: "{{ cifmw_installyamls_repo_url }}"
     dest: "{{ cifmw_reproducer_controller_user_dir }}/{{ cifmw_installyamls_repos_relative }}"
+    version: "{{ cifmw_install_yamls_version_pin }}"
   - src: "{{ cifmw_architecture_repo_url }}"
     dest: "{{ cifmw_reproducer_controller_user_dir }}/{{ cifmw_architecture_repo_relative }}"
+    version: "{{ cifmw_architecture_repo_version_pin }}"
 
 # one place to rule them all
 cifmw_reproducer_nm_dnsmasq: "/etc/NetworkManager/conf.d/zz-dnsmasq.conf"