Skip to content

[skmo] Add Skupper for cross-region RabbitMQ and Keystone#734

Merged
softwarefactory-project-zuul[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
vakwetu:skmo-skupper-keystone
Apr 23, 2026
Merged

[skmo] Add Skupper for cross-region RabbitMQ and Keystone#734
softwarefactory-project-zuul[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
vakwetu:skmo-skupper-keystone

Conversation

@vakwetu
Copy link
Copy Markdown
Contributor

@vakwetu vakwetu commented Apr 7, 2026

[skmo] Add Skupper for cross-region RabbitMQ and Keystone internal routing

Add automation variables and kustomize configuration to establish Skupper
virtual services for RabbitMQ and Keystone internal endpoints, enabling
cross-region connectivity in the multi-namespace SKMO scenario.

The RabbitMQ Skupper connector routes barbican-keystone-listener traffic
from the leaf (openstack2) namespace to the central (openstack) RabbitMQ
over an mTLS tunnel, avoiding exposure on the public network.

The Keystone Skupper connector routes internal service-to-service
authentication traffic from leaf region services to the central Keystone
endpoint, replacing the previous approach of using the public Keystone URL
for internal traffic.

To make the Skupper Keystone virtual service reachable from EDPM compute
nodes (which run outside the OCP cluster and cannot reach ClusterIP
services), a MetalLB LoadBalancer Service and a DNSData CR are created
alongside the Skupper Listener. This ensures nova-compute can resolve
and connect to the Keystone auth_url on startup.

Also adds skupper-keystone-internal.md documenting the full procedure,
including the EDPM DNS workaround and the rationale for each step.

Depends-On: openstack-k8s-operators/ci-framework#3836

@openshift-ci openshift-ci Bot requested review from abays and raukadah April 7, 2026 18:58
@vakwetu vakwetu force-pushed the skmo-skupper-keystone branch 2 times, most recently from efdd056 to d886542 Compare April 9, 2026 18:56
@vakwetu vakwetu requested a review from a team as a code owner April 9, 2026 18:56
@vakwetu vakwetu changed the title [skmo] Add Skupper for cross-region RabbitMQ and Keystone internal ro… [skmo] Add Skupper for cross-region RabbitMQ and Keystone Apr 9, 2026
@vakwetu vakwetu assigned vakwetu and unassigned vakwetu Apr 9, 2026
@vakwetu vakwetu requested a review from dmendiza April 9, 2026 19:19
@vakwetu vakwetu force-pushed the skmo-skupper-keystone branch 3 times, most recently from 556b5d3 to 1b254a8 Compare April 10, 2026 17:55
@vakwetu vakwetu added ready-review Request is ready to be reviewed do-not-merge/work-in-progress and removed do-not-merge/work-in-progress ready-review Request is ready to be reviewed labels Apr 10, 2026
@vakwetu vakwetu force-pushed the skmo-skupper-keystone branch from 1b254a8 to 3103049 Compare April 14, 2026 16:56
@vakwetu vakwetu added ready-review Request is ready to be reviewed and removed do-not-merge/work-in-progress labels Apr 15, 2026
@vakwetu
Copy link
Copy Markdown
Contributor Author

vakwetu commented Apr 15, 2026

recheck

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.

/lgtm

* Skupper is installed and Sites are linked between the central and workload
namespaces. If you are also routing RabbitMQ traffic through Skupper (for
`barbican-keystone-listener`), the Site link is already in place. See the
[Skupper installation and site-link guide](../skupper-install.md) if you are
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.

Where is this skupper-install.md file? In another PR?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added in new commit

@abays abays mentioned this pull request Apr 16, 2026
@vakwetu vakwetu force-pushed the skmo-skupper-keystone branch from 3103049 to 16542fe Compare April 16, 2026 19:27
@openshift-ci openshift-ci Bot added needs-rebase and removed lgtm labels Apr 16, 2026
@vakwetu vakwetu requested a review from danpawlik April 21, 2026 18:33
…uting

Add automation variables and kustomize configuration to establish Skupper
virtual services for RabbitMQ and Keystone internal endpoints, enabling
cross-region connectivity in the multi-namespace SKMO scenario.

The RabbitMQ Skupper connector routes barbican-keystone-listener traffic
from the leaf (openstack2) namespace to the central (openstack) RabbitMQ
over an mTLS tunnel, avoiding exposure on the public network.

The Keystone Skupper connector routes internal service-to-service
authentication traffic from leaf region services to the central Keystone
endpoint, replacing the previous approach of using the public Keystone URL
for internal traffic.

To make the Skupper Keystone virtual service reachable from EDPM compute
nodes (which run outside the OCP cluster and cannot reach ClusterIP
services), a MetalLB LoadBalancer Service and a DNSData CR are created
alongside the Skupper Listener. This ensures nova-compute can resolve
and connect to the Keystone auth_url on startup.

Also adds skupper-keystone-internal.md documenting the full procedure,
including the EDPM DNS workaround and the rationale for each step.

Signed-off-by: Ade Lee <alee@redhat.com>
Co-authored-by: Claude <noreply@anthropic.com>
Made-with: Cursor
@vakwetu vakwetu force-pushed the skmo-skupper-keystone branch from 16542fe to 80b5de0 Compare April 21, 2026 22:02
Copy link
Copy Markdown
Contributor

@fultonj fultonj left a comment

Choose a reason for hiding this comment

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

/approve
/lgtm

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 23, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: danpawlik, evallesp, fultonj, vakwetu

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

The pull request process is described 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

@softwarefactory-project-zuul
Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul softwarefactory-project-zuul Bot merged commit 1856082 into openstack-k8s-operators:main Apr 23, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved lgtm ready-review Request is ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants