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
change clusterrole to role #1728
Conversation
/assign @jlewi |
Would you mind adding some unittests as part of this PR to verify that the manifests are correct? |
Looks like you just need to format the jsonnet files. |
/retest |
1 similar comment
/retest |
Error:
This is fixed by #1747 please rebase and rerun the tests. |
Thanks 👍 |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jlewi 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 |
* change clusterrole to role * add unit test for centralui * run jsonnet fmt
* change clusterrole to role * add unit test for centralui * run jsonnet fmt
Addresses issue: #1450.
Assuming all kubeflow components are going to installed to a single namespace, CentralUI does not need cluster level scope. This PR is to restrict CentralUI's scope down to a single namespace.
This change is