← 输入框|github.com|刷新
Skip to content

ClusterRoles and ClusterRoleBindings: metadata.name too common #2621

Closed

Description

@LittaKake

➹ New Feature implementation request

A minor detail to fix possible nameconflict in non-namespaced resources.

Is your feature request related to a problem?

ClusterRoles and ClusterRolesBindings are not bound to any namespace. The metadata.name (manager-role and manager-rolebinding) of ClusterRoles and ClusterRolesBindings are too common and could cause nameconflicts. (happened in my cluster with another project)

Describe the solution you'd like

Change the names to securecodebox-manager-role and securecodebox-manager-rolebinding.

Describe alternatives you've considered

  1. A solution is to use kustomize to deploy the helm chart and patch the respective fields. However, this is overly complicated when the name can just be initialy fixed.

Additional context

Activity

  1. J12934 commented on Aug 22, 2024

    @J12934
    Member

    Hi @DiiBBz
    Thanks for opening the issue.
    Yes, agreed the naming of these roles have been a oversight when initially setting up the project.

    I was planning on renaming this whith the next major release (5.0.0) which we are planning to have release in a couple of month. Renaming them itself shouldn't be breaking but I would still prefer to mark this as a thing to be aware of while updating in case somebody started using the roles for something else (Which i hope they don't but with that common of a name for a cluster role its hard to rule out)

  2. added
    breakingChanges requiring a major release
    and removed
    enhancementNew feature or request
    on Aug 22, 2024
  3. moved this from Backlog to Todo in secureCodeBoxon Aug 22, 2024
  4. added this to the v5.0.0 milestone on Feb 6, 2025
  5. moved this from Backlog to Todo in secureCodeBoxon Feb 12, 2025
  6. moved this from Todo to In Progress in secureCodeBoxon Apr 13, 2025
  7. self-assigned this
    on Apr 13, 2025
  8. moved this from In Progress to Todo in secureCodeBoxon Apr 13, 2025
  9. removed their assignment
    on Apr 13, 2025
  10. moved this from Todo to In Progress in secureCodeBoxon Apr 14, 2025
  11. self-assigned this
    on Apr 14, 2025
  12. linked a pull request that will close this issueRename ClusterRole and ClusterRoleBinding #3002on Apr 24, 2025
  13. added 4 commits that reference this issue on Apr 24, 2025
  14. added a commit that references this issue on Apr 29, 2025
  15. added a commit that references this issue on Apr 29, 2025
  16. moved this from In Progress to Done in secureCodeBoxon Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

breakingChanges requiring a major releasebugBugs

Type

No type

Projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions