From 7770c3d0e17d05f67840327c1549aa74dbf2eb15 Mon Sep 17 00:00:00 2001 From: Yuan Tang Date: Wed, 3 May 2023 23:49:53 -0400 Subject: [PATCH] fix: Upgrade Go to v1.20. Fixes #11023 (#11027) Signed-off-by: Yuan Tang Signed-off-by: Dillen Padhiar --- .github/workflows/ci-build.yaml | 8 ++++---- .github/workflows/gh-pages.yaml | 2 +- .github/workflows/release.yaml | 2 +- .golangci.yml | 3 ++- Dockerfile | 2 +- Dockerfile.windows | 2 +- Makefile | 2 +- go.mod | 2 +- pkg/apiclient/http1/facade.go | 2 +- 9 files changed, 13 insertions(+), 12 deletions(-) diff --git a/.github/workflows/ci-build.yaml b/.github/workflows/ci-build.yaml index ec04ef99dd02..f6bc01abe61a 100644 --- a/.github/workflows/ci-build.yaml +++ b/.github/workflows/ci-build.yaml @@ -25,7 +25,7 @@ jobs: - uses: actions/checkout@v3 - uses: actions/setup-go@v4 with: - go-version: "1.19" + go-version: "1.20" cache: true - run: make test STATIC_FILES=false GOTEST='go test -p 20 -covermode=atomic -coverprofile=coverage.out' # engineers just ignore this in PRs, so lets not even run it @@ -102,7 +102,7 @@ jobs: - uses: actions/checkout@v3 - uses: actions/setup-go@v4 with: - go-version: "1.19" + go-version: "1.20" cache: true - uses: actions/setup-java@v3 if: ${{matrix.test == 'test-java-sdk'}} @@ -202,7 +202,7 @@ jobs: - uses: actions/checkout@v3 - uses: actions/setup-go@v4 with: - go-version: "1.19" + go-version: "1.20" cache: true - name: Install protoc run: | @@ -235,7 +235,7 @@ jobs: - uses: actions/checkout@v3 - uses: actions/setup-go@v4 with: - go-version: "1.19" + go-version: "1.20" cache: true - run: make lint STATIC_FILES=false - run: git diff --exit-code diff --git a/.github/workflows/gh-pages.yaml b/.github/workflows/gh-pages.yaml index 30d51c85d022..0ccc55d04d04 100644 --- a/.github/workflows/gh-pages.yaml +++ b/.github/workflows/gh-pages.yaml @@ -25,7 +25,7 @@ jobs: python-version: 3.9 - uses: actions/setup-go@v4 with: - go-version: '1.19' + go-version: '1.20' - uses: actions/setup-node@v3 with: node-version: "19" diff --git a/.github/workflows/release.yaml b/.github/workflows/release.yaml index b0a54b0d60ee..1eea359403cb 100644 --- a/.github/workflows/release.yaml +++ b/.github/workflows/release.yaml @@ -385,7 +385,7 @@ jobs: node-version: "16" - uses: actions/setup-go@v4 with: - go-version: "1.19" + go-version: "1.20" - uses: actions/cache@v3 with: path: ui/node_modules diff --git a/.golangci.yml b/.golangci.yml index f0b13cf5ee53..e9de691ae418 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -61,7 +61,8 @@ issues: exclude: # We are leaving io/ioutil for now to make backports easier # https://github.com/argoproj/argo-workflows/pull/6964#issuecomment-946827019 - - "SA1019: \"io/ioutil\" has been deprecated since Go 1.16" + - "SA1019: \"io/ioutil\" has been deprecated since Go 1.19" + - "SA1019: rand.Seed has been deprecated since Go 1.20" exclude-rules: - path: server/artifacts/artifact_server_test.go text: "response body must be closed" diff --git a/Dockerfile b/Dockerfile index d5b141d67390..2c920468efab 100644 --- a/Dockerfile +++ b/Dockerfile @@ -3,7 +3,7 @@ ARG GIT_COMMIT=unknown ARG GIT_TAG=unknown ARG GIT_TREE_STATE=unknown -FROM golang:1.19-alpine3.16 as builder +FROM golang:1.20-alpine3.16 as builder RUN apk update && apk add --no-cache \ git \ diff --git a/Dockerfile.windows b/Dockerfile.windows index b80d41fc9627..49151c1ed482 100644 --- a/Dockerfile.windows +++ b/Dockerfile.windows @@ -11,7 +11,7 @@ ARG GIT_TREE_STATE=unknown # had issues with official golange image for windows so I'm using plain servercore FROM mcr.microsoft.com/windows/servercore:${IMAGE_OS_VERSION} as builder -ENV GOLANG_VERSION=1.19 +ENV GOLANG_VERSION=1.20 SHELL ["powershell", "-Command"] # install chocolatey package manager diff --git a/Makefile b/Makefile index 756988be1e28..a64f81af3713 100644 --- a/Makefile +++ b/Makefile @@ -401,7 +401,7 @@ dist/manifests/%: manifests/% # lint/test/etc $(GOPATH)/bin/golangci-lint: - curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b `go env GOPATH`/bin v1.49.0 + curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b `go env GOPATH`/bin v1.52.2 .PHONY: lint lint: server/static/files.go $(GOPATH)/bin/golangci-lint diff --git a/go.mod b/go.mod index ff38ef19431b..88a7b6b6fe08 100644 --- a/go.mod +++ b/go.mod @@ -1,6 +1,6 @@ module github.com/argoproj/argo-workflows/v3 -go 1.19 +go 1.20 require ( cloud.google.com/go/storage v1.30.1 diff --git a/pkg/apiclient/http1/facade.go b/pkg/apiclient/http1/facade.go index 597b1f315ebd..17ecde268331 100644 --- a/pkg/apiclient/http1/facade.go +++ b/pkg/apiclient/http1/facade.go @@ -72,7 +72,7 @@ func (h Facade) EventStreamReader(in interface{}, path string) (*bufio.Reader, e }, }, } - resp, err := client.Do(req) + resp, err := client.Do(req) //nolint if err != nil { return nil, err }