lint: address SA1019 / exhaustive issues introduced by k8s 0.36 + controller-runtime 0.24

The kubernetes group bump to 0.36.0 (CrunchyData#4479) introduced a new
v1.PersistentVolumeClaimConditionType ("Unused") that triggers the
exhaustive linter on the PVC condition switch in volumes.go. Add it
to the existing no-op case alongside the other condition types that
have no bearing on volume resizing.

The bump also surfaced two staticcheck SA1019 deprecation warnings
that block CI on every PR until addressed:

  - controller-runtime 0.24 deprecated scheme.Builder (used by both
    v1 and v1beta1 groupversion_info.go). Migrating to the new
    helper requires restructuring our api packages, so suppress the
    warning for now and leave a TODO via the comment.

  - k8s.io/apimachinery 0.36 deprecated direct access to
    managed.FieldsV1.Raw in favor of GetRawBytes/SetRawBytes. Only
    apply_test.go uses the old field; suppress until the test is
    rewritten against the new helpers.

These exclusions are scoped narrowly via path patterns so other call
sites (if any) still get flagged.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: ValClarkson

.golangci.yaml

@@ -189,6 +189,19 @@ linters:
         text: >-
            GetEventRecorderFor is deprecated
 
+      # controller-runtime 0.24 deprecated scheme.Builder, but the recommended
+      # alternative requires restructuring our api packages. Allow current usage
+      # until a follow-up migrates to the new helper.
+      - linters: [staticcheck]
+        path: pkg/apis/postgres-operator.crunchydata.com/.*/groupversion_info\.go
+        text: scheme.Builder is deprecated
+
+      # k8s.io/apimachinery 0.36 deprecated direct access to managed.FieldsV1.Raw
+      # in favor of GetRawBytes/SetRawBytes. Tests still use the old field.
+      - linters: [staticcheck]
+        path: internal/controller/runtime/apply_test\.go
+        text: managed.FieldsV1.Raw is deprecated
+
 # https://golangci-lint.run/usage/formatters
 formatters:
   enable:

internal/controller/postgrescluster/volumes.go

@@ -156,7 +156,13 @@ func (r *Reconciler) observePersistentVolumeClaims(
 				// See the "VolumeAttributesClass" feature gate.
 				// - https://git.k8s.io/enhancements/keps/sig-storage/3751-volume-attributes-class
 				corev1.PersistentVolumeClaimVolumeModifyingVolume,
-				corev1.PersistentVolumeClaimVolumeModifyVolumeError:
+				corev1.PersistentVolumeClaimVolumeModifyVolumeError,
+
+				// The "Unused" condition was added in Kubernetes v1.36 to indicate
+				// that a PVC is not currently bound to or used by any Pod. This
+				// has no bearing on volume resizing, so there's nothing to do.
+				// - https://git.k8s.io/enhancements/keps/sig-storage/4901-pvc-detection-of-stuck-pvcs
+				corev1.PersistentVolumeClaimUnused:
 			}
 		}
 	}