From 48bada157e7220b7cc3a79b67619c8dfb78042e1 Mon Sep 17 00:00:00 2001 From: sudobangbang Date: Tue, 9 Jul 2024 11:17:55 -0600 Subject: [PATCH] Revert "Revert "minor: remove static NFS PV (#100)" (#101)" (#102) This reverts commit 5d25b360ad7aa5ec7fb5a1d6391052b5dd9d3a5b. --- .../eoapi/templates/pv-static-efs-rwmany.yaml | 19 ------------------- helm-chart/eoapi/values.yaml | 12 ------------ 2 files changed, 31 deletions(-) delete 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 deleted file mode 100644 index 07319fb..0000000 --- a/helm-chart/eoapi/templates/pv-static-efs-rwmany.yaml +++ /dev/null @@ -1,19 +0,0 @@ -#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 740383d..bea3157 100644 --- a/helm-chart/eoapi/values.yaml +++ b/helm-chart/eoapi/values.yaml @@ -98,18 +98,6 @@ 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