Skip to content

Commit e73f9f3

Browse files
committed
[dataplane_adoption] Do not override images in the nodeset
OpenstackVersion CR have info for all the images, we should utilize that instead of overriding in the nodeset. Any image overrides in nodeset has highest priority so any handling/overrides with OpenstackVersion CR is not used. Also added edpmNodeExporterImage which was missing in OpenstackVersion template. Signed-off-by: Yatin Karel <ykarel@redhat.com>
1 parent 32a1b16 commit e73f9f3

2 files changed

Lines changed: 3 additions & 33 deletions

File tree

tests/roles/backend_services/templates/openstack_version.j2

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,9 @@ spec:
6969
swiftContainerImage: {{ container_registry }}/{{ container_namespace }}/openstack-swift-container:{{ container_tag }}
7070
swiftObjectImage: {{ container_registry }}/{{ container_namespace }}/openstack-swift-object:{{ container_tag }}
7171
swiftProxyImage: {{ container_registry }}/{{ container_namespace }}/openstack-swift-proxy-server:{{ container_tag }}
72+
{% if edpm_telemetry_node_exporter_image is defined -%}
73+
edpmNodeExporterImage: {{ edpm_telemetry_node_exporter_image }}
74+
{% endif %}
7275
{% if (backend_services_cindervolumes | length > 0 or
7376
(backend_services_cindervolumes_extraimages is defined and backend_services_cindervolumes_extraimages is mapping)) %}
7477
cinderVolumeImages:

tests/roles/dataplane_adoption/defaults/main.yaml

Lines changed: 0 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -74,9 +74,6 @@ netconfig_networks:
7474
start: "{{ tenant_allocation_start | default('172.19.0.100') }}"
7575
cidr: "{{ tenant_cidr | default('172.19.0.0/24') }}"
7676
vlan: 22
77-
registry_name: "quay.io"
78-
registry_namespace: "podified-antelope-centos9"
79-
image_tag: "current-podified"
8077
ansible_ssh_private_key_secret: dataplane-adoption-secret
8178
default_timesync_ntp_servers:
8279
- hostname: pool.ntp.org
@@ -345,27 +342,7 @@ dataplane_cr: |
345342
346343
timesync_ntp_servers: {{ timesync_ntp_servers | default(default_timesync_ntp_servers) }}
347344
348-
edpm_ovn_controller_agent_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-ovn-controller:{{ image_tag }}"
349-
edpm_iscsid_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-iscsid:{{ image_tag }}"
350-
edpm_logrotate_crond_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-cron:{{ image_tag }}"
351-
edpm_nova_compute_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-nova-compute:{{ image_tag }}"
352-
edpm_nova_libvirt_container_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-nova-libvirt:{{ image_tag }}"
353-
edpm_ovn_metadata_agent_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-neutron-metadata-agent-ovn:{{ image_tag }}"
354-
{% if telemetry_adoption|bool -%}
355-
edpm_telemetry_ceilometer_compute_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-ceilometer-compute:{{ image_tag }}"
356-
edpm_telemetry_ceilometer_ipmi_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-ceilometer-ipmi:{{ image_tag }}"
357-
{% endif -%}
358-
edpm_neutron_sriov_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-neutron-sriov-agent:{{ image_tag }}"
359-
edpm_neutron_ovn_agent_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-neutron-ovn-agent:{{ image_tag }}"
360-
edpm_neutron_metadata_agent_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-neutron-metadata-agent-ovn:{{ image_tag }}"
361-
edpm_neutron_dhcp_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-neutron-dhcp-agent:{{ image_tag }}"
362-
edpm_multipathd_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-multipathd:{{ image_tag }}"
363-
{% if edpm_telemetry_node_exporter_image is defined -%}
364-
edpm_telemetry_node_exporter_image: "{{ edpm_telemetry_node_exporter_image }}"
365-
{% endif -%}
366345
{% if bgp | bool -%}
367-
edpm_frr_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-frr:{{ image_tag }}"
368-
edpm_ovn_bgp_agent_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-ovn-bgp-agent:{{ image_tag }}"
369346
edpm_frr_bgp_ipv4_src_network: bgpmainnet
370347
edpm_frr_bgp_ipv6_src_network: bgpmainnetv6
371348
edpm_frr_bgp_neighbor_password: f00barZ
@@ -516,17 +493,7 @@ networker_cr: |
516493
edpm_nodes_validation_validate_controllers_icmp: false
517494
edpm_nodes_validation_validate_gateway_icmp: false
518495
timesync_ntp_servers: {{ timesync_ntp_servers | default(default_timesync_ntp_servers) }}
519-
edpm_ovn_controller_agent_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-ovn-controller:{{ image_tag }}"
520-
edpm_iscsid_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-iscsid:{{ image_tag }}"
521-
edpm_logrotate_crond_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-cron:{{ image_tag }}"
522-
edpm_ovn_metadata_agent_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-neutron-metadata-agent-ovn:{{ image_tag }}"
523-
edpm_neutron_ovn_agent_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-neutron-ovn-agent:{{ image_tag }}"
524-
edpm_neutron_metadata_agent_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-neutron-metadata-agent-ovn:{{ image_tag }}"
525-
edpm_neutron_dhcp_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-neutron-dhcp-agent:{{ image_tag }}"
526-
edpm_multipathd_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-multipathd:{{ image_tag }}"
527496
{%+ if bgp | bool +%}
528-
edpm_frr_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-frr:{{ image_tag }}"
529-
edpm_ovn_bgp_agent_image: "{{ registry_name }}/{{ registry_namespace }}/openstack-ovn-bgp-agent:{{ image_tag }}"
530497
edpm_frr_bgp_ipv4_src_network: bgpmainnet
531498
edpm_frr_bgp_ipv6_src_network: bgpmainnetv6
532499
edpm_frr_bgp_neighbor_password: f00barZ

0 commit comments

Comments
 (0)