From 46e521ea9ac581a38033bf8503b404aa70b7f206 Mon Sep 17 00:00:00 2001
From: Jennifer Richards <jennifer@staff.ietf.org>
Date: Tue, 2 Apr 2024 17:51:15 -0300
Subject: [PATCH] ci: shared PersistentVolumeClaim on /a (#7283)

* ci: shared PersistentVolumeClaim on /a

* ci: Increase storage request, add comment
---
 helm/templates/persistentvolumeclaims.yaml | 14 ++++
 helm/values.yaml                           | 81 ++++++++++++++--------
 2 files changed, 65 insertions(+), 30 deletions(-)

diff --git a/helm/templates/persistentvolumeclaims.yaml b/helm/templates/persistentvolumeclaims.yaml
index edf69aebc..db636fb05 100644
--- a/helm/templates/persistentvolumeclaims.yaml
+++ b/helm/templates/persistentvolumeclaims.yaml
@@ -1,5 +1,19 @@
 apiVersion: v1
 kind: PersistentVolumeClaim
+metadata:
+  name: "datatracker-shared-volume-claim"
+  # Note: This is the /a shared volume. The spec should be adjusted to ensure it always
+  # matches the correct EBS volume in production. For the moment, it just requests a
+  # large volume.
+spec:
+  accessModes:
+    - ReadWriteMany  # or ReadWriteOnce and force datatracker/celery/beat to a single node
+  resources:
+    requests:
+      storage: 600Gi  # adjust to something satisfied by the /a PersistentVolume
+---
+apiVersion: v1
+kind: PersistentVolumeClaim
 metadata:
   name: "rabbitmq-data-claim"
 spec:
diff --git a/helm/values.yaml b/helm/values.yaml
index d09df4d65..90ff438b2 100644
--- a/helm/values.yaml
+++ b/helm/values.yaml
@@ -129,12 +129,16 @@ datatracker:
     - name: settings-local-volume
       configMap:
         name: django-configmap
-    - name: cache-volume
-      emptyDir:
-        sizeLimit: 1Gi
-    - name: staging-volume
-      emptyDir:
-        sizeLimit: 1Gi
+    - name: datatracker-shared-volume
+      persistentVolumeClaim:
+        claimName: "datatracker-shared-volume-claim"
+# cache-volume and staging-volume are a hack to create paths for debugging without a /a volume
+#    - name: cache-volume
+#      emptyDir:
+#        sizeLimit: 1Gi
+#    - name: staging-volume
+#      emptyDir:
+#        sizeLimit: 1Gi
   # - name: foo
   #   secret:
   #     secretName: mysecret
@@ -146,10 +150,13 @@ datatracker:
       mountPath: /workspace/ietf/settings_local.py
       subPath: settings_local.py
       readOnly: true
-    - name: cache-volume
-      mountPath: "/a/cache"
-    - name: staging-volume
-      mountPath: "/test/staging"
+    - name: datatracker-shared-volume
+      mountPath: /a
+# cache-volume and staging-volume are a hack to create paths for debugging without a /a volume
+#    - name: cache-volume
+#      mountPath: "/a/cache"
+#    - name: staging-volume
+#      mountPath: "/test/staging"
   # - name: foo
   #   mountPath: "/etc/foo"
   #   readOnly: true
@@ -240,12 +247,16 @@ celery:
     - name: settings-local-volume
       configMap:
         name: django-configmap
-    - name: cache-volume
-      emptyDir:
-        sizeLimit: 1Gi
-    - name: staging-volume
-      emptyDir:
-        sizeLimit: 1Gi
+    - name: datatracker-shared-volume
+      persistentVolumeClaim:
+        claimName: "datatracker-shared-volume-claim"
+# cache-volume and staging-volume are a hack to create paths for debugging without a /a volume
+#    - name: cache-volume
+#      emptyDir:
+#        sizeLimit: 1Gi
+#    - name: staging-volume
+#      emptyDir:
+#        sizeLimit: 1Gi
   # - name: foo
   #   secret:
   #     secretName: mysecret
@@ -257,10 +268,13 @@ celery:
       mountPath: /workspace/ietf/settings_local.py
       subPath: settings_local.py
       readOnly: true
-    - name: cache-volume
-      mountPath: "/a/cache"
-    - name: staging-volume
-      mountPath: "/test/staging"
+    - name: datatracker-shared-volume
+      mountPath: /a
+# cache-volume and staging-volume are a hack to create paths for debugging without a /a volume
+#    - name: cache-volume
+#      mountPath: "/a/cache"
+#    - name: staging-volume
+#      mountPath: "/test/staging"
   # - name: foo
   #   mountPath: "/etc/foo"
   #   readOnly: true
@@ -351,12 +365,16 @@ beat:
     - name: settings-local-volume
       configMap:
         name: django-configmap
-    - name: cache-volume
-      emptyDir:
-        sizeLimit: 1Gi
-    - name: staging-volume
-      emptyDir:
-        sizeLimit: 1Gi
+    - name: datatracker-shared-volume
+      persistentVolumeClaim:
+        claimName: "datatracker-shared-volume-claim"
+# cache-volume and staging-volume are a hack to create paths for debugging without a /a volume
+#    - name: cache-volume
+#      emptyDir:
+#        sizeLimit: 1Gi
+#    - name: staging-volume
+#      emptyDir:
+#        sizeLimit: 1Gi
   # - name: foo
   #   secret:
   #     secretName: mysecret
@@ -368,10 +386,13 @@ beat:
       mountPath: /workspace/ietf/settings_local.py
       subPath: settings_local.py
       readOnly: true
-    - name: cache-volume
-      mountPath: "/a/cache"
-    - name: staging-volume
-      mountPath: "/test/staging"
+    - name: datatracker-shared-volume
+      mountPath: /a
+# cache-volume and staging-volume are a hack to create paths for debugging without a /a volume
+#    - name: cache-volume
+#      mountPath: "/a/cache"
+#    - name: staging-volume
+#      mountPath: "/test/staging"
   # - name: foo
   #   mountPath: "/etc/foo"
   #   readOnly: true