Skip to content

Add Octavia SKMO playbooks for amphora and network setup#3965

Open
vakwetu wants to merge 1 commit into
openstack-k8s-operators:mainfrom
vakwetu:add-octavia-amphora-wait-skmo
Open

Add Octavia SKMO playbooks for amphora and network setup#3965
vakwetu wants to merge 1 commit into
openstack-k8s-operators:mainfrom
vakwetu:add-octavia-amphora-wait-skmo

Conversation

@vakwetu
Copy link
Copy Markdown
Contributor

@vakwetu vakwetu commented May 26, 2026

Add two new SKMO hook playbooks for Octavia multi-region support:

  • upload-octavia-amphora-images.yaml: Wait for amphora image upload to complete in both regionOne and regionTwo before proceeding
  • configure-octavia-network.yaml: Ensure br-octavia OVS bridges are UP on all master nodes for routing between worker pods and amphora instances

These fix connectivity issues where Neutron router gateway ports fail to schedule on chassis due to missing active bridge mappings.

Assisted-by: Claude Opus 4.6
Signed-off-by: Ade Lee alee@redhat.com

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 26, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign brjackma for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@evallesp
Copy link
Copy Markdown
Contributor

First commit already reviewed in #3956
Do we want to close that MR? I'd rather go removing that commit from this MR.

Copy link
Copy Markdown
Contributor

@evallesp evallesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general LGTM

Comment thread hooks/playbooks/skmo/configure-octavia-network.yaml Outdated
Comment thread hooks/playbooks/skmo/configure-octavia-network.yaml Outdated
set -xe -o pipefail
oc debug node/{{ item }} -- chroot /host ip link set br-octavia up
loop: "{{ cifmw_skmo_master_nodes }}"
changed_when: false
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(blocking) suggestion: I think it's safe to remove this changed_when.

@evallesp
Copy link
Copy Markdown
Contributor

First commit already reviewed in #3956 Do we want to close that MR? I'd rather go removing that commit from this MR.

it was merged.

Add two new SKMO hook playbooks for Octavia multi-region support:

- upload-octavia-amphora-images.yaml: Wait for amphora image upload
  to complete in both regionOne and regionTwo before proceeding
- configure-octavia-network.yaml: Ensure br-octavia OVS bridges are
  UP on all master nodes for routing between worker pods and amphora
  instances

These fix connectivity issues where Neutron router gateway ports fail
to schedule on chassis due to missing active bridge mappings.

Assisted-by: Claude Opus 4.6
Signed-off-by: Ade Lee <alee@redhat.com>
@vakwetu vakwetu force-pushed the add-octavia-amphora-wait-skmo branch from e1bc8e2 to e1304ee Compare May 28, 2026 19:51
@vakwetu
Copy link
Copy Markdown
Contributor Author

vakwetu commented May 28, 2026

removed merged commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants