-
Notifications
You must be signed in to change notification settings - Fork 49
Upgrade to Operator SDK 1.41.1 #788
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
Upgrade to Operator SDK 1.41.1 #788
Conversation
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/83fbd28a68f7456b8f3b5f2b6e1674f6 ❌ openstack-k8s-operators-content-provider FAILURE in 12m 16s |
25b6115 to
d22a3f0
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/8267cfd4cd7946ea8a9ec9f214875365 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 50m 03s |
|
/retest |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/f42a8d561f184f7a8d3cb4c7025202f4 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 51m 49s |
|
/retest |
|
those multiple Depends-On to one and another to the same project don't work. removing these for now: Depends-On: #789 |
|
/retest |
|
recheck |
|
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. |
d22a3f0 to
aa282c8
Compare
|
This change depends on a change that failed to merge. Change openstack-k8s-operators/openstack-operator#1683 is needed. |
aa282c8 to
b511207
Compare
This commit upgrades the telemetry-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-22020 Assisted-by: claude-4-sonnet Signed-off-by: Martin Schuppert <[email protected]>
b511207 to
0311e52
Compare
|
/retest |
telemetry-operator is special in the kuttl config compared to others. it defined test dirs in kuttl-test.yaml and uses an sed in install_yamls to change it if needed. Other operators pass an updated test path to the kubectl-kutt command in install_yamls. Because of that we * keep the old tests path in kuttl-test.yaml. * use a link tests -> test dir When a fix landed in install_yamls to be the same as other operators, we remove the link and update the kuttl-test.yaml. Signed-off-by: Martin Schuppert <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a temporary workaround which will be removed in a later follow-up
abays
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abays, 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 |
bf35154
into
openstack-k8s-operators:main
This commit upgrades the telemetry-operator to use Operator SDK version 1.41.1, with the following major changes:
Project Structure:
Configuration Updates:
Webhook Changes:
Cleanup:
Dependencies:
Jira: OSPRH-22020
Assisted-by: claude-4-sonnet
Depends-On: openstack-k8s-operators/openstack-operator#1683
Depends-On: openstack-k8s-operators/install_yamls#1110