Skip to content

Commit 8eadd43

Browse files
committed
Remove OCPBUGS-67159 workaround and enable openstack_test
OCPBUGS-67159 (IP/port collision in CAPO machine network) is fixed upstream and backported to OCP 4.19-4.22. Remove the hardcoded api_vip/ingress_vip workaround from adoption.yaml and the conditional VIP injection block from the IPI install-config template. Also enable the openstack_test stage for the adoption pipeline, previously disabled because tools_cifmw_dnsmasq delegates DNS tasks to the hypervisor which is unreachable from the adoption container. Guard both dnsmasq tasks with "when: hypervisor is defined" so the role gracefully skips DNS configuration in container-only environments while the actual openstack-tests still execute. Closes: OSPRH-25360 Ref: OSPRH-27365 Assisted-By: Claude Code Change-Id: Ie86f127a966d6017da46540defbfc49586b6cd91 Signed-off-by: Itay Matza <imatza@redhat.com>
1 parent 02d1c72 commit 8eadd43

3 files changed

Lines changed: 3 additions & 12 deletions

File tree

collection/stages/roles/install/templates/install-config-ipi.yaml.j2

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -59,11 +59,6 @@ platform:
5959
- subnet:
6060
name: "{{ installcfg_subnet.name }}"
6161
{%- else +%}
62-
{%- if ocp_deployment_topology.api_vip is defined +%}
63-
# Workaround for OCPBUGS-67159 (adoption uses pre-existing VIPs, not FIPs)
64-
apiVIPs: ["{{ ocp_deployment_topology.api_vip }}"]
65-
ingressVIPs: ["{{ ocp_deployment_topology.ingress_vip }}"]
66-
{%- endif +%}
6762
externalNetwork: "{{ installcfg_external_network }}"
6863
apiFloatingIP: "{{ installcfg_api_floating_ip }}"
6964
ingressFloatingIP: "{{ installcfg_ingress_floating_ip }}"

collection/stages/roles/openstack_test/tasks/main.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
- "{{ resources.apps_accessible_ip }}"
3232
delegate_to: "{{ hypervisor }}"
3333
remote_user: root
34+
when: hypervisor is defined
3435

3536
- name: Include Openstack-Test tasks
3637
ansible.builtin.include_tasks: run_openstack_test.yml
@@ -52,3 +53,4 @@
5253
tasks_from: restore.yml
5354
delegate_to: "{{ hypervisor }}"
5455
remote_user: root
56+
when: hypervisor is defined

jobs_definitions/adoption.yaml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,7 @@ stages:
88
- install
99
- post
1010
- verification
11-
# openstack_test delegates DNS tasks to the hypervisor via
12-
# tools_cifmw_dnsmasq, which is unreachable from the adoption
13-
# container context. Tracked in OSPRH-27365.
14-
# - openstack_test
11+
- openstack_test
1512
# Removed LB tests until Octavia is supported and enabled in adoption jobs
1613
# - lb_tests
1714

@@ -45,6 +42,3 @@ ocp_deployment_topology:
4542
servergroups:
4643
master: "soft-anti-affinity" # Required if number of computes < 3, otherwise can be anti-affinity
4744
worker: "soft-anti-affinity" # Required if number of computes < 3, otherwise can be anti-affinity
48-
# Workaround for OCPBUGS-67159
49-
api_vip: "10.196.0.15"
50-
ingress_vip: "10.196.0.17"

0 commit comments

Comments
 (0)