Merge pull request #73 from anfernee/fix-panic

Fix un-initialized once that causes panic

Author: k8s-ci-robot

pkg/admission/controller.go

@@ -37,7 +37,7 @@ type CertProvisioner struct {
 	CertGenerator generator.CertGenerator
 	CertWriter    writer.CertWriter
 
-	once *sync.Once
+	once sync.Once
 }
 
 // Sync takes a runtime.Object which is expected to be either a MutatingWebhookConfiguration or

pkg/admission/controller_test.go

@@ -0,0 +1,20 @@
+package admission
+
+import (
+	"testing"
+
+	"k8s.io/api/admissionregistration/v1beta1"
+	"sigs.k8s.io/controller-runtime/pkg/client/fake"
+)
+
+func TestCertProvisionerInit(t *testing.T) {
+	p := &CertProvisioner{
+		Client: fake.NewFakeClient(),
+	}
+	config := &v1beta1.MutatingWebhookConfiguration{}
+
+	err := p.Sync(config)
+	if err != nil {
+		t.Fatalf("expect nil; got %q", err)
+	}
+}