webhooks/datavolume: comment about remove check of claim adoption (ku…

…bevirt#3481) 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]>
deckhouse · Oct 29, 2024 · f8a64e2 · f8a64e2
1 parent f6acbaf
commit f8a64e2
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)