Skip to content

Commit

Permalink
Added ability to override LATTICE_ENDPOINT env variable with helm (#446)
Browse files Browse the repository at this point in the history
Co-authored-by: Shawn Kaplan <[email protected]>
  • Loading branch information
xWink and Shawn Kaplan authored Oct 23, 2023
1 parent 19d3ea7 commit 7120cce
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 1 deletion.
4 changes: 3 additions & 1 deletion docs/deploy.md
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,9 @@ Run through them again for a second cluster to use with the extended example sho
--set=clusterVpcId= \
--set=clusterName= \
--set=awsAccountId= \
# latticeEndpoint is required for the case where the VPC Lattice endpoint is being overridden
--set=latticeEndpoint= \
```
1. Create the `amazon-vpc-lattice` GatewayClass:
```bash
Expand Down
1 change: 1 addition & 0 deletions helm/templates/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,5 @@ data:
awsAccountId: {{ .Values.awsAccountId | quote }}
clusterVpcId: {{ .Values.clusterVpcId | quote }}
clusterName: {{ .Values.clusterName | quote }}
latticeEndpoint: {{ .Values.latticeEndpoint | quote }}

5 changes: 5 additions & 0 deletions helm/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,11 @@ spec:
configMapKeyRef:
name: env-config
key: clusterName
- name: LATTICE_ENDPOINT
valueFrom:
configMapKeyRef:
name: env-config
key: latticeEndpoint

terminationGracePeriodSeconds: 10
nodeSelector: {{ toYaml .Values.deployment.nodeSelector | nindent 8 }}
Expand Down
1 change: 1 addition & 0 deletions helm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -75,3 +75,4 @@ awsRegion:
awsAccountId:
clusterVpcId:
clusterName:
latticeEndpoint:

0 comments on commit 7120cce

Please sign in to comment.