Skip to content

Commit

Permalink
install/kubernetes: restrict k8s access for relay
Browse files Browse the repository at this point in the history
It seems relay has no interaction with Kubernetes, and as such, it does
not need (cluster)roles/-bindings or a mounted SA token. This prevents
attackers from doing nasty stuff on behalf of relay.

Fixes: e9cb43c ("Helm: full refactor of helm charts..")

Signed-off-by: Jorik Jonker <jorik.jonker@eu.equinix.com>
  • Loading branch information
jonkerj authored and brb committed Jul 22, 2021
1 parent a615288 commit 2857b3e
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 34 deletions.
20 changes: 0 additions & 20 deletions install/kubernetes/cilium/templates/hubble-relay/clusterrole.yaml

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ spec:
restartPolicy: Always
serviceAccount: {{ .Values.serviceAccounts.relay.name | quote }}
serviceAccountName: {{ .Values.serviceAccounts.relay.name | quote }}
automountServiceAccountToken: false
terminationGracePeriodSeconds: 0
{{- with .Values.hubble.relay.nodeSelector }}
nodeSelector:
Expand Down

0 comments on commit 2857b3e

Please sign in to comment.