Skip to content
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

[Bug] KubeRay tries to create ClusterRoleBinding when singleNamespaceInstall and rbacEnable are set to true #1190

Conversation

kevin85421
Copy link
Member

@kevin85421 kevin85421 commented Jun 27, 2023

Why are these changes needed?

  • RBAC files

    • leader_election_role.yaml / leader_election_rolebinding.yaml => Role / RoleBinding for leader election
    • role.yaml / rolebinding.yaml => ClusterRole / ClusterRoleBinding for KubeRay operator to create Kubernetes resources for custom resources
    • multiple_namespaces_role.yaml / multiple_namespaces_rolebinding.yaml => Role / RoleBinding in multiple namespaces for KubeRay operator to create Kubernetes resources for custom resources
  • Test: Firstly, I wanted to try https://github.com/helm-unittest/helm-unittest, but it turned out to be not as versatile as expected.

Related issue number

Closes #1186

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

@kevin85421 kevin85421 marked this pull request as ready for review June 27, 2023 06:32
@kevin85421
Copy link
Member Author

cc @anshulomar would you mind reviewing this PR?

@anshulomar
Copy link

cc @anshulomar would you mind reviewing this PR?

Thanks @kevin85421 . Changes look good to me. Approved.

@kevin85421 kevin85421 merged commit 7aea947 into ray-project:master Jun 27, 2023
20 checks passed
lowang-bh pushed a commit to lowang-bh/kuberay that referenced this pull request Sep 24, 2023
…Install and rbacEnable are set to true (ray-project#1190)

KubeRay tries to create ClusterRoleBinding when singleNamespaceInstall and rbacEnable are set to true
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] KubeRay tries to create ClusterRoleBinding when singleNamespaceInstall and rbacEnable are set to true
4 participants