Skip to content

Conversation

@stuggi
Copy link
Contributor

@stuggi stuggi commented Nov 17, 2025

This commit upgrades the placement-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-21984

Assisted-by: claude-4-sonnet

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

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 17, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: stuggi

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

@softwarefactory-project-zuul
Copy link

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/openstack-k8s-operators/openstack-operator for 1683,19db487cf23204e39d5f67be71ad37d8a1632c92

@stuggi
Copy link
Contributor Author

stuggi commented Nov 17, 2025

recheck

@stuggi
Copy link
Contributor Author

stuggi commented Nov 17, 2025

/retest

3 similar comments
@stuggi
Copy link
Contributor Author

stuggi commented Nov 17, 2025

/retest

@stuggi
Copy link
Contributor Author

stuggi commented Nov 18, 2025

/retest

@stuggi
Copy link
Contributor Author

stuggi commented Nov 19, 2025

/retest

@stuggi stuggi force-pushed the operator_sdk_1.41.1 branch from 0c1214a to e3b398e Compare November 20, 2025 10:46
This commit upgrades the placement-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-21984

Assisted-by: claude-4-sonnet
Signed-off-by: Martin Schuppert <[email protected]>
@stuggi stuggi force-pushed the operator_sdk_1.41.1 branch from e3b398e to e5cfd41 Compare November 21, 2025 13:02
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.

1 participant