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
Add ClusterRole and ClusterRole #2684
Conversation
/cc @avdaredevil |
/retest |
cc: @kunmingg (could you also take a look?) |
/approve |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, asked a Kubeflow team member review.
app: "centraldashboard", | ||
}, | ||
name: "centraldashboard", | ||
namespace: params.namespace, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't need a namespace for ClusterRole as it's scope cross namespaces.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change made
app: "centraldashboard", | ||
}, | ||
name: "centraldashboard", | ||
namespace: params.namespace, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change made
Remove namespace from ClusterRole and ClusterRoleBinding
Changes made. PTAL. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: avdaredevil, kunmingg 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 |
* Add ClusterRole and ClusterRole binding to allow Namespace and Events listing * Remove namespace Remove namespace from ClusterRole and ClusterRoleBinding
* Add ClusterRole and ClusterRole binding to allow Namespace and Events listing * Remove namespace Remove namespace from ClusterRole and ClusterRoleBinding
This enables the Central Dashboard to list namespaces and also retrieve events from those namespaces to display on the Activities tab.
This change is