-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[server] Add kube-rbac-proxy rolebinding to SA #4745
Conversation
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.
Is this supposed to fix agent-smith or server?
If it is agent-smith then you changed the wrong file 😬
kind: ClusterRoleBinding | ||
apiVersion: rbac.authorization.k8s.io/v1 | ||
metadata: | ||
name: {{ .Release.Namespace }}-agent-smith-rb-kube-rbac-proxy |
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.
name: {{ .Release.Namespace }}-agent-smith-rb-kube-rbac-proxy | |
name: {{ .Release.Namespace }}-server-rb-kube-rbac-proxy |
name: {{ .Release.Namespace }}-agent-smith-rb-kube-rbac-proxy | ||
labels: | ||
app: {{ template "gitpod.fullname" . }} | ||
component: agent-smith |
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.
component: agent-smith | |
component: server |
stage: {{ .Values.installation.stage }} | ||
subjects: | ||
- kind: ServiceAccount | ||
name: agent-smith |
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.
name: agent-smith | |
name: server |
Good catch - I was a bit too quick here. Will fix |
98b0bd6
to
9338461
Compare
Server metrics scraping is broken because of a missing role binding:
fixes #4739
/werft no-preview