webhooks/datavolume: comment about remove check of claim adoption

There is an open issue in github to remove the claim adoption feature gate: kubevirt#3480 Once the FG is removed we should remove the check in the webhook that verifies if it allowed to have an existing PVC. Signed-off-by: Shelly Kagan <[email protected]>
ShellyKa13 · Oct 28, 2024 · 0da05aa · 0da05aa
1 parent 75f00e2
commit 0da05aa
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/pkg/apiserver/webhooks/datavolume-validate.go b/pkg/apiserver/webhooks/datavolume-validate.go
@@ -492,6 +492,9 @@ func (wh *dataVolumeValidatingWebhook) Admit(ar admissionv1.AdmissionReview) *ad
 				return toAdmissionResponseError(err)
 			}
 		} else {
+			// We are planning to remove the Claim adoption feature gate
+			// https://github.com/kubevirt/containerized-data-importer/issues/3480
+			// once removed we should remove the check for it in this webhook.
 			allow, err := cc.ClaimMayExistBeforeDataVolume(wh.controllerRuntimeClient, pvc, &dv)
 			if err != nil {
 				return toAdmissionResponseError(err)