Skip to content

Commit

Permalink
Conditionally added AWS_STS_REGIONAL_ENDPOINTS flag to controller's e…
Browse files Browse the repository at this point in the history
…fs-plugin container. Is needed to create persistent volumes on private EKS clusters. Without this, dynamic pv creation errors: "Failed to fetch File System info: Describe File System failed"
  • Loading branch information
holmesb committed Dec 7, 2021
1 parent 5549c90 commit 59757e3
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 2 deletions.
3 changes: 3 additions & 0 deletions charts/aws-efs-csi-driver/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
# Helm chart

# v2.2.2
* Add STS regional endpoints flag to fix PV creation on private EKS

# v2.2.1
* Add controller.volMetricsOptIn for emitting volume metrics

Expand Down
2 changes: 1 addition & 1 deletion charts/aws-efs-csi-driver/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
apiVersion: v2
name: aws-efs-csi-driver
version: 2.2.1
version: 2.2.2
appVersion: 1.3.3
kubeVersion: ">=1.17.0-0"
description: "A Helm chart for AWS EFS CSI Driver"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,10 @@ spec:
env:
- name: CSI_ENDPOINT
value: unix:///var/lib/csi/sockets/pluginproxy/csi.sock
{{- if .Values.controller.regionalStsEndpoints }}
- name: AWS_STS_REGIONAL_ENDPOINTS
value: regional
{{- end }}
volumeMounts:
- name: socket-dir
mountPath: /var/lib/csi/sockets/pluginproxy/
Expand Down
2 changes: 1 addition & 1 deletion charts/aws-efs-csi-driver/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ controller:
## Enable if EKS IAM for SA is used
# eks.amazonaws.com/role-arn: arn:aws:iam::111122223333:role/efs-csi-role
healthPort: 9909

regionalStsEndpoints: false
## Node daemonset variables

node:
Expand Down

0 comments on commit 59757e3

Please sign in to comment.