From da564f1c8ca857b21396fdd44df86307124ab885 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Guillaume=20Cor=C3=A9?= <gucore@redhat.com>
Date: Thu, 17 Feb 2022 17:24:14 +0100
Subject: [PATCH] Add default EE to fix wait for sync

---
 playbooks/service-provision.yaml | 61 +++++++++++++++++++++++++++++++-
 1 file changed, 60 insertions(+), 1 deletion(-)

diff --git a/playbooks/service-provision.yaml b/playbooks/service-provision.yaml
index ec5d0cb1f..2546e211c 100644
--- a/playbooks/service-provision.yaml
+++ b/playbooks/service-provision.yaml
@@ -28,6 +28,30 @@
       file: "{{ catalog_item_params_file }}"
       name: catalog_item_params
 
+  # RHPDS virtualenvs -> EE  automatic creation
+  - name: Override EE with default_execution_environment if not defined
+    vars:
+      ansible_control_plane: >-
+        {{ vars.catalog_item_params.__meta__.ansible_control_plane.type | default('tower') }}
+
+      execution_environment: >-
+        {{ vars.catalog_item_params.__meta__.deployer.execution_environment | default({}) }}
+
+      default_execution_environment:
+        image: image-registry.apps-dev.open.redhat.com/agnosticd/ee-{{ vars.catalog_item_params.__meta__.deployer.virtualenv | default('ansible2.9-python3.6-2021-11-30') }}
+        private: true
+
+      to_merge:
+        __meta__:
+          deployer:
+            execution_environment: "{{ default_execution_environment }}"
+    when: >-
+      ansible_control_plane == 'controller'
+      and execution_environment | default('', true) == ''
+
+    set_fact:
+      catalog_item_params: "{{ vars.catalog_item_params | combine(to_merge, recursive=True) }}"
+
   - name: Show catalog_item_params
     debug:
       var: vars.catalog_item_params
@@ -86,8 +110,43 @@
         vars.catalog_item_params != vars.catalog_item_params | combine(check_governor_job_vars)
       until: r_governor is success
       delay: 5
-      retries: 60
+      retries: 1
     rescue:
+    - name: Try to print diff between catalog_item_params and governor
+      ignore_errors: true
+      register: r_diff
+      vars:
+        # Check AnarchyGovernor vars that are not listed in params_to_variables
+        check_governor_job_vars: >-
+          {{ r_governor.resources[0].spec.vars.job_vars
+           | default({})
+           | dict2items
+           | json_query(filter_var_query)
+           | items2dict
+          }}
+        filter_var_query: >-
+          [?!contains(`{{ filter_var_list | to_json }}`, key)]
+        filter_var_list: >-
+          {{ (vars.catalog_item_params.agnosticv_meta.params_to_variables | default({})).values() | list
+           + ["agnosticv_meta"] }}
+      shell: |
+        temp1=$(mktemp --suffix=_admin_host)
+        temp2=$(mktemp --suffix=_governor)
+        cat | jq . > $temp1 <<EOF
+        {{ vars.catalog_item_params | to_json }}
+        EOF
+        cat | jq . > $temp2 <<EOF
+        {{ vars.catalog_item_params | combine(check_governor_job_vars) | to_json }}
+        EOF
+
+        diff -u $temp1 $temp2
+        rm $temp1 $temp2
+
+    - name: Print diff
+      when: r_diff is succeeded
+      debug:
+        msg: "{{ r_diff.stdout }}"
+
     - name: Report failure waiting for AnarchyGovernor vars sync
       fail:
         msg: |-