Skip to content

Commit

Permalink
small helm template fixes (#159)
Browse files Browse the repository at this point in the history
Fixes a couple tiny issues I saw in generating release artifacts
recently: some rogue whitespace and wrapping comments at 80 chars

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.
  • Loading branch information
jaypipes authored Aug 18, 2021
1 parent 7ce2d04 commit 821a0da
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
2 changes: 1 addition & 1 deletion templates/helm/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ spec:
- name: AWS_ENDPOINT_URL
value: {{ .Values.aws.endpoint_url | quote }}
- name: ACK_WATCH_NAMESPACE
value: {{ include "watch-namespace" . }}
value: {{ include "watch-namespace" . }}
- name: ACK_ENABLE_DEVELOPMENT_LOGGING
value: {{ .Values.log.enable_development_logging | quote }}
- name: ACK_LOG_LEVEL
Expand Down
8 changes: 5 additions & 3 deletions templates/helm/values.yaml.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -44,12 +44,14 @@ log:
enable_development_logging: false
level: info

# Set to "namespace" to install the controller in a namespaced scope, will only watch for object creation
# in the namespace. By default installScope is cluster wide.
# Set to "namespace" to install the controller in a namespaced scope, will only
# watch for object creation in the namespace. By default installScope is
# cluster wide.
installScope: cluster

resourceTags:
# Configures the ACK service controller to always set key/value pairs tags on resources that it manages.
# Configures the ACK service controller to always set key/value pairs tags on
# resources that it manages.
- services.k8s.aws/managed=true
- services.k8s.aws/created=%UTCNOW%
- services.k8s.aws/namespace=%KUBERNETES_NAMESPACE%
Expand Down

0 comments on commit 821a0da

Please sign in to comment.