Skip to content

Commit 4814605

Browse files
authored
Merge pull request #24 from alphagov/BAU-Deprecate-scheme-builder
BAU - Deprecate scheme builder
2 parents 48ef7c7 + f9e62bc commit 4814605

6 files changed

Lines changed: 21 additions & 11 deletions

File tree

api/v1/groupversion_info.go

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,9 @@ limitations under the License.
2020
package v1
2121

2222
import (
23+
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
24+
"k8s.io/apimachinery/pkg/runtime"
2325
"k8s.io/apimachinery/pkg/runtime/schema"
24-
"sigs.k8s.io/controller-runtime/pkg/scheme"
2526
)
2627

2728
var (
@@ -33,8 +34,21 @@ var (
3334
GroupVersion = SchemeGroupVersion
3435

3536
// SchemeBuilder is used to add go types to the GroupVersionKind scheme.
36-
SchemeBuilder = &scheme.Builder{GroupVersion: SchemeGroupVersion}
37+
SchemeBuilder = runtime.NewSchemeBuilder(addKnownTypes)
3738

3839
// AddToScheme adds the types in this group-version to the given scheme.
3940
AddToScheme = SchemeBuilder.AddToScheme
4041
)
42+
43+
func addKnownTypes(scheme *runtime.Scheme) error {
44+
// Add your API types here
45+
scheme.AddKnownTypes(GroupVersion,
46+
&JobRequest{},
47+
&JobRequestList{},
48+
&JobRequestReview{},
49+
&JobRequestReviewList{},
50+
)
51+
52+
metav1.AddToGroupVersion(scheme, GroupVersion)
53+
return nil
54+
}

api/v1/jobrequest_types.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,3 @@ type JobRequestList struct {
8787
metav1.ListMeta `json:"metadata,omitzero"`
8888
Items []JobRequest `json:"items"`
8989
}
90-
91-
func init() {
92-
SchemeBuilder.Register(&JobRequest{}, &JobRequestList{})
93-
}

api/v1/jobrequestreview_types.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,3 @@ type JobRequestReviewList struct {
6666
metav1.ListMeta `json:"metadata,omitzero"`
6767
Items []JobRequestReview `json:"items"`
6868
}
69-
70-
func init() {
71-
SchemeBuilder.Register(&JobRequestReview{}, &JobRequestReviewList{})
72-
}

api/v1/zz_generated.deepcopy.go

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/kustomization.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
# It should be run by config/default
44
resources:
55
- bases/platform.publishing.service.gov.uk_jobrequests.yaml
6+
- bases/platform.publishing.service.gov.uk_jobrequestreviews.yaml
67
# +kubebuilder:scaffold:crdkustomizeresource
78

89
patches:

config/rbac/role.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ rules:
77
- apiGroups:
88
- platform.publishing.service.gov.uk
99
resources:
10+
- jobrequestreviews
1011
- jobrequests
1112
verbs:
1213
- create
@@ -19,12 +20,14 @@ rules:
1920
- apiGroups:
2021
- platform.publishing.service.gov.uk
2122
resources:
23+
- jobrequestreviews/finalizers
2224
- jobrequests/finalizers
2325
verbs:
2426
- update
2527
- apiGroups:
2628
- platform.publishing.service.gov.uk
2729
resources:
30+
- jobrequestreviews/status
2831
- jobrequests/status
2932
verbs:
3033
- get

0 commit comments

Comments
 (0)