Expect either PV or it's pointer type as arg to PVStore 'Add()' method

We were getting following error message while adding a PV to PVStore as we were expecting only pointer to PV as arg to PVStore Add() method. ``` err: unexpected object of type v1.PersistentVolume ``` Signed-off-by: Arun Kumar Mohan <[email protected]>
red-hat-storage · Dec 14, 2023 · 1dd4603 · 1dd4603
1 parent 9f55e6d
commit 1dd4603
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/metrics/internal/cache/pv.go b/metrics/internal/cache/pv.go
@@ -94,8 +94,14 @@ func appendIfNotExists(slice []string, value string) []string {
 
 // Add inserts to the PersistentVolumeStore.
 func (p *PersistentVolumeStore) Add(obj interface{}) error {
-	pv, ok := obj.(*corev1.PersistentVolume)
-	if !ok {
+	var pv *corev1.PersistentVolume
+	// obj can be of PV type or a pointer to it
+	switch pvFromObj := obj.(type) {
+	case *corev1.PersistentVolume:
+		pv = pvFromObj
+	case corev1.PersistentVolume:
+		pv = &pvFromObj
+	default:
 		return fmt.Errorf("unexpected object of type %T", obj)
 	}