Add custom conversion funcs not directly into pkg/api.Scheme
This commit is contained in:
		| @@ -40,7 +40,7 @@ func addConversionFuncs(scheme *runtime.Scheme) error { | |||||||
| 		return err | 		return err | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
| 	return api.Scheme.AddFieldLabelConversionFunc("apps/v1beta1", "StatefulSet", | 	return scheme.AddFieldLabelConversionFunc("apps/v1beta1", "StatefulSet", | ||||||
| 		func(label, value string) (string, string, error) { | 		func(label, value string) (string, string, error) { | ||||||
| 			switch label { | 			switch label { | ||||||
| 			case "metadata.name", "metadata.namespace", "status.successful": | 			case "metadata.name", "metadata.namespace", "status.successful": | ||||||
|   | |||||||
| @@ -21,7 +21,6 @@ import ( | |||||||
|  |  | ||||||
| 	"k8s.io/apimachinery/pkg/conversion" | 	"k8s.io/apimachinery/pkg/conversion" | ||||||
| 	"k8s.io/apimachinery/pkg/runtime" | 	"k8s.io/apimachinery/pkg/runtime" | ||||||
| 	"k8s.io/kubernetes/pkg/api" |  | ||||||
| 	v1 "k8s.io/kubernetes/pkg/api/v1" | 	v1 "k8s.io/kubernetes/pkg/api/v1" | ||||||
| 	"k8s.io/kubernetes/pkg/apis/batch" | 	"k8s.io/kubernetes/pkg/apis/batch" | ||||||
| ) | ) | ||||||
| @@ -36,7 +35,7 @@ func addConversionFuncs(scheme *runtime.Scheme) error { | |||||||
| 		return err | 		return err | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
| 	return api.Scheme.AddFieldLabelConversionFunc("batch/v1", "Job", | 	return scheme.AddFieldLabelConversionFunc("batch/v1", "Job", | ||||||
| 		func(label, value string) (string, string, error) { | 		func(label, value string) (string, string, error) { | ||||||
| 			switch label { | 			switch label { | ||||||
| 			case "metadata.name", "metadata.namespace", "status.successful": | 			case "metadata.name", "metadata.namespace", "status.successful": | ||||||
|   | |||||||
| @@ -21,7 +21,6 @@ import ( | |||||||
|  |  | ||||||
| 	"k8s.io/apimachinery/pkg/conversion" | 	"k8s.io/apimachinery/pkg/conversion" | ||||||
| 	"k8s.io/apimachinery/pkg/runtime" | 	"k8s.io/apimachinery/pkg/runtime" | ||||||
| 	"k8s.io/kubernetes/pkg/api" |  | ||||||
| 	v1 "k8s.io/kubernetes/pkg/api/v1" | 	v1 "k8s.io/kubernetes/pkg/api/v1" | ||||||
| 	"k8s.io/kubernetes/pkg/apis/batch" | 	"k8s.io/kubernetes/pkg/apis/batch" | ||||||
| ) | ) | ||||||
| @@ -38,7 +37,7 @@ func addConversionFuncs(scheme *runtime.Scheme) error { | |||||||
|  |  | ||||||
| 	// Add field label conversions for kinds having selectable nothing but ObjectMeta fields. | 	// Add field label conversions for kinds having selectable nothing but ObjectMeta fields. | ||||||
| 	for _, kind := range []string{"Job", "JobTemplate", "CronJob"} { | 	for _, kind := range []string{"Job", "JobTemplate", "CronJob"} { | ||||||
| 		err = api.Scheme.AddFieldLabelConversionFunc("batch/v2alpha1", kind, | 		err = scheme.AddFieldLabelConversionFunc("batch/v2alpha1", kind, | ||||||
| 			func(label, value string) (string, string, error) { | 			func(label, value string) (string, string, error) { | ||||||
| 				switch label { | 				switch label { | ||||||
| 				case "metadata.name", "metadata.namespace", "status.successful": | 				case "metadata.name", "metadata.namespace", "status.successful": | ||||||
|   | |||||||
| @@ -25,7 +25,6 @@ go_library( | |||||||
|     ], |     ], | ||||||
|     tags = ["automanaged"], |     tags = ["automanaged"], | ||||||
|     deps = [ |     deps = [ | ||||||
|         "//pkg/api:go_default_library", |  | ||||||
|         "//pkg/apis/certificates:go_default_library", |         "//pkg/apis/certificates:go_default_library", | ||||||
|         "//vendor:github.com/gogo/protobuf/proto", |         "//vendor:github.com/gogo/protobuf/proto", | ||||||
|         "//vendor:github.com/ugorji/go/codec", |         "//vendor:github.com/ugorji/go/codec", | ||||||
|   | |||||||
| @@ -20,13 +20,12 @@ import ( | |||||||
| 	"fmt" | 	"fmt" | ||||||
|  |  | ||||||
| 	"k8s.io/apimachinery/pkg/runtime" | 	"k8s.io/apimachinery/pkg/runtime" | ||||||
| 	"k8s.io/kubernetes/pkg/api" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| func addConversionFuncs(scheme *runtime.Scheme) error { | func addConversionFuncs(scheme *runtime.Scheme) error { | ||||||
| 	// Add non-generated conversion functions here. Currently there are none. | 	// Add non-generated conversion functions here. Currently there are none. | ||||||
|  |  | ||||||
| 	return api.Scheme.AddFieldLabelConversionFunc(SchemeGroupVersion.String(), "CertificateSigningRequest", | 	return scheme.AddFieldLabelConversionFunc(SchemeGroupVersion.String(), "CertificateSigningRequest", | ||||||
| 		func(label, value string) (string, string, error) { | 		func(label, value string) (string, string, error) { | ||||||
| 			switch label { | 			switch label { | ||||||
| 			case "metadata.name": | 			case "metadata.name": | ||||||
|   | |||||||
| @@ -23,7 +23,6 @@ import ( | |||||||
| 	"k8s.io/apimachinery/pkg/conversion" | 	"k8s.io/apimachinery/pkg/conversion" | ||||||
| 	"k8s.io/apimachinery/pkg/runtime" | 	"k8s.io/apimachinery/pkg/runtime" | ||||||
| 	"k8s.io/apimachinery/pkg/util/intstr" | 	"k8s.io/apimachinery/pkg/util/intstr" | ||||||
| 	"k8s.io/kubernetes/pkg/api" |  | ||||||
| 	v1 "k8s.io/kubernetes/pkg/api/v1" | 	v1 "k8s.io/kubernetes/pkg/api/v1" | ||||||
| 	"k8s.io/kubernetes/pkg/apis/extensions" | 	"k8s.io/kubernetes/pkg/apis/extensions" | ||||||
| ) | ) | ||||||
| @@ -49,7 +48,7 @@ func addConversionFuncs(scheme *runtime.Scheme) error { | |||||||
| 	// Add field label conversions for kinds having selectable nothing but ObjectMeta fields. | 	// Add field label conversions for kinds having selectable nothing but ObjectMeta fields. | ||||||
| 	for _, k := range []string{"DaemonSet", "Deployment", "Ingress"} { | 	for _, k := range []string{"DaemonSet", "Deployment", "Ingress"} { | ||||||
| 		kind := k // don't close over range variables | 		kind := k // don't close over range variables | ||||||
| 		err = api.Scheme.AddFieldLabelConversionFunc("extensions/v1beta1", kind, | 		err = scheme.AddFieldLabelConversionFunc("extensions/v1beta1", kind, | ||||||
| 			func(label, value string) (string, string, error) { | 			func(label, value string) (string, string, error) { | ||||||
| 				switch label { | 				switch label { | ||||||
| 				case "metadata.name", "metadata.namespace": | 				case "metadata.name", "metadata.namespace": | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Dr. Stefan Schimanski
					Dr. Stefan Schimanski