Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove use of CtlplaneNetmask #1245

Merged

Conversation

bshephar
Copy link
Contributor

@bshephar bshephar commented Dec 23, 2024

This change removes use of CtlplaneNetmask in line with the changes proposed in:
openstack-k8s-operators/openstack-baremetal-operator#241

Jira: https://issues.redhat.com/browse/OSPRH-12686

This change removes use of CtlplaneNetmask in line with
the changes proposed in:
openstack-k8s-operators/openstack-baremetal-operator#241

Signed-off-by: Brendan Shephard <[email protected]>
@bshephar bshephar force-pushed the no-default-netmask branch from f39669a to f0a222c Compare January 3, 2025 04:33
Copy link

This change depends on a change that failed to merge.

Change openstack-k8s-operators/openstack-baremetal-operator#241 is needed.

Bumping the openstack-baremetal-operator version so that we can move
towards removal of unnecesasry and no-longer used parameters from the
operator.

Signed-off-by: Brendan Shephard <[email protected]>
@bshephar bshephar force-pushed the no-default-netmask branch from 9bccd15 to 913bdc5 Compare January 3, 2025 06:32
@bshephar
Copy link
Contributor Author

bshephar commented Jan 3, 2025

This change depends on a change that failed to merge.

Change openstack-k8s-operators/openstack-baremetal-operator#241 is needed.

This is no longer needed for this change to merge.

Copy link
Contributor

openshift-ci bot commented Jan 3, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bshephar, hjensas

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

The pull request process is described here

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 3cca2eb into openstack-k8s-operators:main Jan 3, 2025
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.

2 participants