Skip to content

Commit

Permalink
Changed initialDelaySeconds to 120 and set CPU to 1vcpu and memory to 4G
Browse files Browse the repository at this point in the history
  • Loading branch information
cabeaulac committed Jan 4, 2024
1 parent ce47b15 commit 4abd91b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
1 change: 1 addition & 0 deletions charts/localstack/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ spec:
httpGet:
path: /_localstack/health
port: {{ .Values.service.edgeService.name }}
initialDelaySeconds: 120
resources:
{{- toYaml .Values.resources | nindent 12 }}
{{- if coalesce (and (.Values.mountDind.enabled) (.Values.mountDind.forceTLS)) .Values.enableStartupScripts .Values.persistence.enabled .Values.volumeMounts }}
Expand Down
6 changes: 5 additions & 1 deletion charts/localstack/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@ persistence:
annotations: {}


resources: {}
#resources: {}

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

View workflow job for this annotation

GitHub Actions / lint-test

165:2 [comments] missing starting space in comment
# We usually recommend not to specify default resources and to leave this as a conscious
# choice for the user. This also increases chances charts run on environments with little
# resources, such as Minikube. If you do want to specify resources, uncomment the following
Expand All @@ -173,6 +173,10 @@ resources: {}
# requests:
# cpu: 100m
# memory: 128Mi
resources:
requests:
cpu: 1000m
memory: 4Gi

# All settings inside the lambda values section are only applicable to the new v2 lambda provider
lambda:
Expand Down

0 comments on commit 4abd91b

Please sign in to comment.