From ac8adc6f4c500a92182e920542eff17c704b20c5 Mon Sep 17 00:00:00 2001 From: wojtekt Date: Thu, 24 Oct 2019 13:18:31 +0200 Subject: [PATCH] Cleanup explicitly registered functions Kubernetes-commit: d7011f1bdbee285cdfc9bdb0f5b0716f4e02adfd --- scale/scheme/appsv1beta1/conversion.go | 14 -------------- scale/scheme/appsv1beta1/register.go | 2 +- scale/scheme/appsv1beta2/conversion.go | 14 -------------- scale/scheme/appsv1beta2/register.go | 2 +- scale/scheme/autoscalingv1/conversion.go | 15 --------------- scale/scheme/autoscalingv1/register.go | 2 +- scale/scheme/extensionsv1beta1/conversion.go | 14 -------------- scale/scheme/extensionsv1beta1/register.go | 2 +- 8 files changed, 4 insertions(+), 61 deletions(-) diff --git a/scale/scheme/appsv1beta1/conversion.go b/scale/scheme/appsv1beta1/conversion.go index af062b3c..f271c825 100644 --- a/scale/scheme/appsv1beta1/conversion.go +++ b/scale/scheme/appsv1beta1/conversion.go @@ -22,23 +22,9 @@ import ( v1beta1 "k8s.io/api/apps/v1beta1" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" "k8s.io/apimachinery/pkg/conversion" - "k8s.io/apimachinery/pkg/runtime" scheme "k8s.io/client-go/scale/scheme" ) -// addConversions registers conversions between the internal version -// of Scale and supported external versions of Scale. -func addConversionFuncs(scheme *runtime.Scheme) error { - err := scheme.AddConversionFuncs( - Convert_scheme_ScaleStatus_To_v1beta1_ScaleStatus, - Convert_v1beta1_ScaleStatus_To_scheme_ScaleStatus, - ) - if err != nil { - return err - } - - return nil -} func Convert_scheme_ScaleStatus_To_v1beta1_ScaleStatus(in *scheme.ScaleStatus, out *v1beta1.ScaleStatus, s conversion.Scope) error { out.Replicas = in.Replicas out.Selector = nil diff --git a/scale/scheme/appsv1beta1/register.go b/scale/scheme/appsv1beta1/register.go index a684f2d5..f11fcbd0 100644 --- a/scale/scheme/appsv1beta1/register.go +++ b/scale/scheme/appsv1beta1/register.go @@ -41,5 +41,5 @@ func init() { // We only register manually written functions here. The registration of the // generated functions takes place in the generated files. The separation // makes the code compile even when the generated files are missing. - localSchemeBuilder.Register(addConversionFuncs) + localSchemeBuilder.Register() } diff --git a/scale/scheme/appsv1beta2/conversion.go b/scale/scheme/appsv1beta2/conversion.go index f07de6bd..35d15c30 100644 --- a/scale/scheme/appsv1beta2/conversion.go +++ b/scale/scheme/appsv1beta2/conversion.go @@ -22,23 +22,9 @@ import ( v1beta2 "k8s.io/api/apps/v1beta2" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" "k8s.io/apimachinery/pkg/conversion" - "k8s.io/apimachinery/pkg/runtime" scheme "k8s.io/client-go/scale/scheme" ) -// addConversions registers conversions between the internal version -// of Scale and supported external versions of Scale. -func addConversionFuncs(scheme *runtime.Scheme) error { - err := scheme.AddConversionFuncs( - Convert_scheme_ScaleStatus_To_v1beta2_ScaleStatus, - Convert_v1beta2_ScaleStatus_To_scheme_ScaleStatus, - ) - if err != nil { - return err - } - - return nil -} func Convert_scheme_ScaleStatus_To_v1beta2_ScaleStatus(in *scheme.ScaleStatus, out *v1beta2.ScaleStatus, s conversion.Scope) error { out.Replicas = in.Replicas out.Selector = nil diff --git a/scale/scheme/appsv1beta2/register.go b/scale/scheme/appsv1beta2/register.go index 88de0893..5e8a5d20 100644 --- a/scale/scheme/appsv1beta2/register.go +++ b/scale/scheme/appsv1beta2/register.go @@ -41,5 +41,5 @@ func init() { // We only register manually written functions here. The registration of the // generated functions takes place in the generated files. The separation // makes the code compile even when the generated files are missing. - localSchemeBuilder.Register(addConversionFuncs) + localSchemeBuilder.Register() } diff --git a/scale/scheme/autoscalingv1/conversion.go b/scale/scheme/autoscalingv1/conversion.go index a775bcc2..36ef82b9 100644 --- a/scale/scheme/autoscalingv1/conversion.go +++ b/scale/scheme/autoscalingv1/conversion.go @@ -22,24 +22,9 @@ import ( v1 "k8s.io/api/autoscaling/v1" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" "k8s.io/apimachinery/pkg/conversion" - "k8s.io/apimachinery/pkg/runtime" scheme "k8s.io/client-go/scale/scheme" ) -// addConversions registers conversions between the internal version -// of Scale and supported external versions of Scale. -func addConversionFuncs(scheme *runtime.Scheme) error { - err := scheme.AddConversionFuncs( - Convert_scheme_ScaleStatus_To_v1_ScaleStatus, - Convert_v1_ScaleStatus_To_scheme_ScaleStatus, - ) - if err != nil { - return err - } - - return nil -} - func Convert_scheme_ScaleStatus_To_v1_ScaleStatus(in *scheme.ScaleStatus, out *v1.ScaleStatus, s conversion.Scope) error { out.Replicas = in.Replicas out.Selector = "" diff --git a/scale/scheme/autoscalingv1/register.go b/scale/scheme/autoscalingv1/register.go index b15701c4..4339376c 100644 --- a/scale/scheme/autoscalingv1/register.go +++ b/scale/scheme/autoscalingv1/register.go @@ -41,5 +41,5 @@ func init() { // We only register manually written functions here. The registration of the // generated functions takes place in the generated files. The separation // makes the code compile even when the generated files are missing. - localSchemeBuilder.Register(addConversionFuncs) + localSchemeBuilder.Register() } diff --git a/scale/scheme/extensionsv1beta1/conversion.go b/scale/scheme/extensionsv1beta1/conversion.go index 1b6b9e61..821eb33d 100644 --- a/scale/scheme/extensionsv1beta1/conversion.go +++ b/scale/scheme/extensionsv1beta1/conversion.go @@ -22,23 +22,9 @@ import ( v1beta1 "k8s.io/api/extensions/v1beta1" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" "k8s.io/apimachinery/pkg/conversion" - "k8s.io/apimachinery/pkg/runtime" scheme "k8s.io/client-go/scale/scheme" ) -// addConversions registers conversions between the internal version -// of Scale and supported external versions of Scale. -func addConversionFuncs(scheme *runtime.Scheme) error { - err := scheme.AddConversionFuncs( - Convert_scheme_ScaleStatus_To_v1beta1_ScaleStatus, - Convert_v1beta1_ScaleStatus_To_scheme_ScaleStatus, - ) - if err != nil { - return err - } - - return nil -} func Convert_scheme_ScaleStatus_To_v1beta1_ScaleStatus(in *scheme.ScaleStatus, out *v1beta1.ScaleStatus, s conversion.Scope) error { out.Replicas = in.Replicas out.Selector = nil diff --git a/scale/scheme/extensionsv1beta1/register.go b/scale/scheme/extensionsv1beta1/register.go index aed1174e..248a0071 100644 --- a/scale/scheme/extensionsv1beta1/register.go +++ b/scale/scheme/extensionsv1beta1/register.go @@ -41,5 +41,5 @@ func init() { // We only register manually written functions here. The registration of the // generated functions takes place in the generated files. The separation // makes the code compile even when the generated files are missing. - localSchemeBuilder.Register(addConversionFuncs) + localSchemeBuilder.Register() }