Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(vault): switch to nfs snapshots #4405

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

jazzlyn
Copy link
Collaborator

@jazzlyn jazzlyn commented Jan 28, 2025

No description provided.

@tyriis-automation
Copy link
Contributor

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ EDITORCONFIG editorconfig-checker 5 0 0.02s
✅ REPOSITORY gitleaks yes no 2.95s
✅ YAML prettier 5 0 0.43s
✅ YAML yamllint 5 0 0.43s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@tyriis-automation
Copy link
Contributor

--- kubernetes/talos-flux/apps/kube-system/csi-driver-nfs/app Kustomization: flux-system/apps-csi-driver-nfs HelmRelease: kube-system/csi-driver-nfs

+++ kubernetes/talos-flux/apps/kube-system/csi-driver-nfs/app Kustomization: flux-system/apps-csi-driver-nfs HelmRelease: kube-system/csi-driver-nfs

@@ -28,7 +28,11 @@

     cleanupOnFail: true
     remediation:
       retries: 3
   values:
     externalSnapshotter:
       enabled: false
+    volumeSnapshotClass:
+      create: true
+      deletionPolicy: Delete
+      name: nfs-csi-snapshot
 
--- kubernetes/talos-flux/apps/secops/vault/app Kustomization: flux-system/vault HelmRelease: secops/vault

+++ kubernetes/talos-flux/apps/secops/vault/app Kustomization: flux-system/vault HelmRelease: secops/vault

@@ -44,13 +44,13 @@

         size: 1Gi
         storageClass: ceph-block
       dataStorage:
         enabled: true
         mountPath: /vault/data
         size: 1Gi
-        storageClass: ceph-block
+        storageClass: nfs-csi
       extraEnvironmentVars:
         TZ: Europe/Vienna
       extraLabels:
         reloader.stakater.com/auto: 'true'
       ha:
         enabled: true
--- kubernetes/talos-flux/apps/secops/vault/app Kustomization: flux-system/vault PersistentVolumeClaim: secops/data-vault-0

+++ kubernetes/talos-flux/apps/secops/vault/app Kustomization: flux-system/vault PersistentVolumeClaim: secops/data-vault-0

@@ -14,8 +14,8 @@

     apiGroup: volsync.backube
     kind: ReplicationDestination
     name: data-vault-0
   resources:
     requests:
       storage: 1Gi
-  storageClassName: ceph-block
+  storageClassName: nfs-csi
 
--- kubernetes/talos-flux/apps/secops/vault/app Kustomization: flux-system/vault ReplicationSource: secops/data-vault-0

+++ kubernetes/talos-flux/apps/secops/vault/app Kustomization: flux-system/vault ReplicationSource: secops/data-vault-0

@@ -11,23 +11,23 @@

   restic:
     accessModes:
     - ReadWriteOnce
     cacheAccessModes:
     - ReadWriteOnce
     cacheCapacity: 1Gi
-    cacheStorageClassName: ceph-block
+    cacheStorageClassName: nfs-csi
     copyMethod: Snapshot
     moverSecurityContext:
       fsGroup: 1000
       runAsGroup: 1000
       runAsUser: 1000
     pruneIntervalDays: 14
     repository: vault-volsync
     retain:
       daily: 7
       hourly: 24
-    storageClassName: ceph-block
-    volumeSnapshotClassName: csi-ceph-blockpool
+    storageClassName: nfs-csi
+    volumeSnapshotClassName: nfs-csi-snapshot
   sourcePVC: data-vault-0
   trigger:
     schedule: 0 * * * *
 
--- kubernetes/talos-flux/apps/secops/vault/app Kustomization: flux-system/vault ReplicationDestination: secops/data-vault-0

+++ kubernetes/talos-flux/apps/secops/vault/app Kustomization: flux-system/vault ReplicationDestination: secops/data-vault-0

@@ -12,22 +12,22 @@

   restic:
     accessModes:
     - ReadWriteOnce
     cacheAccessModes:
     - ReadWriteOnce
     cacheCapacity: 1Gi
-    cacheStorageClassName: ceph-block
+    cacheStorageClassName: nfs-csi
     capacity: 1Gi
     cleanupCachePVC: true
     cleanupTempPVC: true
     copyMethod: Snapshot
     enableFileDeletion: true
     moverSecurityContext:
       fsGroup: 1000
       runAsGroup: 1000
       runAsUser: 1000
     repository: vault-volsync
-    storageClassName: ceph-block
-    volumeSnapshotClassName: csi-ceph-blockpool
+    storageClassName: nfs-csi
+    volumeSnapshotClassName: nfs-csi-snapshot
   trigger:
     manual: restore-once

@tyriis-automation
Copy link
Contributor

--- HelmRelease: kube-system/csi-driver-nfs VolumeSnapshotClass: kube-system/nfs-csi-snapshot

+++ HelmRelease: kube-system/csi-driver-nfs VolumeSnapshotClass: kube-system/nfs-csi-snapshot

@@ -0,0 +1,8 @@

+---
+apiVersion: snapshot.storage.k8s.io/v1
+kind: VolumeSnapshotClass
+metadata:
+  name: nfs-csi-snapshot
+driver: nfs.csi.k8s.io
+deletionPolicy: Delete
+
--- HelmRelease: secops/vault StatefulSet: secops/vault

+++ HelmRelease: secops/vault StatefulSet: secops/vault

@@ -157,13 +157,13 @@

     spec:
       accessModes:
       - ReadWriteOnce
       resources:
         requests:
           storage: 1Gi
-      storageClassName: ceph-block
+      storageClassName: nfs-csi
   - metadata:
       name: audit
     spec:
       accessModes:
       - ReadWriteOnce
       resources:

@jazzlyn jazzlyn merged commit 3ab222e into main Jan 28, 2025
16 checks passed
@jazzlyn jazzlyn deleted the feature/vault-switch-nfs-snapshots branch January 28, 2025 20:25
@jazzlyn jazzlyn mentioned this pull request Jan 29, 2025
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant