Make Logging options configurable using configfile.

Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>

cmd/webhook/app/webhook.go

@@ -95,7 +95,7 @@ func NewServerCommand(stopCh <-chan struct{}) *cobra.Command {
 				os.Exit(1)
 			}
 
-			if err := logf.ValidateAndApply(webhookFlags.Logging); err != nil {
+			if err := logf.ValidateAndApply(&webhookConfig.Logging); err != nil {
 				log.Error(err, "Failed to validate webhook flags")
 				os.Exit(1)
 			}
@@ -117,6 +117,11 @@ func NewServerCommand(stopCh <-chan struct{}) *cobra.Command {
 					log.Error(err, "Failed to set feature gates from config file")
 					os.Exit(1)
 				}
+
+				if err := logf.ValidateAndApply(&webhookConfig.Logging); err != nil {
+					log.Error(err, "Failed to validate webhook flags")
+					os.Exit(1)
+				}
 			}
 
 			srv, err := cmwebhook.NewCertManagerWebhookServer(log, *webhookConfig)

hack/k8s-codegen.sh

@@ -104,6 +104,7 @@ conversion_inputs=(
   internal/apis/acme/v1alpha3 \
   internal/apis/acme/v1beta1 \
   internal/apis/acme/v1 \
+  internal/apis/config/util \
   internal/apis/config/webhook/v1alpha1 \
   internal/apis/config/controller/v1alpha1 \
   internal/apis/meta/v1 \

internal/apis/config/controller/types.go

@@ -19,9 +19,8 @@ package controller
 import (
 	"time"
 
-	"k8s.io/component-base/logs"
-
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+	"k8s.io/component-base/logs"
 )
 
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

internal/apis/config/controller/v1alpha1/conversion.go

@@ -15,71 +15,3 @@ limitations under the License.
 */
 
 package v1alpha1
-
-import (
-	controller "github.com/cert-manager/cert-manager/internal/apis/config/controller"
-	v1alpha1 "github.com/cert-manager/cert-manager/pkg/apis/config/controller/v1alpha1"
-	conversion "k8s.io/apimachinery/pkg/conversion"
-	"k8s.io/component-base/logs"
-	logsapi "k8s.io/component-base/logs/api/v1"
-)
-
-func Convert_v1alpha1_ControllerConfiguration_To_controller_ControllerConfiguration(in *v1alpha1.ControllerConfiguration, out *controller.ControllerConfiguration, s conversion.Scope) error {
-	if err := autoConvert_v1alpha1_ControllerConfiguration_To_controller_ControllerConfiguration(in, out, s); err != nil {
-		return err
-	}
-	return nil
-}
-
-func Convert_controller_ControllerConfiguration_To_v1alpha1_ControllerConfiguration(in *controller.ControllerConfiguration, out *v1alpha1.ControllerConfiguration, s conversion.Scope) error {
-	if err := autoConvert_controller_ControllerConfiguration_To_v1alpha1_ControllerConfiguration(in, out, s); err != nil {
-		return err
-	}
-	return nil
-}
-
-func Convert_Pointer_float32_To_float32(in **float32, out *float32, s conversion.Scope) error {
-	if *in == nil {
-		*out = 0
-		return nil
-	}
-	*out = float32(**in)
-	return nil
-}
-
-func Convert_float32_To_Pointer_float32(in *float32, out **float32, s conversion.Scope) error {
-	temp := float32(*in)
-	*out = &temp
-	return nil
-}
-
-func Convert_Pointer_int32_To_int(in **int32, out *int, s conversion.Scope) error {
-	if *in == nil {
-		*out = 0
-		return nil
-	}
-	*out = int(**in)
-	return nil
-}
-
-func Convert_int_To_Pointer_int32(in *int, out **int32, s conversion.Scope) error {
-	temp := int32(*in)
-	*out = &temp
-	return nil
-}
-
-func Convert_Pointer_v1_LoggingConfiguration_To_v1_LoggingConfiguration(in **logsapi.LoggingConfiguration, out *logsapi.LoggingConfiguration, s conversion.Scope) error {
-	if *in == nil {
-		temp := logs.NewOptions()
-		temp.DeepCopyInto(out)
-		return nil
-	}
-	(*in).DeepCopyInto(out)
-	return nil
-}
-
-func Convert_v1_LoggingConfiguration_To_Pointer_v1_LoggingConfiguration(in *logsapi.LoggingConfiguration, out **logsapi.LoggingConfiguration, s conversion.Scope) error {
-	temp := in.DeepCopy()
-	*out = temp
-	return nil
-}