From e2744201328deae57880d6b1e801df271c503804 Mon Sep 17 00:00:00 2001
From: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
Date: Thu, 8 Oct 2026 15:35:44 -0700
Subject: [PATCH] Remove StorageClass override from installation builder

Use the cluster default in generated installation commands and remove the optional storage field and its obsolete test assertions. Chart-level storage configuration remains unchanged.

Signed-off-by: Rohan Adwankar <39285979+RohanAdwankar@users.noreply.github.com>
---
 docs/_static/install-builder.mjs    | 3 ---
 tests/docs/install-builder.test.mjs | 7 ++-----
 2 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/docs/_static/install-builder.mjs b/docs/_static/install-builder.mjs
index 92afde6..69af675 100644
--- a/docs/_static/install-builder.mjs
+++ b/docs/_static/install-builder.mjs
@@ -20,7 +20,6 @@ const fields = {
   evidenceRoot: ['Slurm evidence directory on the collector node', '', '/slurm'],
   evidenceNode: ['Slurm collector node hostname', '', 'slurm-login-node'],
   corpus: ['Research corpus directory on the node', '', '/cm/shared/iraop-corpus'],
-  storageClass: ['StorageClass override (optional)', '', 'Leave empty to use the cluster default'],
 };
 export function enabledModules(state) {
   const enabled = new Set(state.modules || []);
@@ -66,7 +65,6 @@ export function buildCommand(state) {
     set('researchAgent.iraop.corpus.hostPath', value('corpus'), true);
   }
   if (enabled.has('edit')) set('modules.edit.hitl', true);
-  if (state.storageClass?.trim()) for (const key of ['openclaw.pvc.storageClassName', 'mosaicUi.auditPvc.storageClassName']) set(key, state.storageClass.trim(), true);
   if (state.sandboxInstalled) set('agentSandbox.install', false);
   let target = quote(chart);
   let prefix = '';
@@ -128,7 +126,6 @@ if (root) {
       if (enabled.has('bcm')) keys.push('bcmHead', 'bcmSecret');
       if (enabled.has('slurm') && state.slurmBackend === 'vanilla') keys.push('evidenceRoot', 'evidenceNode');
       if (enabled.has('research')) keys.push('corpus');
-      keys.push('storageClass');
       for (const key of keys) {
         const [label, initial, placeholder] = fields[key]; const wrapper = document.createElement('label'); wrapper.textContent = label;
         const input = document.createElement('input'); input.name = key; input.value = state[key] ?? initial; input.placeholder = placeholder || '';
diff --git a/tests/docs/install-builder.test.mjs b/tests/docs/install-builder.test.mjs
index c157091..5b0e89c 100644
--- a/tests/docs/install-builder.test.mjs
+++ b/tests/docs/install-builder.test.mjs
@@ -44,10 +44,7 @@ test('shell metacharacters remain literal arguments and Helm list separators are
     assert.ok(args.includes(`llm.external.model=${model.replaceAll(',', '\\,')}`));
   }
 });
-test('pinning replaces the development selector and storage overrides are optional', () => {
-  assert.ok(!argumentsFor({...example,modules:[]}).some(arg=>arg.startsWith('openclaw.pvc.storageClassName=')));
-  const args=argumentsFor({...example,modules:[],version:'0.0.1',storageClass:'workspace-storage'});
+test('pinning replaces the development selector', () => {
+  const args=argumentsFor({...example,modules:[],version:'0.0.1'});
   assert.ok(!args.includes('--devel'));assert.ok(args.includes('0.0.1'));
-  assert.ok(args.includes('openclaw.pvc.storageClassName=workspace-storage'));
-  assert.ok(args.includes('mosaicUi.auditPvc.storageClassName=workspace-storage'));
 });