Skip to content

Commit

Permalink
Merge pull request #400 from Nordix/add/new-scheme-builder
Browse files Browse the repository at this point in the history
🌱 Use k8s apimachinery scheme builder instead of controller-runtime scheme builder
  • Loading branch information
metal3-io-bot authored Dec 21, 2023
2 parents 497b7fc + a247f0d commit b2f74de
Show file tree
Hide file tree
Showing 5 changed files with 16 additions and 10 deletions.
18 changes: 12 additions & 6 deletions api/v1alpha1/groupversion_info.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,24 +23,30 @@ limitations under the License.
package v1alpha1

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"
)

var (
// GroupVersion is group version used to register these objects.
GroupVersion = schema.GroupVersion{Group: "ipam.metal3.io", Version: "v1alpha1"}

// 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

// localSchemeBuilder is for automatically generated conversions.
// localSchemeBuilder = SchemeBuilder.SchemeBuilder.
objectTypes = []runtime.Object{}
)

func addKnownTypes(scheme *runtime.Scheme) error {
scheme.AddKnownTypes(GroupVersion, objectTypes...)
metav1.AddToGroupVersion(scheme, GroupVersion)
return nil
}

// Resource is required by pkg/client/listers/...
// func Resource(resource string) schema.GroupResource {
// return SchemeGroupVersion.WithResource(resource).GroupResource()
Expand Down
2 changes: 1 addition & 1 deletion api/v1alpha1/ipaddress_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -73,5 +73,5 @@ type IPAddressList struct {
}

func init() {
SchemeBuilder.Register(&IPAddress{}, &IPAddressList{})
objectTypes = append(objectTypes, &IPAddress{}, &IPAddressList{})
}
2 changes: 1 addition & 1 deletion api/v1alpha1/ipclaim_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -69,5 +69,5 @@ type IPClaimList struct {
}

func init() {
SchemeBuilder.Register(&IPClaim{}, &IPClaimList{})
objectTypes = append(objectTypes, &IPClaim{}, &IPClaimList{})
}
2 changes: 1 addition & 1 deletion api/v1alpha1/ippool_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -116,5 +116,5 @@ type IPPoolList struct {
}

func init() {
SchemeBuilder.Register(&IPPool{}, &IPPoolList{})
objectTypes = append(objectTypes, &IPPool{}, &IPPoolList{})
}
2 changes: 1 addition & 1 deletion api/v1alpha1/v1alpha4_suite_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ func TestMain(m *testing.M) {
CRDDirectoryPaths: []string{filepath.Join("..", "..", "config", "crd", "bases")},
}

err := SchemeBuilder.AddToScheme(scheme.Scheme)
err := schemeBuilder.AddToScheme(scheme.Scheme)
if err != nil {
log.Fatal(err)
}
Expand Down

0 comments on commit b2f74de

Please sign in to comment.