Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

BorisShek
Copy link
Contributor

@BorisShek BorisShek commented Apr 24, 2025

Rename the ClusterRole 'manager-role' and ClusterRoleBinding 'manager-rolebinding' to 'securecodebox-manager-role' and 'securecodebox-manager-rolebinding', respectively, because the original names are too generic and could collide with other projects in the same cluster.

Description

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

Copy link

netlify bot commented Apr 24, 2025

Deploy Preview for docs-securecodebox ready!

Name Link
🔨 Latest commit 0039b48
🔍 Latest deploy log https://app.netlify.com/sites/docs-securecodebox/deploys/680a54d2e8814500086304f9
😎 Deploy Preview https://deploy-preview-3002--docs-securecodebox.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@BorisShek BorisShek linked an issue Apr 24, 2025 that may be closed by this pull request
@BorisShek BorisShek added bug Bugs documentation Improvements or additions to documentation breaking Changes requiring a major release labels Apr 24, 2025
@BorisShek BorisShek self-assigned this Apr 24, 2025
@secureCodeBoxBot
Copy link
Contributor

This pull request includes breaking changes. Please make sure that you included the breaking changes and the steps required to upgrade in Upgrading.
✨ Thank you for your contribution! ✨

@BorisShek BorisShek force-pushed the scb-2621_rename_clusterroles_and_clusterrolebindings branch 2 times, most recently from 3abe5ed to 1de62fe Compare April 24, 2025 14:56
Rename the ClusterRole 'manager-role' and ClusterRoleBinding 'manager-rolebinding' to 'securecodebox-manager-role' and 'securecodebox-manager-rolebinding', respectively, because the original names are too generic and could collide with other projects in the same cluster.

Signed-off-by: Boris Shek <[email protected]>
@BorisShek BorisShek force-pushed the scb-2621_rename_clusterroles_and_clusterrolebindings branch from 1de62fe to 632942e Compare April 24, 2025 15:03
…g in upgrading.md

Add a "From 4.X to 5.X" section in upgrading.md.
Add an explanation that the operator’s ClusterRole
'manager-role' and ClusterRoleBinding 'manager-rolebinding' have been renamed to 'securecodebox-manager-role' and 'securecodebox-manager-rolebinding'.

Signed-off-by: Boris Shek <[email protected]>
@BorisShek BorisShek moved this from Triage to To Review in secureCodeBox Apr 24, 2025
@BorisShek BorisShek marked this pull request as ready for review April 24, 2025 15:13
Copy link

@BorisShek
Copy link
Contributor Author

Resolves #2621

@J12934
Copy link
Member

J12934 commented Apr 25, 2025

looks good :)
will hold back merging this until one last v4 feature release, but code wise this is ready to go from my view :)

@github-project-automation github-project-automation bot moved this from To Review to Reviewer Approved in secureCodeBox Apr 29, 2025
@J12934 J12934 merged commit 9543d78 into secureCodeBox:main Apr 29, 2025
53 checks passed
@github-project-automation github-project-automation bot moved this from Reviewer Approved to Done in secureCodeBox Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Changes requiring a major release bug Bugs documentation Improvements or additions to documentation
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

ClusterRoles and ClusterRoleBindings: metadata.name too common
3 participants