From 124eff9ee4a4c8964866d4a960db9ab615c383cb Mon Sep 17 00:00:00 2001 From: OdooSky Bot Date: Sat, 2 May 2026 21:16:41 +0300 Subject: [PATCH] =?UTF-8?q?0.3.1:=20drop=20premature=20VolumeSnapshotClass?= =?UTF-8?q?=20=E2=80=94=20depends=20on=20external=20snapshotter=20CRDs=20(?= =?UTF-8?q?deferred=20to=20phase=203)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- templates/longhorn.yaml | 23 ++++++----------------- 1 file changed, 6 insertions(+), 17 deletions(-) diff --git a/templates/longhorn.yaml b/templates/longhorn.yaml index 2266b04..1545d64 100644 --- a/templates/longhorn.yaml +++ b/templates/longhorn.yaml @@ -23,21 +23,10 @@ parameters: fromBackup: "" # fsType: ext4 is the default; explicit so the contract is in Git. fsType: "ext4" ---- -# VolumeSnapshotClass — the API surface the Tower backend will -# consume in Phase 3 to take snapshots of source PVCs before -# spawn-env / Refresh ↓. -# -# `type: snap` (Longhorn's in-place snapshot) is what we want for -# the fast Clone path: the snapshot is a CoW reference, not an S3 -# upload. `type: bak` is the slower S3-bound block backup, used -# only by the daily DR layer. -apiVersion: snapshot.storage.k8s.io/v1 -kind: VolumeSnapshotClass -metadata: - name: longhorn-snapshot-class -driver: driver.longhorn.io -deletionPolicy: Delete -parameters: - type: snap {{- end }} +# NOTE: The CSI VolumeSnapshotClass (snapshot.storage.k8s.io/v1) lives +# in Phase 3 of ADR 0003, alongside the Tower code that consumes it. +# It depends on the external-snapshotter CRDs which Longhorn does not +# ship — they're a separate install. Adding it here would fail the +# initial sync on a fresh cluster. Phase 3 will land both together +# (snapshotter CRDs + VolumeSnapshotClass + Tower VolumeClone path).