Skip to content

Commit 837012e

Browse files
authored
Merge pull request #6820 from zalando-incubator/deployment-key-old-alias
Allow deployment service role to decrypt any secret.
2 parents 984d15b + eaa43d7 commit 837012e

File tree

3 files changed

+4
-1
lines changed

3 files changed

+4
-1
lines changed

cluster/cluster.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -881,9 +881,11 @@ Resources:
881881
- !GetAtt DeploymentSecretKey.Arn
882882
{{- else }}
883883
Resource: "*"
884+
{{- if and (eq .Cluster.ConfigItems.deployment_secret_decrypt_any "false") (ne .Cluster.Environment "e2e") }}
884885
Condition:
885886
StringLike:
886887
"kms:RequestAlias": "alias/deployment-secret"
888+
{{- end }}
887889
{{- end }}
888890
- Action:
889891
- 'sts:AssumeRole'

cluster/config-defaults.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1036,3 +1036,4 @@ cronjob_time_zone_enabled: "true"
10361036
# or not. When set to a value != "true" the key will be removed from the stack.
10371037
# TODO: remove after migrating out of all cluster stacks.
10381038
deployment_secret_key_managed: "true"
1039+
deployment_secret_decrypt_any: "true"

cluster/manifests/deployment-service/controller-statefulset.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ spec:
3232
image: "container-registry.zalando.net/teapot/deployment-controller:master-163"
3333
args:
3434
- "--config-namespace=kube-system"
35-
{{- if ne .Cluster.ConfigItems.deployment_secret_key_managed "true" }}
35+
{{- if eq .Cluster.ConfigItems.deployment_secret_decrypt_any "false" }}
3636
- "--decrypt-kms-alias-arn=arn:aws:kms:{{ .Cluster.Region }}:{{ .Cluster.InfrastructureAccount | getAWSAccountID }}:alias/deployment-secret"
3737
{{- end }}
3838
env:

0 commit comments

Comments
 (0)