Skip to content

Change BGPConfiguration default namespace to openshift-frr-k8s and add legacy cleanup#592

Open
lmiccini wants to merge 1 commit into
openstack-k8s-operators:mainfrom
lmiccini:bgp-default-openshift-frr-k8s-namespace
Open

Change BGPConfiguration default namespace to openshift-frr-k8s and add legacy cleanup#592
lmiccini wants to merge 1 commit into
openstack-k8s-operators:mainfrom
lmiccini:bgp-default-openshift-frr-k8s-namespace

Conversation

@lmiccini
Copy link
Copy Markdown
Contributor

@lmiccini lmiccini commented Jun 2, 2026

The default FRRConfigurationNamespace is changed from metallb-system to openshift-frr-k8s to match the OpenShift 4.20+ FRR namespace. To ensure existing FRRConfigurations in the legacy metallb-system namespace are cleaned up on upgrade, the controller now explicitly cleans up owned FRRConfigurations from metallb-system in both reconcileNormal and reconcileDelete.

…d legacy cleanup

The default FRRConfigurationNamespace is changed from metallb-system to
openshift-frr-k8s to match the OpenShift 4.20+ FRR namespace. To ensure
existing FRRConfigurations in the legacy metallb-system namespace are
cleaned up on upgrade, the controller now explicitly cleans up owned
FRRConfigurations from metallb-system in both reconcileNormal and
reconcileDelete.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-ci openshift-ci Bot requested review from dciabrin and stuggi June 2, 2026 16:27
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 2, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lmiccini

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-ci openshift-ci Bot added the approved label Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant