Skip to content

Commit 0107350

Browse files
authored
Merge pull request #468 from estroz/feature/default-crd-v1
⚠️ pkg/crd: default CRD version to v1
2 parents 4a903dd + 41209ab commit 0107350

File tree

3 files changed

+7
-4
lines changed

3 files changed

+7
-4
lines changed

pkg/crd/gen.go

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,9 @@ import (
3131
"sigs.k8s.io/controller-tools/pkg/version"
3232
)
3333

34+
// The default CustomResourceDefinition version to generate.
35+
const defaultVersion = "v1"
36+
3437
// +controllertools:marker:generateHelp
3538

3639
// Generator generates CustomResourceDefinition objects.
@@ -71,7 +74,7 @@ type Generator struct {
7174
MaxDescLen *int `marker:",optional"`
7275

7376
// CRDVersions specifies the target API versions of the CRD type itself to
74-
// generate. Defaults to v1beta1.
77+
// generate. Defaults to v1.
7578
//
7679
// The first version listed will be assumed to be the "default" version and
7780
// will not get a version suffix in the output filename.
@@ -113,7 +116,7 @@ func (g Generator) Generate(ctx *genall.GenerationContext) error {
113116
crdVersions := g.CRDVersions
114117

115118
if len(crdVersions) == 0 {
116-
crdVersions = []string{"v1beta1"}
119+
crdVersions = []string{defaultVersion}
117120
}
118121

119122
for groupKind := range kubeKinds {

pkg/crd/testdata/cronjob_types.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ limitations under the License.
1616
// TODO(directxman12): test this across both versions (right now we're just
1717
// trusting k/k conversion, which is probably fine though)
1818

19-
//go:generate ../../../.run-controller-gen.sh crd:crdVersions=v1 paths=. output:dir=.
19+
//go:generate ../../../.run-controller-gen.sh paths=. output:dir=.
2020

2121
// +groupName=testdata.kubebuilder.io
2222
// +versionName=v1

pkg/crd/zz_generated.markerhelp.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.

0 commit comments

Comments
 (0)