Skip to content

Commit 0133abb

Browse files
committed
Use update_containers role to update openstack services container image
This pr keeps the set_openstack_containers role when cifmw_update_containers_openstack var is not defined and use update_containers only when cifmw_update_containers_openstack is defined. Apart from this, this pr also updates: - Reproducer content provider to use update_containers role var - Removes duplicates OsContainer Image var from update_containers role Signed-off-by: Chandan Kumar <[email protected]>
1 parent 2ef6e09 commit 0133abb

File tree

5 files changed

+11
-6
lines changed

5 files changed

+11
-6
lines changed

Diff for: roles/edpm_prepare/tasks/kustomize_and_deploy.yml

+4-2
Original file line numberDiff line numberDiff line change
@@ -74,10 +74,12 @@
7474
).metadata.name
7575
}}
7676
77-
- name: Update BM CSV or Ansibleee CSV to update proper image
77+
- name: Update Openstack containers or BM CSV or Ansibleee CSV to update proper image
7878
when: >-
7979
(cifmw_update_containers_edpm_image_url is defined) or
80-
(cifmw_update_containers_ansibleee_image_url is defined)
80+
(cifmw_update_containers_ansibleee_image_url is defined) or
81+
((cifmw_update_containers_openstack is defined and
82+
cifmw_update_containers_openstack | bool))
8183
vars:
8284
cifmw_update_containers_metadata: "{{ _ctlplane_name }}"
8385
ansible.builtin.include_role:

Diff for: roles/edpm_prepare/tasks/main.yml

+5-1
Original file line numberDiff line numberDiff line change
@@ -119,8 +119,12 @@
119119
--namespace={{ cifmw_install_yamls_defaults['OPERATOR_NAMESPACE'] }}
120120
--for=jsonpath='{.status.phase}'=Complete --timeout=20m
121121
122+
# Note(chkumar): Keeping set_openstack_containers role
123+
# till we migrate this task to update_containers role
122124
- name: Update OpenStack Services containers Env
123-
when: cifmw_edpm_prepare_update_os_containers | bool
125+
when:
126+
- cifmw_edpm_prepare_update_os_containers | bool
127+
- cifmw_update_containers_openstack is not defined
124128
vars:
125129
cifmw_set_openstack_containers_extra_vars: "{{ cifmw_edpm_prepare_extra_vars }}"
126130
ansible.builtin.include_role:

Diff for: roles/reproducer/tasks/configure_controller.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -355,7 +355,7 @@
355355
rejectattr('key', 'equalto', 'cifmw_extras') |
356356
rejectattr('key', 'equalto', 'cifmw_openshift_kubeconfig') |
357357
rejectattr('key', 'equalto', 'cifmw_openshift_token') |
358-
rejectattr('key', 'equalto', 'cifmw_set_openstack_containers_registry') |
358+
rejectattr('key', 'equalto', 'cifmw_update_containers_registry') |
359359
rejectattr('key', 'match', '^cifmw_use_(?!lvms).*') |
360360
rejectattr('key', 'match', '^cifmw_reproducer.*') |
361361
rejectattr('key', 'match', '^cifmw_rhol.*') |

Diff for: roles/reproducer/templates/content-provider.yml.j2

+1-1
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@
103103
{{
104104
{'cifmw_operator_build_output': cifmw_operator_build_output,
105105
'content_provider_registry_ip': cifmw_rp_registry_ip,
106-
'cifmw_set_openstack_containers_registry:': cifmw_rp_registry_ip
106+
'cifmw_update_containers_registry': cifmw_rp_registry_ip
107107
} | to_nice_yaml
108108
}}
109109
{% endraw %}

Diff for: roles/update_containers/templates/update_containers.j2

-1
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,6 @@ spec:
5757
octaviaHousekeepingImage: {{ cifmw_update_containers_registry }}/{{ cifmw_update_containers_org }}/openstack-octavia-housekeeping:{{ cifmw_update_containers_tag }}
5858
octaviaWorkerImage: {{ cifmw_update_containers_registry }}/{{ cifmw_update_containers_org }}/openstack-octavia-worker:{{ cifmw_update_containers_tag }}
5959
openstackClientImage: {{ cifmw_update_containers_registry }}/{{ cifmw_update_containers_org }}/openstack-openstackclient:{{ cifmw_update_containers_tag }}
60-
osContainerImage: {{ cifmw_update_containers_registry }}/{{ cifmw_update_containers_org }}/edpm-hardened-uefi:{{ cifmw_update_containers_tag }}
6160
ovnControllerImage: {{ cifmw_update_containers_registry }}/{{ cifmw_update_containers_org }}/openstack-ovn-controller:{{ cifmw_update_containers_tag }}
6261
ovnControllerOvsImage: {{ cifmw_update_containers_registry }}/{{ cifmw_update_containers_org }}/openstack-ovn-base:{{ cifmw_update_containers_tag }}
6362
ovnNbDbclusterImage: {{ cifmw_update_containers_registry }}/{{ cifmw_update_containers_org }}/openstack-ovn-nb-db-server:{{ cifmw_update_containers_tag }}

0 commit comments

Comments
 (0)