Update pkg/handlers/generic/mutation/extraapiservercertsans/inject.go

faiq · dkoshkin · web-flow · commit da3bcb608f24 · 2024-04-04T15:43:46.000-04:00
Co-authored-by: Dimitri Koshkin &lt;dimitri.koshkin@nutanix.com&gt;
Signed-off-by: Faiq &lt;faiq.raza@nutanix.com&gt;
diff --git a/pkg/handlers/generic/mutation/extraapiservercertsans/inject.go b/pkg/handlers/generic/mutation/extraapiservercertsans/inject.go
@@ -70,6 +70,7 @@ func (h *extraAPIServerCertSANsPatchHandler) Mutate(
 	if err := h.client.Get(ctx, clusterKey, cluster); err != nil {
 		return err
 	}
+        TODO: move the defaults to a mutating webhook
 	defaultAPICertSANs := getDefaultAPIServerSANs(cluster)
 	extraAPIServerCertSANsVar, found, err := variables.Get[v1alpha1.ExtraAPIServerCertSANs](
 		vars,

Original file line number	Diff line number	Diff line change
`@@ -70,6 +70,7 @@ func (h *extraAPIServerCertSANsPatchHandler) Mutate(`
`70`	`70`	`if err := h.client.Get(ctx, clusterKey, cluster); err != nil {`
`71`	`71`	`return err`
`72`	`72`	`}`
	`73`	`+ TODO: move the defaults to a mutating webhook`
`73`	`74`	`defaultAPICertSANs := getDefaultAPIServerSANs(cluster)`
`74`	`75`	`extraAPIServerCertSANsVar, found, err := variables.Get[v1alpha1.ExtraAPIServerCertSANs](`
`75`	`76`	`vars,`