Improve API package dependencies on controller-runtime

laozc · laozc · commit 3a2771b76a49 · 2023-08-17T09:26:09.000+08:00
diff --git a/apis/v1alpha3/groupversion_info.go b/apis/v1alpha3/groupversion_info.go
@@ -17,8 +17,9 @@ limitations under the License.
 package v1alpha3
 
 import (
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/runtime/schema"
-	"sigs.k8s.io/controller-runtime/pkg/scheme"
 )
 
 const (
@@ -33,12 +34,21 @@ var (
 	// GroupVersion is group version used to register these objects.
 	GroupVersion = schema.GroupVersion{Group: GroupName, Version: Version}
 
-	// SchemeBuilder is used to add go types to the GroupVersionKind scheme.
-	SchemeBuilder = &scheme.Builder{GroupVersion: GroupVersion}
+	// schemeBuilder is used to add go types to the GroupVersionKind scheme.
+	schemeBuilder = runtime.NewSchemeBuilder(addKnownTypes)
 
 	// AddToScheme adds the types in this group-version to the given scheme.
-	AddToScheme = SchemeBuilder.AddToScheme
+	AddToScheme = schemeBuilder.AddToScheme
+
+	// objectTypes contains all types to be registered to the GroupVersion.
+	objectTypes = []runtime.Object{}
 
 	// localSchemeBuilder is used for type conversions.
-	localSchemeBuilder = SchemeBuilder.SchemeBuilder
+	localSchemeBuilder = schemeBuilder
 )
+
+func addKnownTypes(scheme *runtime.Scheme) error {
+	scheme.AddKnownTypes(GroupVersion, objectTypes...)
+	metav1.AddToGroupVersion(scheme, GroupVersion)
+	return nil
+}
diff --git a/apis/v1alpha3/haproxyloadbalancer_types.go b/apis/v1alpha3/haproxyloadbalancer_types.go
@@ -89,5 +89,5 @@ type HAProxyLoadBalancerList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&HAProxyLoadBalancer{}, &HAProxyLoadBalancerList{})
+	objectTypes = append(objectTypes, &HAProxyLoadBalancer{}, &HAProxyLoadBalancerList{})
 }
diff --git a/apis/v1alpha3/vspherecluster_types.go b/apis/v1alpha3/vspherecluster_types.go
@@ -123,5 +123,5 @@ type VSphereClusterList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereCluster{}, &VSphereClusterList{})
+	objectTypes = append(objectTypes, &VSphereCluster{}, &VSphereClusterList{})
 }
diff --git a/apis/v1alpha3/vsphereclusteridentity_types.go b/apis/v1alpha3/vsphereclusteridentity_types.go
@@ -105,5 +105,5 @@ type VSphereClusterIdentityList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereClusterIdentity{}, &VSphereClusterIdentityList{})
+	objectTypes = append(objectTypes, &VSphereClusterIdentity{}, &VSphereClusterIdentityList{})
 }
diff --git a/apis/v1alpha3/vspheredeploymentzone_types.go b/apis/v1alpha3/vspheredeploymentzone_types.go
@@ -105,5 +105,5 @@ type VSphereDeploymentZoneList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereDeploymentZone{}, &VSphereDeploymentZoneList{})
+	objectTypes = append(objectTypes, &VSphereDeploymentZone{}, &VSphereDeploymentZoneList{})
 }
diff --git a/apis/v1alpha3/vspherefailuredomain_types.go b/apis/v1alpha3/vspherefailuredomain_types.go
@@ -114,5 +114,5 @@ type VSphereFailureDomainList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereFailureDomain{}, &VSphereFailureDomainList{})
+	objectTypes = append(objectTypes, &VSphereFailureDomain{}, &VSphereFailureDomainList{})
 }
diff --git a/apis/v1alpha3/vspheremachine_types.go b/apis/v1alpha3/vspheremachine_types.go
@@ -142,5 +142,5 @@ type VSphereMachineList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereMachine{}, &VSphereMachineList{})
+	objectTypes = append(objectTypes, &VSphereMachine{}, &VSphereMachineList{})
 }
diff --git a/apis/v1alpha3/vspheremachinetemplate_types.go b/apis/v1alpha3/vspheremachinetemplate_types.go
@@ -52,5 +52,5 @@ type VSphereMachineTemplateList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereMachineTemplate{}, &VSphereMachineTemplateList{})
+	objectTypes = append(objectTypes, &VSphereMachineTemplate{}, &VSphereMachineTemplateList{})
 }
diff --git a/apis/v1alpha3/vspherevm_types.go b/apis/v1alpha3/vspherevm_types.go
@@ -161,5 +161,5 @@ type VSphereVMList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereVM{}, &VSphereVMList{})
+	objectTypes = append(objectTypes, &VSphereVM{}, &VSphereVMList{})
 }
diff --git a/apis/v1alpha3/zz_generated.deepcopy.go b/apis/v1alpha3/zz_generated.deepcopy.go
diff --git a/apis/v1alpha4/groupversion_info.go b/apis/v1alpha4/groupversion_info.go
@@ -17,8 +17,9 @@ limitations under the License.
 package v1alpha4
 
 import (
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/runtime/schema"
-	"sigs.k8s.io/controller-runtime/pkg/scheme"
 )
 
 const (
@@ -33,12 +34,21 @@ var (
 	// GroupVersion is group version used to register these objects.
 	GroupVersion = schema.GroupVersion{Group: GroupName, Version: Version}
 
-	// SchemeBuilder is used to add go types to the GroupVersionKind scheme.
-	SchemeBuilder = &scheme.Builder{GroupVersion: GroupVersion}
+	// schemeBuilder is used to add go types to the GroupVersionKind scheme.
+	schemeBuilder = runtime.NewSchemeBuilder(addKnownTypes)
 
 	// AddToScheme adds the types in this group-version to the given scheme.
-	AddToScheme = SchemeBuilder.AddToScheme
+	AddToScheme = schemeBuilder.AddToScheme
+
+	// objectTypes contains all types to be registered to the GroupVersion.
+	objectTypes = []runtime.Object{}
 
 	// localSchemeBuilder is used for type conversions.
-	localSchemeBuilder = SchemeBuilder.SchemeBuilder
+	localSchemeBuilder = schemeBuilder
 )
+
+func addKnownTypes(scheme *runtime.Scheme) error {
+	scheme.AddKnownTypes(GroupVersion, objectTypes...)
+	metav1.AddToGroupVersion(scheme, GroupVersion)
+	return nil
+}
diff --git a/apis/v1alpha4/vspherecluster_types.go b/apis/v1alpha4/vspherecluster_types.go
@@ -101,5 +101,5 @@ type VSphereClusterList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereCluster{}, &VSphereClusterList{})
+	objectTypes = append(objectTypes, &VSphereCluster{}, &VSphereClusterList{})
 }
diff --git a/apis/v1alpha4/vsphereclusteridentity_types.go b/apis/v1alpha4/vsphereclusteridentity_types.go
@@ -105,5 +105,5 @@ type VSphereClusterIdentityList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereClusterIdentity{}, &VSphereClusterIdentityList{})
+	objectTypes = append(objectTypes, &VSphereClusterIdentity{}, &VSphereClusterIdentityList{})
 }
diff --git a/apis/v1alpha4/vsphereclustertemplate_types.go b/apis/v1alpha4/vsphereclustertemplate_types.go
@@ -52,7 +52,7 @@ type VSphereClusterTemplateList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereClusterTemplate{}, &VSphereClusterTemplateList{})
+	objectTypes = append(objectTypes, &VSphereClusterTemplate{}, &VSphereClusterTemplateList{})
 }
 
 type VSphereClusterTemplateResource struct {
diff --git a/apis/v1alpha4/vspheredeploymentzone_types.go b/apis/v1alpha4/vspheredeploymentzone_types.go
@@ -113,5 +113,5 @@ type VSphereDeploymentZoneList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereDeploymentZone{}, &VSphereDeploymentZoneList{})
+	objectTypes = append(objectTypes, &VSphereDeploymentZone{}, &VSphereDeploymentZoneList{})
 }
diff --git a/apis/v1alpha4/vspherefailuredomain_types.go b/apis/v1alpha4/vspherefailuredomain_types.go
@@ -115,5 +115,5 @@ type VSphereFailureDomainList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereFailureDomain{}, &VSphereFailureDomainList{})
+	objectTypes = append(objectTypes, &VSphereFailureDomain{}, &VSphereFailureDomainList{})
 }
diff --git a/apis/v1alpha4/vspheremachine_types.go b/apis/v1alpha4/vspheremachine_types.go
@@ -142,5 +142,5 @@ type VSphereMachineList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereMachine{}, &VSphereMachineList{})
+	objectTypes = append(objectTypes, &VSphereMachine{}, &VSphereMachineList{})
 }
diff --git a/apis/v1alpha4/vspheremachinetemplate_types.go b/apis/v1alpha4/vspheremachinetemplate_types.go
@@ -52,5 +52,5 @@ type VSphereMachineTemplateList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereMachineTemplate{}, &VSphereMachineTemplateList{})
+	objectTypes = append(objectTypes, &VSphereMachineTemplate{}, &VSphereMachineTemplateList{})
 }
diff --git a/apis/v1alpha4/vspherevm_types.go b/apis/v1alpha4/vspherevm_types.go
@@ -161,5 +161,5 @@ type VSphereVMList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereVM{}, &VSphereVMList{})
+	objectTypes = append(objectTypes, &VSphereVM{}, &VSphereVMList{})
 }
diff --git a/apis/v1alpha4/zz_generated.deepcopy.go b/apis/v1alpha4/zz_generated.deepcopy.go
diff --git a/apis/v1beta1/groupversion_info.go b/apis/v1beta1/groupversion_info.go
@@ -17,8 +17,9 @@ limitations under the License.
 package v1beta1
 
 import (
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/runtime/schema"
-	"sigs.k8s.io/controller-runtime/pkg/scheme"
 )
 
 const (
@@ -33,9 +34,18 @@ var (
 	// GroupVersion is group version used to register these objects.
 	GroupVersion = schema.GroupVersion{Group: GroupName, Version: Version}
 
-	// SchemeBuilder is used to add go types to the GroupVersionKind scheme.
-	SchemeBuilder = &scheme.Builder{GroupVersion: GroupVersion}
+	// schemeBuilder is used to add go types to the GroupVersionKind scheme.
+	schemeBuilder = runtime.NewSchemeBuilder(addKnownTypes)
 
 	// AddToScheme adds the types in this group-version to the given scheme.
-	AddToScheme = SchemeBuilder.AddToScheme
+	AddToScheme = schemeBuilder.AddToScheme
+
+	// objectTypes contains all types to be registered to the GroupVersion.
+	objectTypes = []runtime.Object{}
 )
+
+func addKnownTypes(scheme *runtime.Scheme) error {
+	scheme.AddKnownTypes(GroupVersion, objectTypes...)
+	metav1.AddToGroupVersion(scheme, GroupVersion)
+	return nil
+}
diff --git a/apis/v1beta1/vspherecluster_types.go b/apis/v1beta1/vspherecluster_types.go
@@ -134,5 +134,5 @@ type VSphereClusterList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereCluster{}, &VSphereClusterList{})
+	objectTypes = append(objectTypes, &VSphereCluster{}, &VSphereClusterList{})
 }
diff --git a/apis/v1beta1/vsphereclusteridentity_types.go b/apis/v1beta1/vsphereclusteridentity_types.go
@@ -101,5 +101,5 @@ type VSphereClusterIdentityList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereClusterIdentity{}, &VSphereClusterIdentityList{})
+	objectTypes = append(objectTypes, &VSphereClusterIdentity{}, &VSphereClusterIdentityList{})
 }
diff --git a/apis/v1beta1/vsphereclustertemplate_types.go b/apis/v1beta1/vsphereclustertemplate_types.go
@@ -48,7 +48,7 @@ type VSphereClusterTemplateList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereClusterTemplate{}, &VSphereClusterTemplateList{})
+	objectTypes = append(objectTypes, &VSphereClusterTemplate{}, &VSphereClusterTemplateList{})
 }
 
 type VSphereClusterTemplateResource struct {
diff --git a/apis/v1beta1/vspheredeploymentzone_types.go b/apis/v1beta1/vspheredeploymentzone_types.go
@@ -116,5 +116,5 @@ type VSphereDeploymentZoneList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereDeploymentZone{}, &VSphereDeploymentZoneList{})
+	objectTypes = append(objectTypes, &VSphereDeploymentZone{}, &VSphereDeploymentZoneList{})
 }
diff --git a/apis/v1beta1/vspherefailuredomain_types.go b/apis/v1beta1/vspherefailuredomain_types.go
@@ -112,5 +112,5 @@ type VSphereFailureDomainList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereFailureDomain{}, &VSphereFailureDomainList{})
+	objectTypes = append(objectTypes, &VSphereFailureDomain{}, &VSphereFailureDomainList{})
 }
diff --git a/apis/v1beta1/vspheremachine_types.go b/apis/v1beta1/vspheremachine_types.go
@@ -165,5 +165,5 @@ type VSphereMachineList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereMachine{}, &VSphereMachineList{})
+	objectTypes = append(objectTypes, &VSphereMachine{}, &VSphereMachineList{})
 }
diff --git a/apis/v1beta1/vspheremachinetemplate_types.go b/apis/v1beta1/vspheremachinetemplate_types.go
@@ -48,5 +48,5 @@ type VSphereMachineTemplateList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereMachineTemplate{}, &VSphereMachineTemplateList{})
+	objectTypes = append(objectTypes, &VSphereMachineTemplate{}, &VSphereMachineTemplateList{})
 }
diff --git a/apis/v1beta1/vspherevm_types.go b/apis/v1beta1/vspherevm_types.go
@@ -213,5 +213,5 @@ type VSphereVMList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereVM{}, &VSphereVMList{})
+	objectTypes = append(objectTypes, &VSphereVM{}, &VSphereVMList{})
 }
diff --git a/apis/vmware/v1beta1/groupversion_info.go b/apis/vmware/v1beta1/groupversion_info.go
@@ -17,8 +17,9 @@ limitations under the License.
 package v1beta1
 
 import (
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/runtime/schema"
-	"sigs.k8s.io/controller-runtime/pkg/scheme"
 )
 
 const (
@@ -33,9 +34,18 @@ var (
 	// GroupVersion is group version used to register these objects.
 	GroupVersion = schema.GroupVersion{Group: GroupName, Version: Version}
 
-	// SchemeBuilder is used to add go types to the GroupVersionKind scheme.
-	SchemeBuilder = &scheme.Builder{GroupVersion: GroupVersion}
+	// schemeBuilder is used to add go types to the GroupVersionKind scheme.
+	schemeBuilder = runtime.NewSchemeBuilder(addKnownTypes)
 
 	// AddToScheme adds the types in this group-version to the given scheme.
-	AddToScheme = SchemeBuilder.AddToScheme
+	AddToScheme = schemeBuilder.AddToScheme
+
+	// objectTypes contains all types to be registered to the GroupVersion.
+	objectTypes = []runtime.Object{}
 )
+
+func addKnownTypes(scheme *runtime.Scheme) error {
+	scheme.AddKnownTypes(GroupVersion, objectTypes...)
+	metav1.AddToGroupVersion(scheme, GroupVersion)
+	return nil
+}
diff --git a/apis/vmware/v1beta1/providerserviceaccount_types.go b/apis/vmware/v1beta1/providerserviceaccount_types.go
@@ -73,5 +73,5 @@ type ProviderServiceAccountList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&ProviderServiceAccount{}, &ProviderServiceAccountList{})
+	objectTypes = append(objectTypes, &ProviderServiceAccount{}, &ProviderServiceAccountList{})
 }
diff --git a/apis/vmware/v1beta1/vspherecluster_types.go b/apis/vmware/v1beta1/vspherecluster_types.go
@@ -87,5 +87,5 @@ func (r *VSphereCluster) SetConditions(conditions clusterv1.Conditions) {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereCluster{}, &VSphereClusterList{})
+	objectTypes = append(objectTypes, &VSphereCluster{}, &VSphereClusterList{})
 }
diff --git a/apis/vmware/v1beta1/vsphereclustertemplate_types.go b/apis/vmware/v1beta1/vsphereclustertemplate_types.go
@@ -48,7 +48,7 @@ type VSphereClusterTemplateList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereClusterTemplate{}, &VSphereClusterTemplateList{})
+	objectTypes = append(objectTypes, &VSphereClusterTemplate{}, &VSphereClusterTemplateList{})
 }
 
 type VSphereClusterTemplateResource struct {
diff --git a/apis/vmware/v1beta1/vspheremachine_types.go b/apis/vmware/v1beta1/vspheremachine_types.go
@@ -182,5 +182,5 @@ func (r *VSphereMachine) SetConditions(conditions clusterv1.Conditions) {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereMachine{}, &VSphereMachineList{})
+	objectTypes = append(objectTypes, &VSphereMachine{}, &VSphereMachineList{})
 }
diff --git a/apis/vmware/v1beta1/vspheremachinetemplate_types.go b/apis/vmware/v1beta1/vspheremachinetemplate_types.go
@@ -48,5 +48,5 @@ type VSphereMachineTemplateList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&VSphereMachineTemplate{}, &VSphereMachineTemplateList{})
+	objectTypes = append(objectTypes, &VSphereMachineTemplate{}, &VSphereMachineTemplateList{})
 }
diff --git a/apis/vmware/v1beta1/zz_generated.deepcopy.go b/apis/vmware/v1beta1/zz_generated.deepcopy.go

Original file line number	Diff line number	Diff line change
`@@ -89,5 +89,5 @@ type HAProxyLoadBalancerList struct {`
`89`	`89`	`}`
`90`	`90`
`91`	`91`	`func init() {`
`92`		`- SchemeBuilder.Register(&HAProxyLoadBalancer{}, &HAProxyLoadBalancerList{})`
	`92`	`+ objectTypes = append(objectTypes, &HAProxyLoadBalancer{}, &HAProxyLoadBalancerList{})`
`93`	`93`	`}`
Original file line number	Diff line number	Diff line change
`@@ -123,5 +123,5 @@ type VSphereClusterList struct {`
`123`	`123`	`}`
`124`	`124`
`125`	`125`	`func init() {`
`126`		`- SchemeBuilder.Register(&VSphereCluster{}, &VSphereClusterList{})`
	`126`	`+ objectTypes = append(objectTypes, &VSphereCluster{}, &VSphereClusterList{})`
`127`	`127`	`}`
Original file line number	Diff line number	Diff line change
`@@ -105,5 +105,5 @@ type VSphereClusterIdentityList struct {`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`func init() {`
`108`		`- SchemeBuilder.Register(&VSphereClusterIdentity{}, &VSphereClusterIdentityList{})`
	`108`	`+ objectTypes = append(objectTypes, &VSphereClusterIdentity{}, &VSphereClusterIdentityList{})`
`109`	`109`	`}`
Original file line number	Diff line number	Diff line change
`@@ -105,5 +105,5 @@ type VSphereDeploymentZoneList struct {`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`func init() {`
`108`		`- SchemeBuilder.Register(&VSphereDeploymentZone{}, &VSphereDeploymentZoneList{})`
	`108`	`+ objectTypes = append(objectTypes, &VSphereDeploymentZone{}, &VSphereDeploymentZoneList{})`
`109`	`109`	`}`
Original file line number	Diff line number	Diff line change
`@@ -114,5 +114,5 @@ type VSphereFailureDomainList struct {`
`114`	`114`	`}`
`115`	`115`
`116`	`116`	`func init() {`
`117`		`- SchemeBuilder.Register(&VSphereFailureDomain{}, &VSphereFailureDomainList{})`
	`117`	`+ objectTypes = append(objectTypes, &VSphereFailureDomain{}, &VSphereFailureDomainList{})`
`118`	`118`	`}`
Original file line number	Diff line number	Diff line change
`@@ -142,5 +142,5 @@ type VSphereMachineList struct {`
`142`	`142`	`}`
`143`	`143`
`144`	`144`	`func init() {`
`145`		`- SchemeBuilder.Register(&VSphereMachine{}, &VSphereMachineList{})`
	`145`	`+ objectTypes = append(objectTypes, &VSphereMachine{}, &VSphereMachineList{})`
`146`	`146`	`}`
Original file line number	Diff line number	Diff line change
`@@ -52,5 +52,5 @@ type VSphereMachineTemplateList struct {`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`func init() {`
`55`		`- SchemeBuilder.Register(&VSphereMachineTemplate{}, &VSphereMachineTemplateList{})`
	`55`	`+ objectTypes = append(objectTypes, &VSphereMachineTemplate{}, &VSphereMachineTemplateList{})`
`56`	`56`	`}`
Original file line number	Diff line number	Diff line change
`@@ -161,5 +161,5 @@ type VSphereVMList struct {`
`161`	`161`	`}`
`162`	`162`
`163`	`163`	`func init() {`
`164`		`- SchemeBuilder.Register(&VSphereVM{}, &VSphereVMList{})`
	`164`	`+ objectTypes = append(objectTypes, &VSphereVM{}, &VSphereVMList{})`
`165`	`165`	`}`