change InstallWebhookConfig to pointer

Author: Mengqi Yu

example/main.go

@@ -110,7 +110,7 @@ func main() {
 	as, err := webhook.NewServer("foo-admission-server", mgr, webhook.ServerOptions{
 		Port:                 9876,
 		CertDir:              "/tmp/cert",
-		InstallWebhookConfig: installWebhookConfig,
+		InstallWebhookConfig: &installWebhookConfig,
 		BootstrapOptions: &webhook.BootstrapOptions{
 			Secret: &apitypes.NamespacedName{
 				Namespace: "default",

pkg/webhook/bootstrap.go

@@ -63,6 +63,10 @@ func (s *Server) setServerDefault() {
 	if len(s.CertDir) == 0 {
 		s.CertDir = path.Join("k8s-webhook-server", "cert")
 	}
+	if s.InstallWebhookConfig == nil {
+		iwc := true
+		s.InstallWebhookConfig = &iwc
+	}
 
 	if s.Client == nil {
 		cfg, err := config.GetConfig()

pkg/webhook/server.go

@@ -56,7 +56,7 @@ type ServerOptions struct {
 
 	// InstallWebhookConfig controls if the server will automatically create webhook related objects
 	// during bootstrapping. e.g. webhookConfiguration, service and secret.
-	InstallWebhookConfig bool
+	InstallWebhookConfig *bool
 
 	// BootstrapOptions contains the options for bootstrapping the admission server.
 	*BootstrapOptions
@@ -190,7 +190,12 @@ var _ manager.Runnable = &Server{}
 // Start runs the server.
 // It will install the webhook related resources depend on the server configuration.
 func (s *Server) Start(stop <-chan struct{}) error {
-	if s.InstallWebhookConfig {
+	s.once.Do(s.setDefault)
+	if s.err != nil {
+		return s.err
+	}
+
+	if s.InstallWebhookConfig != nil && *s.InstallWebhookConfig {
 		log.Info("installing webhook configuration in cluster")
 		err := s.InstallWebhookManifests()
 		if err != nil {
@@ -200,6 +205,10 @@ func (s *Server) Start(stop <-chan struct{}) error {
 		log.Info("webhook installer is disabled")
 	}
 
+	return s.run(stop)
+}
+
+func (s *Server) run(stop <-chan struct{}) error {
 	srv := &http.Server{
 		Addr:    fmt.Sprintf(":%v", s.Port),
 		Handler: s.sMux,