Skip to content

Commit ed8c87d

Browse files
hjensasopenshift-cherrypick-robot
authored andcommitted
Add ML2 baremetal mechanism driver support for Ironic adoption
Configure Neutron to include the baremetal mechanism driver when Ironic adoption is enabled. This adds a new patch configuration that extends ml2MechanismDrivers to include both ovn and baremetal, and applies it conditionally during the neutron_adoption role when ironic_adoption is true. Closes: OSPRH-28478 Signed-off-by: Harald Jensås <hjensas@redhat.com>
1 parent 067c084 commit ed8c87d

4 files changed

Lines changed: 33 additions & 0 deletions

File tree

docs_user/modules/proc_adopting-the-networking-service.adoc

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,23 @@ $ oc patch openstackcontrolplane openstack --type=merge --patch '
7575
'
7676
----
7777
====
78+
+
79+
[NOTE]
80+
====
81+
If you are adopting the {bare_metal_first_ref}, you must configure the ML2 mechanism drivers to include both `ovn` and `baremetal`:
82+
83+
[source,shell]
84+
----
85+
$ oc patch openstackcontrolplane openstack --type=merge --patch '
86+
spec:
87+
neutron:
88+
template:
89+
ml2MechanismDrivers:
90+
- ovn
91+
- baremetal
92+
'
93+
----
94+
====
7895

7996
.Verification
8097

docs_user/modules/proc_deploying-the-bare-metal-provisioning-service.adoc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ $ oc get openstackcontrolplanes.core.openstack.org <name> -o jsonpath='{.spec.ir
2020
+
2121
** Replace `<name>` with the name of your existing `OpenStackControlPlane` CR, for example, `openstack-control-plane`.
2222
* The {identity_service_first_ref}, {networking_first_ref}, and {image_service_first_ref} are operational.
23+
* The {networking_service} is configured with the ML2 baremetal mechanism driver. For more information, see xref:adopting-the-networking-service_adopt-control-plane[Adopting the {networking_service}].
2324
+
2425
[NOTE]
2526
If you use the {bare_metal} in a Bare Metal as a Service configuration, do not adopt the {compute_service_first_ref} before you adopt the {bare_metal}.

tests/roles/neutron_adoption/defaults/main.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,3 +30,11 @@ neutron_config_patch: |
3030
networkAttachments:
3131
- internalapi
3232
neutron_retry_delay: 5
33+
34+
ironic_ml2_baremetal_patch:
35+
spec:
36+
neutron:
37+
template:
38+
ml2MechanismDrivers:
39+
- ovn
40+
- baremetal

tests/roles/neutron_adoption/tasks/main.yaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,13 @@
44
{{ oc_header }}
55
oc patch openstackcontrolplane openstack --type=merge --patch '{{ neutron_config_patch }}'
66
7+
- name: Patch neutron ml2MechanismDrivers to inlcude ml2 baremetal
8+
when: ironic_adoption|bool
9+
ansible.builtin.shell: |
10+
{{ shell_header }}
11+
{{ oc_header }}
12+
oc patch openstackcontrolplane openstack --type=merge --patch '{{ ironic_ml2_baremetal_patch }}'
13+
714
- name: wait for Neutron to start up
815
ansible.builtin.shell: |
916
{{ shell_header }}

0 commit comments

Comments
 (0)