Skip to content

Add octavia service to SKMO central and leaf regions#759

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

Add octavia service to SKMO central and leaf regions#759
vakwetu wants to merge 1 commit into
openstack-k8s-operators:mainfrom
vakwetu:add-octavia-to-skmo

Conversation

@vakwetu
Copy link
Copy Markdown
Contributor

@vakwetu vakwetu commented May 14, 2026

Enable octavia (Load Balancing as a Service) in both control-plane (central region) and control-plane2 (leaf region) for SKMO deployment.

Changes:

  • Add octavia service configuration to service-values.yaml using minimal flat pattern (similar to osasinfra)
  • Create SKMO-specific networking directories with octavia network definitions (vlan 23, bridge mode, 172.23.0.0/24)
  • Add octavia-ca-passphrase secret generator
  • Add octavia NetworkAttachmentDefinition resources
  • Add kustomization replacements for octavia service configuration (enabled, container images, networkAttachments for all components)
  • Add kustomization replacement for octavia network-attach-def
  • Update automation vars to use SKMO-specific network values that merge base networks with octavia via kustomize composition

SKMO networking structure uses kustomize composition to avoid duplication: base multi-namespace network values are included as resources, with SKMO-specific values.yaml adding only octavia config.

Assisted-by: Claude Sonnet 4.5
Depends-On: openstack-k8s-operators/ci-framework#3965
Signed-off-by: Ade Lee alee@redhat.com

@openshift-ci openshift-ci Bot requested review from abays and leifmadsen May 14, 2026 23:23
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented May 14, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: vakwetu
Once this PR has been reviewed and has the lgtm label, please assign abays 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

@vakwetu vakwetu requested review from fultonj and removed request for abays and leifmadsen May 14, 2026 23:23
@vakwetu vakwetu requested review from beagles and gthiemonge May 14, 2026 23:26
@vakwetu
Copy link
Copy Markdown
Contributor Author

vakwetu commented May 14, 2026

hey @beagles and @gthiemonge . I'm adding octavia to the skmo ci job. Still testing - but can you confirm I've done everything I need to do. Thanks.

@vakwetu vakwetu force-pushed the add-octavia-to-skmo branch from 19143ae to 424a81b Compare May 26, 2026 18:28
Add Octavia load balancer service to both central (regionOne) and leaf
(regionTwo) control planes with full kustomize integration:

- Add octavia service configuration to both region service-values
- Add OVN bridge mapping (octavia: octbr) for amphora connectivity
- Add kustomize replacements for octavia and OVN nicMappings
- Add automation hooks to wait for amphora image upload and
  configure br-octavia network infrastructure on master nodes

Assisted-by: Claude Opus 4.6
Signed-off-by: Ade Lee <alee@redhat.com>
@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

This change depends on a change that failed to merge.

Change openstack-k8s-operators/ci-framework#3965 is needed.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant