Skip to content

Commit 0b5dc38

Browse files
Add missing Kustomize env vars (#306)
Fixes aws-controllers-k8s/community#1224 Description of changes: The environment variables being passed to the deployment in the generated Kustomization are not up-to-date with the [Helm chart](https://github.com/aws-controllers-k8s/code-generator/blob/7052f6808b237f97a2000e4c4f296b9ed2a0d7c0/templates/helm/templates/deployment.yaml#L66-L82). This pull request adds each of the environment variables with the same default values as the ones supplied in `values.yaml`. By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent 7052f68 commit 0b5dc38

File tree

1 file changed

+14
-0
lines changed

1 file changed

+14
-0
lines changed

templates/config/controller/deployment.yaml.tpl

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,8 @@ spec:
2828
args:
2929
- --aws-region
3030
- "$(AWS_REGION)"
31+
- --aws-endpoint-url
32+
- "$(AWS_ENDPOINT_URL)"
3133
- --enable-development-logging
3234
- "$(ACK_ENABLE_DEVELOPMENT_LOGGING)"
3335
- --log-level
@@ -53,6 +55,18 @@ spec:
5355
valueFrom:
5456
fieldRef:
5557
fieldPath: metadata.namespace
58+
- name: AWS_REGION
59+
value: ""
60+
- name: AWS_ENDPOINT_URL
61+
value: ""
62+
- name: ACK_WATCH_NAMESPACE
63+
value: ""
64+
- name: ACK_ENABLE_DEVELOPMENT_LOGGING
65+
value: "false"
66+
- name: ACK_LOG_LEVEL
67+
value: "info"
68+
- name: ACK_RESOURCE_TAGS
69+
value: "services.k8s.aws/managed=true,services.k8s.aws/created=%UTCNOW%,services.k8s.aws/namespace=%KUBERNETES_NAMESPACE%"
5670
securityContext:
5771
allowPrivilegeEscalation: false
5872
privileged: false

0 commit comments

Comments
 (0)