Skip to content

Commit

Permalink
fix: serviceAccount must be specified at pod spec level in dual stack…
Browse files Browse the repository at this point in the history
… goldpinger deployments (#2208)

service account needs to be specified at the pod spec level, not the container
  • Loading branch information
ramiro-gamarra authored Sep 8, 2023
1 parent 61da686 commit c281674
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ spec:
labels:
app: goldpinger
spec:
serviceAccount: goldpinger-serviceaccount
containers:
- name: goldpinger
env:
Expand Down Expand Up @@ -41,7 +42,6 @@ spec:
- name: HOSTS_TO_RESOLVE
value: "2001:4860:4860::8888 www.bing.com"
image: "docker.io/bloomberg/goldpinger:v3.7.0"
serviceAccount: goldpinger-serviceaccount
tolerations:
- key: node-role.kubernetes.io/master
effect: NoSchedule
Expand Down
2 changes: 1 addition & 1 deletion test/integration/manifests/datapath/linux-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ spec:
labels:
app: goldpinger
spec:
serviceAccount: goldpinger-serviceaccount
containers:
- name: goldpinger
env:
Expand All @@ -39,7 +40,6 @@ spec:
- name: HOSTS_TO_RESOLVE
value: "1.1.1.1 8.8.8.8 www.bing.com"
image: "docker.io/bloomberg/goldpinger:v3.7.0"
serviceAccount: goldpinger-serviceaccount
tolerations:
- key: node-role.kubernetes.io/master
effect: NoSchedule
Expand Down

0 comments on commit c281674

Please sign in to comment.