-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Create cluster-admin rolebinding #569
Conversation
Jenkins job istio.github.io/presubmit passed |
Jenkins job istio.github.io/presubmit passed |
credentials for kubectl: | ||
2. Grant cluster admin permissions to the current user - admin permissions are required to create the necessary RBAC rules for Istio: | ||
```bash | ||
kubectl create clusterrolebinding cluster-admin-binding --clusterrole=cluster-admin --user=$(gcloud config get-value core/account) |
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.
this seems overly indented.. shouldn't it start from the same column as backtick?
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.
yup, the indentation is wrong everywhere.
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.
looks fine everywhere except here as far as I can see
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.
approved but why not fix the indent in the new green section - also this assumes you looked at it with jekyll
credentials for kubectl: | ||
2. Grant cluster admin permissions to the current user - admin permissions are required to create the necessary RBAC rules for Istio: | ||
```bash | ||
kubectl create clusterrolebinding cluster-admin-binding --clusterrole=cluster-admin --user=$(gcloud config get-value core/account) |
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.
looks fine everywhere except here as far as I can see
Jenkins job istio.github.io/presubmit passed |
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.
thx
Jenkins job istio.github.io/presubmit passed |
Jenkins job istio.github.io/presubmit passed |
merged so I can close |
Added instructions specific to GKE.