Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
build: add support for go 1.22
Browse files Browse the repository at this point in the history
Few day's back go 1.22 is release. Let's add that
as valid version to build on. Also, adding 1.22 as
test matric for linux build CI.

https://tip.golang.org/doc/go1.22

Signed-off-by: subhamkrai <srai@redhat.com>
subhamkrai authored and Nikhil-Ladha committed Feb 9, 2024
1 parent 5163d9d commit 1b46681
Showing 5 changed files with 3,501 additions and 755 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
@@ -77,7 +77,7 @@ jobs:
strategy:
fail-fast: false
matrix:
go-version: ["1.21"]
go-version: ["1.21","1.22"]
steps:
- name: checkout
uses: actions/checkout@v4
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
@@ -32,7 +32,7 @@ all: build

# Controller-gen version
# f284e2e8... is master ahead of v0.5.0 which has ability to generate embedded objectmeta in CRDs
CONTROLLER_GEN_VERSION=v0.11.3
CONTROLLER_GEN_VERSION=v0.14.0

# Set GOBIN
ifeq (,$(shell go env GOBIN))
2 changes: 1 addition & 1 deletion build/makelib/golang.mk
Original file line number Diff line number Diff line change
@@ -48,7 +48,7 @@ GO_TEST_FLAGS ?=
# ====================================================================================
# Setup go environment

GO_SUPPORTED_VERSIONS ?= 1.21
GO_SUPPORTED_VERSIONS ?= 1.21|1.22

GO_PACKAGES := $(foreach t,$(GO_SUBDIRS),$(GO_PROJECT)/$(t)/...)
GO_INTEGRATION_TEST_PACKAGES := $(foreach t,$(GO_INTEGRATION_TESTS_SUBDIRS),$(GO_PROJECT)/$(t)/integration)
2,125 changes: 1,749 additions & 376 deletions deploy/charts/rook-ceph/templates/resources.yaml

Large diffs are not rendered by default.

2,125 changes: 1,749 additions & 376 deletions deploy/examples/crds.yaml

Large diffs are not rendered by default.

0 comments on commit 1b46681

Please sign in to comment.