Skip to content

Commit

Permalink
Bump golang version to 1.23
Browse files Browse the repository at this point in the history
  • Loading branch information
vsethi09 committed Dec 16, 2024
1 parent a62c0ad commit 903ec53
Show file tree
Hide file tree
Showing 11 changed files with 433 additions and 13 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/go.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ jobs:
- name: Set up Go
uses: actions/setup-go@v3
with:
go-version: 1.21
go-version: 1.23

- name: Install Kubebuilder
run: |
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# Build the manager binary
FROM golang:1.19 as builder
FROM golang:1.23 as builder

# Copy everything in the go src
WORKDIR /go/src/cdap.io/cdap-operator
Expand Down
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
# Image URL to use all building/pushing image targets
IMG ?= controller:latest
# ENVTEST_K8S_VERSION refers to the version of kubebuilder assets to be downloaded by envtest binary.
ENVTEST_K8S_VERSION = 1.22
ENVTEST_K8S_VERSION = 1.30

# Get the currently used golang install path (in GOPATH/bin, unless GOBIN is set)
ifeq (,$(shell go env GOBIN))
Expand Down Expand Up @@ -113,7 +113,7 @@ ENVTEST ?= $(LOCALBIN)/setup-envtest

## Tool Versions
KUSTOMIZE_VERSION ?= v3.8.7
CONTROLLER_TOOLS_VERSION ?= v0.7.0
CONTROLLER_TOOLS_VERSION ?= v0.16.5
KUSTOMIZE_INSTALL_SCRIPT ?= "https://raw.githubusercontent.com/kubernetes-sigs/kustomize/master/hack/install_kustomize.sh"

.PHONY: kustomize
Expand Down
5 changes: 0 additions & 5 deletions controllers/testdata/appfabric.json
Original file line number Diff line number Diff line change
Expand Up @@ -151,11 +151,6 @@
"name": "cdap-sysappconf",
"readOnly": true
},
{
"mountPath": "/data",
"name": "cdap-test-appfabric-data",
"readOnly": true
},
{
"mountPath": "/etc/cdap/security",
"name": "cdap-security",
Expand Down
Loading

0 comments on commit 903ec53

Please sign in to comment.