Skip to content

Commit 64f2ff2

Browse files
authored
Merge pull request #9478 from zalando-incubator/dev-to-alpha
dev to alpha
2 parents a703e69 + 8bda3f7 commit 64f2ff2

File tree

3 files changed

+29
-8
lines changed

3 files changed

+29
-8
lines changed

cluster/config-defaults.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1270,6 +1270,8 @@ eks_zalando_iam_aws_proxy_hpa_max_replicas: "10"
12701270
eks_zalando_iam_aws_proxy_hpa_cpu_target: "80"
12711271
eks_zalando_iam_aws_proxy_hpa_memory_target: "80"
12721272
eks_okta_identity_provider: "true"
1273+
eks_legacy_cluster_local_id: "kube-1"
1274+
eks_oidc_issuer_url: "https://"
12731275
eks_fis_support_enabled: "false"
12741276
eks_fis_namespaces: "default"
12751277

cluster/manifests/deployment-service/01-config.yaml

Lines changed: 26 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -13,17 +13,35 @@ data:
1313
create-namespaces: "true"
1414
aws-available: "true"
1515
worker-role-arn: "arn:aws:iam::{{accountID .Cluster.InfrastructureAccount}}:role/{{.Cluster.LocalID}}-worker"
16-
{{ $oidc_issuer := "" }}
1716
{{- if eq .Cluster.Provider "zalando-eks" }}
18-
{{ $oidc_issuer = index (split .Cluster.ConfigItems.eks_oidc_issuer_url "//") 1 }}
17+
{{ $oidc_issuer_aws := printf "%s.%s" .Cluster.ConfigItems.eks_legacy_cluster_local_id .Values.hosted_zone }}
18+
{{ $oidc_issuer_eks := index (split .Cluster.ConfigItems.eks_oidc_issuer_url "//") 1 }}
19+
{{ $oidc_provider_arn_aws := printf "arn:aws:iam::%s:oidc-provider/%s" (accountID .Cluster.InfrastructureAccount) $oidc_issuer_aws }}
20+
{{ $oidc_provider_arn_eks := printf "arn:aws:iam::%s:oidc-provider/%s" (accountID .Cluster.InfrastructureAccount) $oidc_issuer_eks }}
21+
{{ $oidc_subject_key_aws := printf "%s:sub" $oidc_issuer_aws }}
22+
{{ $oidc_subject_key_eks := printf "%s:sub" $oidc_issuer_eks }}
23+
oidc-provider-arn: "{{$oidc_provider_arn_eks}}"
24+
oidc-subject-key: "{{$oidc_subject_key_eks}}"
25+
{{- if ne .Cluster.ConfigItems.eks_legacy_cluster_local_id "" }}
26+
oidc-trust-relationship-template: '{"Version":"2012-10-17","Statement":[{"Effect":"Allow","Principal":{"Federated":"{{$oidc_provider_arn_aws}}"},"Action":"sts:AssumeRoleWithWebIdentity","Condition":{"StringLike":{"{{$oidc_subject_key_aws}}":"system:serviceaccount:${SERVICE_ACCOUNT}"}}},{"Effect":"Allow","Principal":{"Federated":"{{$oidc_provider_arn_eks}}"},"Action":"sts:AssumeRoleWithWebIdentity","Condition":{"StringLike":{"{{$oidc_subject_key_eks}}":"system:serviceaccount:${SERVICE_ACCOUNT}"}}}]}'
27+
{{- else }}
28+
oidc-trust-relationship-template: '{"Version":"2012-10-17","Statement":[{"Effect":"Allow","Principal":{"Federated":"{{$oidc_provider_arn_eks}}"},"Action":"sts:AssumeRoleWithWebIdentity","Condition":{"StringLike":{"{{$oidc_subject_key_eks}}":"system:serviceaccount:${SERVICE_ACCOUNT}"}}}]}'
29+
{{- end }}
1930
{{- else }}
20-
{{ $oidc_issuer = printf "%s.%s" .Cluster.LocalID .Values.hosted_zone }}
31+
{{ $oidc_issuer_aws := printf "%s.%s" .Cluster.LocalID .Values.hosted_zone }}
32+
{{ $oidc_issuer_eks := index (split .Cluster.ConfigItems.eks_oidc_issuer_url "//") 1 }}
33+
{{ $oidc_provider_arn_aws := printf "arn:aws:iam::%s:oidc-provider/%s" (accountID .Cluster.InfrastructureAccount) $oidc_issuer_aws }}
34+
{{ $oidc_provider_arn_eks := printf "arn:aws:iam::%s:oidc-provider/%s" (accountID .Cluster.InfrastructureAccount) $oidc_issuer_eks }}
35+
{{ $oidc_subject_key_aws := printf "%s:sub" $oidc_issuer_aws }}
36+
{{ $oidc_subject_key_eks := printf "%s:sub" $oidc_issuer_eks }}
37+
oidc-provider-arn: "{{$oidc_provider_arn_aws}}"
38+
oidc-subject-key: "{{$oidc_subject_key_aws}}"
39+
{{- if ne $oidc_issuer_eks "" }}
40+
oidc-trust-relationship-template: '{"Version":"2012-10-17","Statement":[{"Effect":"Allow","Principal":{"Federated":"{{$oidc_provider_arn_aws}}"},"Action":"sts:AssumeRoleWithWebIdentity","Condition":{"StringLike":{"{{$oidc_subject_key_aws}}":"system:serviceaccount:${SERVICE_ACCOUNT}"}}},{"Effect":"Allow","Principal":{"Federated":"{{$oidc_provider_arn_eks}}"},"Action":"sts:AssumeRoleWithWebIdentity","Condition":{"StringLike":{"{{$oidc_subject_key_eks}}":"system:serviceaccount:${SERVICE_ACCOUNT}"}}}]}'
41+
{{- else }}
42+
oidc-trust-relationship-template: '{"Version":"2012-10-17","Statement":[{"Effect":"Allow","Principal":{"Federated":"{{$oidc_provider_arn_aws}}"},"Action":"sts:AssumeRoleWithWebIdentity","Condition":{"StringLike":{"{{$oidc_subject_key_aws}}":"system:serviceaccount:${SERVICE_ACCOUNT}"}}}]}'
43+
{{- end }}
2144
{{- end }}
22-
{{ $oidc_provider_arn := printf "arn:aws:iam::%s:oidc-provider/%s" (accountID .Cluster.InfrastructureAccount) $oidc_issuer }}
23-
{{ $oidc_subject_key := printf "%s:sub" $oidc_issuer }}
24-
oidc-provider-arn: "{{$oidc_provider_arn}}"
25-
oidc-subject-key: "{{$oidc_subject_key}}"
26-
oidc-trust-relationship-template: '{"Version":"2012-10-17","Statement":[{"Effect":"Allow","Principal":{"Federated":"{{$oidc_provider_arn}}"},"Action":"sts:AssumeRoleWithWebIdentity","Condition":{"StringLike":{"{{$oidc_subject_key}}":"system:serviceaccount:${SERVICE_ACCOUNT}"}}}]}'
2745
s3-bucket-name: "zalando-deployment-service-{{accountID .Cluster.InfrastructureAccount}}-{{.Cluster.LocalID}}"
2846
status-service-url: "https://depl-status-{{.Cluster.Alias}}.{{.Values.hosted_zone}}"
2947
status-service-url-local: "http://deployment-status-service.ingress.cluster.local."

test/e2e/cluster_config.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ clusters:
4646
okta_auth_client_id: "kubernetes.cluster.teapot-e2e"
4747
teapot_admission_controller_validate_pod_images_soft_fail_namespaces: "^kube-system$"
4848
eks_okta_identity_provider: "false" # disabled to speed up EKS cluster creation for e2e.
49+
eks_legacy_cluster_local_id: "e2e-${CDP_BUILD_VERSION}-aws"
4950
skipper_open_policy_agent_enabled: "${SKIPPER_OPA_ENABLED}"
5051
skipper_open_policy_agent_styra_token: "${STYRA_TOKEN}"
5152
skipper_open_policy_agent_bucket_arn: "${SKIPPER_OPA_BUCKET_ARN}"

0 commit comments

Comments
 (0)