refactor : adjust WebhookManagedBy as per controller-runtime upgrade

rohanKanojia · rohanKanojia · commit d254509aa7fe · 2026-03-27T21:37:18.000+05:30
This method arguments were changed in controller-runtime v0.23.0.

Existing code of the form builder.WebhookManagedBy(mgr).For(&amp;corev1.Deployment{}) has to be changed to builder.WebhookManagedBy(mgr, &amp;appsv1.Deployment{})

Signed-off-by: Rohan Kumar &lt;rohaan@redhat.com&gt;
diff --git a/main.go b/main.go
@@ -213,10 +213,10 @@ func main() {
 		os.Exit(1)
 	}
 
-	if err := ctrl.NewWebhookManagedBy(mgr).For(&dwv1.DevWorkspace{}).Complete(); err != nil {
+	if err := ctrl.NewWebhookManagedBy(mgr, &dwv1.DevWorkspace{}).Complete(); err != nil {
 		setupLog.Error(err, "failed creating conversion webhook for DevWorkspaces v1alpha1")
 	}
-	if err := ctrl.NewWebhookManagedBy(mgr).For(&dwv2.DevWorkspace{}).Complete(); err != nil {
+	if err := ctrl.NewWebhookManagedBy(mgr, &dwv2.DevWorkspace{}).Complete(); err != nil {
 		setupLog.Error(err, "failed creating conversion webhook for DevWorkspaces v1alpha2")
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -213,10 +213,10 @@ func main() {`
`213`	`213`	`os.Exit(1)`
`214`	`214`	`}`
`215`	`215`
`216`		`- if err := ctrl.NewWebhookManagedBy(mgr).For(&dwv1.DevWorkspace{}).Complete(); err != nil {`
	`216`	`+ if err := ctrl.NewWebhookManagedBy(mgr, &dwv1.DevWorkspace{}).Complete(); err != nil {`
`217`	`217`	`setupLog.Error(err, "failed creating conversion webhook for DevWorkspaces v1alpha1")`
`218`	`218`	`}`
`219`		`- if err := ctrl.NewWebhookManagedBy(mgr).For(&dwv2.DevWorkspace{}).Complete(); err != nil {`
	`219`	`+ if err := ctrl.NewWebhookManagedBy(mgr, &dwv2.DevWorkspace{}).Complete(); err != nil {`
`220`	`220`	`setupLog.Error(err, "failed creating conversion webhook for DevWorkspaces v1alpha2")`
`221`	`221`	`}`
`222`	`222`