Now, moving from dev to sta

Added to this branch <a href="https://github.com/IBM/core-dump-handler/tree/claim-ref"

Hey <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url=

pvc -> pv mismatch when bindng about core-dump-handler HOT 6 CLOSED

ibm commented on May 31, 2024

pvc -> pv mismatch when bindng

from core-dump-handler.

Comments (6)

No9 commented on May 31, 2024

That's a nasty one that was missed because the default is to create the core directory as a sub-folder of the host directory.

Looking at the docs this can be fixed with a reservation using a claimRef.

diff --git a/charts/core-dump-handler/templates/core-storage-pv.yaml b/charts/core-dump-handler/templates/core-storage-pv.yaml
index a89c5a7..c8ed150 100644
--- a/charts/core-dump-handler/templates/core-storage-pv.yaml
+++ b/charts/core-dump-handler/templates/core-storage-pv.yaml
@@ -6,6 +6,9 @@ metadata:
     type: local
 spec:
   storageClassName: {{ .Values.storageClass }}
+  claimRef:
+    name: core-storage-pvc
+    namespace: {{ .Release.Namespace }}
   capacity:
     storage: {{ .Values.coreStorage }}
   accessModes:
diff --git a/charts/core-dump-handler/templates/host-storage-pv.yaml b/charts/core-dump-handler/templates/host-storage-pv.yaml
index 840a4fd..2e69897 100644
--- a/charts/core-dump-handler/templates/host-storage-pv.yaml
+++ b/charts/core-dump-handler/templates/host-storage-pv.yaml
@@ -6,6 +6,9 @@ metadata:
     type: local
 spec:
   storageClassName: {{ .Values.storageClass }}
+  claimRef:
+    name: host-storage-pvc
+    namespace: {{ .Release.Namespace }}
   capacity:
     storage: {{ .Values.hostStorage }}
   accessModes:

This now gives the intended result

$ kubectl get pv core-volume
NAME          CAPACITY   ACCESS MODES   RECLAIM POLICY   STATUS   CLAIM                      STORAGECLASS   REASON   AGE
core-volume   10Gi       RWO            Retain           Bound    observe/core-storage-pvc   hostclass               6m32s

$ kubectl get pv host-volume
NAME          CAPACITY   ACCESS MODES   RECLAIM POLICY   STATUS   CLAIM                      STORAGECLASS   REASON   AGE
host-volume   1Gi        RWO            Retain           Bound    observe/host-storage-pvc   hostclass               6m42s

$ kubectl get pvc -n observe
NAME               STATUS   VOLUME        CAPACITY   ACCESS MODES   STORAGECLASS   AGE
core-storage-pvc   Bound    core-volume   10Gi       RWO            hostclass      6m55s
host-storage-pvc   Bound    host-volume   1Gi        RWO            hostclass      6m55s

And passes the integration tests.

.Release.namespace is picked up by helm template so it should work for you.
If your in agreement I'll commit and release.

from core-dump-handler.

No9 commented on May 31, 2024

Added to this branch https://github.com/IBM/core-dump-handler/tree/claim-ref

from core-dump-handler.