Skip to content

Commit

Permalink
Added dnsConfig set to Default. \The default if not specified is Clus…
Browse files Browse the repository at this point in the history
…terFirst which causes a circular DNS lookup and crashes LocalStack
  • Loading branch information
cabeaulac committed Jan 16, 2024
1 parent 1f7970e commit 2a70efc
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 0 deletions.
3 changes: 3 additions & 0 deletions charts/localstack/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@ spec:
serviceAccountName: {{ include "localstack.serviceAccountName" . }}
securityContext:
{{- toYaml .Values.podSecurityContext | nindent 8 }}
{{- if .Values.dnsPolicy }}
dnsPolicy: {{ .Values.dnsPolicy }}
{{- end }}
containers:
- name: {{ .Chart.Name }}
securityContext:
Expand Down
5 changes: 5 additions & 0 deletions charts/localstack/test-values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,11 @@ command:
- -c
- echo 'ulimit -Sn 32767' >> /root/.bashrc && echo 'ulimit -Su 16383' >> /root/.bashrc && docker-entrypoint.sh

## @param dnsPolicy Allows you to set the Pod dnsPolicy.
## The default is actually ClusterFirst. Uncomment this to avoid a circular DNS path that will
## cause the LocalStack instance to crash.
## Ref: https://kubernetes.io/docs/concepts/services-networking/dns-pod-service/#pod-s-dns-policy
dnsPolicy: "Default"

# enable startup scripts, create a startup script which creates an SQS queue
enableStartupScripts: true
Expand Down
5 changes: 5 additions & 0 deletions charts/localstack/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,11 @@ debug: false
##
command: []

## @param dnsPolicy Allows you to set the Pod dnsPolicy.
## The default is actually ClusterFirst. Uncomment this to avoid a circular DNS path that will
## cause the LocalStack instance to crash.
## Ref: https://kubernetes.io/docs/concepts/services-networking/dns-pod-service/#pod-s-dns-policy
#dnsPolicy: "Default"

Check failure on line 73 in charts/localstack/values.yaml

View workflow job for this annotation

GitHub Actions / lint-test

73:2 [comments] missing starting space in comment

startServices: ""
# Comma-separated list of AWS CLI service names which are the only ones allowed to be used (other services will then by default be prevented from being loaded).
Expand Down

0 comments on commit 2a70efc

Please sign in to comment.