Skip to content

Commit 56153d4

Browse files
ciecierskicursoragent
authored andcommitted
[adoption_osp_deploy] Remove unused undercloud ctlplane_vip
The `_ctlplane_vip` variable was set from `cifmw_adoption_osp_deploy_scenario.undercloud.ctlplane_vip` and injected into the undercloud os-net-config templates, but no service endpoints actually use this VIP -- they all use `_ctlplane_ip`. The dependency OSPRH-12150 (TLS-e for adoption) is now closed without needing the VIP. Remove the variable assignment from `prepare_undercloud.yml` and the VIP CIDR detection block plus address line from both the standard and BGP os-net-config templates. Related-Issue: #OSPRH-17418 Co-authored-by: Cursor <cursoragent@cursor.com> Signed-off-by: Mikołaj Ciecierski <mikolaj.ciecierski@redhat.com> Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 81ed403 commit 56153d4

3 files changed

Lines changed: 0 additions & 13 deletions

File tree

roles/adoption_osp_deploy/tasks/prepare_undercloud.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,6 @@
119119
_ctlplane_undercloud_net: "{{ _undercloud_net.networks[_ctlplane_net_name] }}"
120120
_ctlplane_net: "{{ cifmw_networking_env_definition.networks[_ctlplane_net_name] }}"
121121
_ctlplane_ip: "{{ _ctlplane_undercloud_net[ip_version|default('ip_v4')] }}"
122-
_ctlplane_vip: "{{ cifmw_adoption_osp_deploy_scenario.undercloud.ctlplane_vip }}"
123122
_dns_server: "{{ _ctlplane_net[dns_version|default('dns_v4')] }}"
124123
_gateway_ip: "{{ _ctlplane_net[gw_version|default('gw_v4')] }}"
125124
_interface_mtu: "{{ _ctlplane_undercloud_net.mtu }}"

roles/adoption_osp_deploy/templates/os_net_config_undercloud.yml.j2

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,6 @@
44
{% else %}
55
{% set _ctlplane_ip_cidr = 32 %}
66
{% endif %}
7-
{% if ':' in _ctlplane_vip %}
8-
{% set _ctlplane_vip_cidr = 128 %}
9-
{% else %}
10-
{% set _ctlplane_vip_cidr = 32 %}
11-
{% endif %}
127
network_config:
138
- type: ovs_bridge
149
name: br-ctlplane
@@ -25,7 +20,6 @@ network_config:
2520
addresses:
2621
- ip_netmask: {{ _ctlplane_ip }}/{{ _ctlplane_cidr }}
2722
- ip_netmask: {{ _ctlplane_ip }}/{{ _ctlplane_ip_cidr }}
28-
- ip_netmask: {{ _ctlplane_vip }}/{{ _ctlplane_vip_cidr }}
2923
{% if cifmw_adoption_osp_deploy_scenario.undercloud.routes is defined %}
3024
{%- for route in cifmw_adoption_osp_deploy_scenario.undercloud.routes %}
3125
routes:

roles/adoption_osp_deploy/templates/os_net_config_undercloud_bgp.yml.j2

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,6 @@
44
{% else %}
55
{% set _ctlplane_ip_cidr = 32 %}
66
{% endif %}
7-
{% if ':' in _ctlplane_vip %}
8-
{% set _ctlplane_vip_cidr = 128 %}
9-
{% else %}
10-
{% set _ctlplane_vip_cidr = 32 %}
11-
{% endif %}
127
network_config:
138
- type: interface
149
name: nic1
@@ -28,7 +23,6 @@ network_config:
2823
addresses:
2924
- ip_netmask: {{ _ctlplane_ip }}/{{ _ctlplane_cidr }}
3025
- ip_netmask: {{ _ctlplane_ip }}/{{ _ctlplane_ip_cidr }}
31-
- ip_netmask: {{ _ctlplane_vip }}/{{ _ctlplane_vip_cidr }}
3226
routes:
3327
- ip_netmask: 192.168.123.0/24
3428
next_hop: 192.168.122.1

0 commit comments

Comments
 (0)