From 5d25b360ad7aa5ec7fb5a1d6391052b5dd9d3a5b Mon Sep 17 00:00:00 2001
From: sudobangbang <gregcorradini@gmail.com>
Date: Tue, 9 Jul 2024 11:14:56 -0600
Subject: [PATCH] Revert "minor: remove static NFS PV (#100)" (#101)

This reverts commit 9792689909b9a471798df82ae4aa8e68641ed964.
---
 .../eoapi/templates/pv-static-efs-rwmany.yaml | 19 +++++++++++++++++++
 helm-chart/eoapi/values.yaml                  | 12 ++++++++++++
 2 files changed, 31 insertions(+)
 create mode 100644 helm-chart/eoapi/templates/pv-static-efs-rwmany.yaml

diff --git a/helm-chart/eoapi/templates/pv-static-efs-rwmany.yaml b/helm-chart/eoapi/templates/pv-static-efs-rwmany.yaml
new file mode 100644
index 0000000..07319fb
--- /dev/null
+++ b/helm-chart/eoapi/templates/pv-static-efs-rwmany.yaml
@@ -0,0 +1,19 @@
+#https://github.com/kubernetes-sigs/aws-efs-csi-driver/blob/master/examples/kubernetes/static_provisioning/README.md
+{{- if and (.Values.postgresclusterNFSStaticPV.enabled) (.Values.postgrescluster.enabled) }}
+---
+apiVersion: v1
+kind: PersistentVolume
+metadata:
+  name: efs-pv
+spec:
+  capacity:
+    storage: 300Mi
+  volumeMode: Filesystem
+  accessModes:
+    - ReadWriteMany
+  persistentVolumeReclaimPolicy: Retain
+  storageClassName: {{ .Values.postgresclusterNFSStaticPV.storageClassName }}
+  csi:
+    driver: efs.csi.aws.com
+    volumeHandle: {{ .Values.postgresclusterNFSStaticPV.volumeHandle }}
+{{- end }}
\ No newline at end of file
diff --git a/helm-chart/eoapi/values.yaml b/helm-chart/eoapi/values.yaml
index bea3157..740383d 100644
--- a/helm-chart/eoapi/values.yaml
+++ b/helm-chart/eoapi/values.yaml
@@ -98,6 +98,18 @@ db:
       PGPASSWORD: ""
       PGDATABASE: "postgis"
 
+# on AWS we use the EBS and EFS CSI drivers that dynamically create
+# and mount storage on request via PVC(s) but for NFS
+# the EFS CSI driver by default creates mounts with access points
+# that restrict read/write permissions to the uid:gid of the mount.
+# in simple cases we don't want to have to think about this extra layer for NFS
+# and this is a workaround to mount a static NFS without access points.
+# NOTE: GCP and other IaC haven't been set up to use NFS yet
+postgresclusterNFSStaticPV:
+  enabled: false
+  storageClassName: ""
+  volumeHandle: ""
+
 # this is declared as a dependency of eoapi in helm-chart/eoapi/Chart.yaml
 postgrescluster:
   enabled: true