From 5ea633f47195db5fe591799ed9cefc90ea37b7c0 Mon Sep 17 00:00:00 2001
From: Conor Schaefer <conor@penumbralabs.xyz>
Date: Mon, 29 Jan 2024 11:56:26 -0800
Subject: [PATCH] ci: adjust storage requests

Makes two changes to the PVCs:

  * preview 50G -> 20G
  * testnet 200G -> 300G

We don't need 50GB of storage per node on preview. That's >2 weeks
runtime, and we've never gone >2 weeks without deploying to preview.
For testnet, though, we grow the storage requests: 200GB is roughly ten
weeks, and we're approaching full utilization on Testnet 64 already.

We can still adjust these storage requests over time, but updating them
now to reflect our expectations.
---
 deployments/helmfile.d/penumbra-preview.yaml  |  6 ++--
 .../helmfile.d/penumbra-testnet-cuiloa.yaml   | 29 -------------------
 deployments/helmfile.d/penumbra-testnet.yaml  | 27 +++++++++++++++--
 3 files changed, 28 insertions(+), 34 deletions(-)
 delete mode 100644 deployments/helmfile.d/penumbra-testnet-cuiloa.yaml

diff --git a/deployments/helmfile.d/penumbra-preview.yaml b/deployments/helmfile.d/penumbra-preview.yaml
index 1d61d79f94..5a185edd60 100644
--- a/deployments/helmfile.d/penumbra-preview.yaml
+++ b/deployments/helmfile.d/penumbra-preview.yaml
@@ -5,7 +5,7 @@ releases:
     values:
       - persistence:
           enabled: true
-          size: 50G
+          size: 20G
       - preserve_lb_svc: true
       - only_lb_svc: false
       - image:
@@ -37,7 +37,7 @@ releases:
       - penumbra_bootstrap_node_cometbft_rpc_url: "http://penumbra-preview-val-0:26657"
       - persistence:
           enabled: true
-          size: 50G
+          size: 20G
       - part_of: penumbra-preview
       # Node config info, including ip address, monikers, and seed-mode status.
       - vars/penumbra-preview-nodes-ips.yml
@@ -78,7 +78,7 @@ releases:
           tag: main
       - persistence:
           enabled: true
-          size: 50G
+          size: 20G
       - cometbft:
           config:
             indexer: psql
diff --git a/deployments/helmfile.d/penumbra-testnet-cuiloa.yaml b/deployments/helmfile.d/penumbra-testnet-cuiloa.yaml
deleted file mode 100644
index c4d7f8efe0..0000000000
--- a/deployments/helmfile.d/penumbra-testnet-cuiloa.yaml
+++ /dev/null
@@ -1,29 +0,0 @@
----
-releases:
-  - name: penumbra-testnet-cuiloa-node
-    chart: ../charts/penumbra-node
-# Disabling the strict dependency on the "testnet" environment,
-# because this release was first deployed manually. Post Testnet 64
-# we plan to retire the CI pipeline for deploying the testnet.
-# If breaking changes are merged to Penumbra or Cuiloa, we'll need
-# to redeploy this node manually.
-#    needs:
-#      - penumbra-testnet
-#      # It's not strictly necessary to wait for node deploys, but doing so allows us to exercise
-#      # the public HTTPS RPC endpoint for joining, which is nice.
-#      - penumbra-testnet-nodes
-    values:
-      - penumbra_bootstrap_node_cometbft_rpc_url: "https://rpc.testnet.penumbra.zone"
-      - ingressRoute:
-          enabled: false
-      - image:
-          tag: "v0.64.1"
-      - persistence:
-          enabled: true
-          size: 200G
-      - cometbft:
-          config:
-            indexer: psql
-      - part_of: penumbra-testnet
-      - nodes:
-        - moniker: cuiloa
diff --git a/deployments/helmfile.d/penumbra-testnet.yaml b/deployments/helmfile.d/penumbra-testnet.yaml
index f2d715a049..1cc0bc037c 100644
--- a/deployments/helmfile.d/penumbra-testnet.yaml
+++ b/deployments/helmfile.d/penumbra-testnet.yaml
@@ -5,7 +5,7 @@ releases:
     values:
       - persistence:
           enabled: true
-          size: 200G
+          size: 300G
       - preserve_lb_svc: true
       - only_lb_svc: false
       - image:
@@ -38,7 +38,7 @@ releases:
       - penumbra_bootstrap_node_cometbft_rpc_url: "http://penumbra-testnet-val-0:26657"
       - persistence:
           enabled: true
-          size: 200G
+          size: 300G
       - part_of: penumbra-testnet
       # Node config info, including ip address, monikers, and seed-mode status.
       - vars/penumbra-testnet-nodes-ips.yml
@@ -63,3 +63,26 @@ releases:
       - persistence:
           enabled: true
           size: 20G
+
+  - name: penumbra-testnet-cuiloa-node
+    chart: ../charts/penumbra-node
+    needs:
+      - penumbra-testnet
+      # It's not strictly necessary to wait for node deploys, but doing so allows us to exercise
+      # the public HTTPS RPC endpoint for joining, which is nice.
+      - penumbra-testnet-nodes
+   values:
+      - penumbra_bootstrap_node_cometbft_rpc_url: "https://rpc.testnet.penumbra.zone"
+      - ingressRoute:
+          enabled: false
+      - image:
+          tag: latest
+      - persistence:
+          enabled: true
+          size: 300G
+      - cometbft:
+          config:
+            indexer: psql
+      - part_of: penumbra-testnet
+      - nodes:
+        - moniker: cuiloa