Skip to content

🐛 remove unused RBAC permissions for secrets#1356

Merged
metal3-io-bot merged 1 commit into
metal3-io:release-1.12from
Nordix:tuomo/manual-cherrypick-1355-release-1.12
Apr 1, 2026
Merged

🐛 remove unused RBAC permissions for secrets#1356
metal3-io-bot merged 1 commit into
metal3-io:release-1.12from
Nordix:tuomo/manual-cherrypick-1355-release-1.12

Conversation

@tuminoid

Copy link
Copy Markdown
Member

Remove the kubebuilder RBAC marker granting full CRUD on secrets. The controller never accesses secrets. Also remove the dead secret cache configuration and stale secretCachingClient comment from main.go.

Manual cherry-pick of PR #1355.

Remove the kubebuilder RBAC marker granting full CRUD on secrets. The
controller never accesses secrets. Also remove the dead secret cache
configuration and stale secretCachingClient comment from main.go.

Manual cherry-pick of PR#1355.

Signed-off-by: Tuomo Tanskanen <tuomo.tanskanen@est.tech>
@tuminoid

Copy link
Copy Markdown
Member Author

/test metal3-centos-e2e-integration-test-release-1-12

@metal3-io-bot metal3-io-bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Mar 31, 2026

@lentzi90 lentzi90 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@metal3-io-bot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lentzi90

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

@metal3-io-bot metal3-io-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 31, 2026

@Sunnatillo Sunnatillo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@metal3-io-bot metal3-io-bot added the lgtm Indicates that a PR is ready to be merged. label Apr 1, 2026
@metal3-io-bot metal3-io-bot merged commit ed6571e into metal3-io:release-1.12 Apr 1, 2026
16 checks passed
@metal3-io-bot metal3-io-bot deleted the tuomo/manual-cherrypick-1355-release-1.12 branch April 1, 2026 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants