Skip to content

Upgrade to Operator SDK 1.41.1#415

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
stuggi:operator_sdk_1.41.1
Nov 24, 2025
Merged

Upgrade to Operator SDK 1.41.1#415
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
stuggi:operator_sdk_1.41.1

Conversation

@stuggi

@stuggi stuggi commented Nov 17, 2025

Copy link
Copy Markdown
Contributor

This commit upgrades the ovn-operator to use Operator SDK version 1.41.1, with the following major changes:

Project Structure:

  • Move pkg/ packages to internal/ following Go best practices
  • Move controllers/ to internal/controller/ per new operator-sdk layout
  • Replace main.go with cmd/main.go
  • Add controller and webhook test suites

Configuration Updates:

  • Split certificate management into separate metrics and webhook certs
  • Add network policies for metrics and webhook traffic
  • Rename auth_proxy resources to metrics for clarity
  • Add admin RBAC roles for all CRD types
  • Update manager deployment configuration and patches
  • Modernize kustomization configurations

Webhook Changes:

  • Add dedicated webhook implementation in internal/webhook/v1beta1/
  • Remove old CRD webhook and CA injection patches
  • Update webhook manifests and service configuration

Cleanup:

  • Remove obsolete config patches and overlays

Dependencies:

  • Update go.mod and go.sum for operator-sdk 1.41.1 dependencies

Jira: OSPRH-22013

Assisted-by: claude-4-sonnet

Depends-On: openstack-k8s-operators/openstack-operator#1683

@stuggi

stuggi commented Nov 17, 2025

Copy link
Copy Markdown
Contributor Author

/retest

2 similar comments
@stuggi

stuggi commented Nov 17, 2025

Copy link
Copy Markdown
Contributor Author

/retest

@stuggi

stuggi commented Nov 18, 2025

Copy link
Copy Markdown
Contributor Author

/retest

@cschwede

Copy link
Copy Markdown
Collaborator

/retest

failed to wait for the created cluster claim to become ready: timed out

@stuggi

stuggi commented Nov 19, 2025

Copy link
Copy Markdown
Contributor Author

/retest

@stuggi

stuggi commented Nov 19, 2025

Copy link
Copy Markdown
Contributor Author

missed to add the Depends-On to the description :(

@stuggi

stuggi commented Nov 19, 2025

Copy link
Copy Markdown
Contributor Author

/retest

@cschwede

Copy link
Copy Markdown
Collaborator

/test swift-operator-build-deploy-kuttl

Timeout on Galera, looked like some network issues in the background

@cschwede

Copy link
Copy Markdown
Collaborator

Hmm, again failed with " error: timed out waiting for the condition on openstackcontrolplanes/openstack-galera "

@stuggi stuggi force-pushed the operator_sdk_1.41.1 branch from f5ffec1 to 25b6fbb Compare November 20, 2025 14:23
This commit upgrades the ovn-operator to use Operator SDK version 1.41.1,
with the following major changes:

Project Structure:
- Move pkg/ packages to internal/ following Go best practices
- Move controllers/ to internal/controller/ per new operator-sdk layout
- Replace main.go with cmd/main.go
- Add controller and webhook test suites

Configuration Updates:
- Split certificate management into separate metrics and webhook certs
- Add network policies for metrics and webhook traffic
- Rename auth_proxy resources to metrics for clarity
- Add admin RBAC roles for all CRD types
- Update manager deployment configuration and patches
- Modernize kustomization configurations

Webhook Changes:
- Add dedicated webhook implementation in internal/webhook/v1beta1/
- Remove old CRD webhook and CA injection patches
- Update webhook manifests and service configuration

Cleanup:
- Remove obsolete config patches and overlays

Dependencies:
- Update go.mod and go.sum for operator-sdk 1.41.1 dependencies

Jira: OSPRH-22013

Assisted-by: claude-4-sonnet
Signed-off-by: Martin Schuppert <mschuppert@redhat.com>
@stuggi stuggi force-pushed the operator_sdk_1.41.1 branch from 25b6fbb to cce65c5 Compare November 21, 2025 13:03
@stuggi

stuggi commented Nov 21, 2025

Copy link
Copy Markdown
Contributor Author

/retest

1 similar comment
@stuggi

stuggi commented Nov 21, 2025

Copy link
Copy Markdown
Contributor Author

/retest

@abays abays left a comment

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.

/lgtm

@openshift-ci

openshift-ci Bot commented Nov 24, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, dprince, stuggi

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:
  • OWNERS [abays,dprince,stuggi]

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 d8f4061 into openstack-k8s-operators:main Nov 24, 2025
5 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.

4 participants