PB-7579: Creation of webhook controller in runtime should be only if stork is running with a driver. (#1813)

Author: diptiranjanpx

cmd/stork/stork.go

@@ -588,12 +588,12 @@ func runStork(mgr manager.Manager, ctx context.Context, d volume.Driver, recorde
 				log.Fatalf("Error initializing cluster domain controllers: %v", err)
 			}
 		}
-	}
 
-	if c.Bool("webhook-controller") {
-		log.Infof("Creating mutating webhook after leader election")
-		if err := webhookadmission.CreateMutateWebhookRuntime(); err != nil {
-			log.Fatalf("Error creating webhook: %v", err)
+		if c.Bool("webhook-controller") {
+			log.Infof("Creating mutating webhook after leader election")
+			if err := webhookadmission.CreateMutateWebhookRuntime(); err != nil {
+				log.Fatalf("Error creating webhook: %v", err)
+			}
 		}
 	}
 

pkg/webhookadmission/utils.go

@@ -234,6 +234,7 @@ func createWebhookV1(caBundle []byte, ns string) error {
 			if err != nil {
 				log.Errorf("Unable to create webhook configuration: %v", err)
 			}
+			log.Debugf("Stork webhook v1 created: %v", webhookName)
 		}
 		return err
 	}

pkg/webhookadmission/webhook.go

@@ -279,6 +279,7 @@ func (c *Controller) Stop() error {
 		log.Errorf("unable to delete webhook configuration, %v", err)
 		return err
 	}
+	log.Infof("Successfully deleted webhook configuration %s", storkAdmissionController)
 	ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second)
 	defer cancel()