Skip to content

Enable multi-namespace support#91

Merged
openshift-merge-bot[bot] merged 1 commit intoopenstack-lightspeed:mainfrom
lpiwowar:lpiwowar/multi-namespace
Mar 5, 2026
Merged

Enable multi-namespace support#91
openshift-merge-bot[bot] merged 1 commit intoopenstack-lightspeed:mainfrom
lpiwowar:lpiwowar/multi-namespace

Conversation

@lpiwowar
Copy link
Contributor

@lpiwowar lpiwowar commented Mar 5, 2026

The operator needs to access resources in the openstack namespace while running in the openshift-lightspeed namespace. This enables the operator to be deployed alongside OpenShift Lightspeed operator in its default namespace while managing OpenStack resources in a separate namespace.

Note that OpenShift Lightspeed does not behave properly when in a different namespace. We can revisit the namespace where our operator gets deployed once we migrate to Lightspeed Core.

@openshift-ci
Copy link

openshift-ci bot commented Mar 5, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci openshift-ci bot added the approved label Mar 5, 2026
@lpiwowar
Copy link
Contributor Author

lpiwowar commented Mar 5, 2026

/test all

@lpiwowar lpiwowar marked this pull request as ready for review March 5, 2026 09:12
@openshift-ci openshift-ci bot requested review from Akrog and umago March 5, 2026 09:12
@lpiwowar lpiwowar force-pushed the lpiwowar/multi-namespace branch from dd6cd4b to 02fc25a Compare March 5, 2026 10:40
Copy link
Contributor

@umago umago left a comment

Choose a reason for hiding this comment

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

Looks good overall, two comments but not blockers

The operator needs to access resources in the openstack namespace
while running in the openshift-lightspeed namespace. This enables
the operator to be deployed alongside OpenShift Lightspeed
operator in its default namespace while managing OpenStack
resources in a separate namespace.

Note that OpenShift Lightspeed does not behave properly when in
a different namespace. We can revisit the namespace where our operator
gets deployed once we migrate to Lightspeed Core.
@lpiwowar lpiwowar force-pushed the lpiwowar/multi-namespace branch from 02fc25a to 38af111 Compare March 5, 2026 11:37
Copy link
Contributor

@Akrog Akrog left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Mar 5, 2026
@openshift-ci
Copy link

openshift-ci bot commented Mar 5, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Akrog, lpiwowar

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

@openshift-merge-bot openshift-merge-bot bot merged commit 37f1651 into openstack-lightspeed:main Mar 5, 2026
7 checks passed
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.

3 participants