Merge pull request #37 from adamviktora/error-filter-fix

CNV-74688: fix Status filter for Error state

Author: openshift-merge-bot[bot]

util/filters.go

@@ -0,0 +1,31 @@
+package util
+
+import "slices"
+
+const (
+	VMStatusCrashLoopBackOff        = "CrashLoopBackOff"
+	VMStatusErrorUnschedulable      = "ErrorUnschedulable"
+	VMStatusErrImagePull            = "ErrImagePull"
+	VMStatusImagePullBackOff        = "ImagePullBackOff"
+	VMStatusErrorPvcNotFound        = "ErrorPvcNotFound"
+	VMStatusErrorDataVolumeNotFound = "ErrorDataVolumeNotFound"
+	VMStatusDataVolumeError         = "DataVolumeError"
+	VMStatusUnknown                 = "Unknown"
+	VMStatusWaitingForVolumeBinding = "WaitingForVolumeBinding"
+)
+
+var VM_ERROR_STATUSES = []string{
+	VMStatusCrashLoopBackOff,
+	VMStatusErrorUnschedulable,
+	VMStatusErrImagePull,
+	VMStatusImagePullBackOff,
+	VMStatusErrorPvcNotFound,
+	VMStatusErrorDataVolumeNotFound,
+	VMStatusDataVolumeError,
+	VMStatusUnknown,
+	VMStatusWaitingForVolumeBinding,
+}
+
+func isErrorStatus(status string) bool {
+	return slices.Contains(VM_ERROR_STATUSES, status)
+}

util/util.go

@@ -224,6 +224,9 @@ func itemValueMatchesFilter(itemValue gjson.Result, key string, queryValue strin
 	case "String":
 		// ANY search value can match (OR logic)
 		for _, search := range searchValues {
+			if key == "status.printableStatus" && search == "Error" && isErrorStatus(itemValue.Str) {
+				return true
+			}
 			if strings.Contains(strings.ToLower(itemValue.Str), strings.ToLower(search)) {
 				return true
 			}