Do not set minimum go toolchain version in release branches. #8397

kaovilai · 2024-11-12T19:33:57Z

Describe the problem/challenge you have

User requests that go.mod go and toolchain remain at patch 0. ie. go 1.22.0 because they do not have available in their downstream system a new enough go to compile and build 1.22.8.

The Go toolchain refuses to load a module or workspace that declares a minimum required Go version greater than the toolchain’s own version.

Describe the solution you'd like

The ask is that golang pinning like in 7cd92a5 should be limited to outside of go.mod files.

We will still be able to fix CVEs by bumping toolchains used to newer go even with go.mod go 1.22.0

When GOTOOLCHAIN is set to local, the go command always runs the bundled Go toolchain.

If the $GOROOT/go.env file is missing or does not set a default, the go command assumes GOTOOLCHAIN=local.

Anything else you would like to add:

Being fixed in #8399

Slack: https://kubernetes.slack.com/archives/C6VCGP4MT/p1731439622318149?thread_ts=1730991728.424729&cid=C6VCGP4MT

Environment:

Velero version (use velero version):
Kubernetes version (use kubectl version):
Kubernetes installer & version:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):

Vote on this issue!

This is an invitation to the Velero community to vote on issues, you can see the project's top voted issues listed here.
Use the "reaction smiley face" up to the right of this comment to vote.

👍 for "The project would be better with this feature added"
👎 for "This feature will not enhance the project in a meaningful way"

The text was updated successfully, but these errors were encountered:

``` make container && docker run --rm velero/velero:main /velero version --client-only [+] Building 60.0s (18/18) FINISHED docker-container:colima-multiplat => [internal] load build definition from Dockerfile 0.0s => => transferring dockerfile: 2.37kB 0.0s => [internal] load metadata for docker.io/paketobuildpacks/run-jammy-tiny:0.2.52 3.6s => [internal] load metadata for docker.io/library/golang:1.22.8-bookworm 3.4s => [auth] paketobuildpacks/run-jammy-tiny:pull token for registry-1.docker.io 0.0s => [auth] library/golang:pull token for registry-1.docker.io 0.0s => [internal] load .dockerignore 0.0s => => transferring context: 60B 0.0s => CACHED [stage-2 1/3] FROM docker.io/paketobuildpacks/run-jammy-tiny:0.2.52@sha256:402b925a81a4c6985438fd37d0b22022ca688e528bbd46a38831a3702067cced 0.0s => => resolve docker.io/paketobuildpacks/run-jammy-tiny:0.2.52@sha256:402b925a81a4c6985438fd37d0b22022ca688e528bbd46a38831a3702067cced 0.0s => CACHED [restic-builder 1/3] FROM docker.io/library/golang:1.22.8-bookworm@sha256:3f0457a0a56a926d93c2baf4cf0057a645e8ff69ff31314080fcc62389643b8e 0.0s => => resolve docker.io/library/golang:1.22.8-bookworm@sha256:3f0457a0a56a926d93c2baf4cf0057a645e8ff69ff31314080fcc62389643b8e 0.0s => [internal] load build context 0.2s => => transferring context: 883.86kB 0.2s => CACHED [velero-builder 2/4] WORKDIR /go/src/github.com/vmware-tanzu/velero 0.0s => [velero-builder 3/4] COPY . /go/src/github.com/vmware-tanzu/velero 1.6s => [restic-builder 2/3] COPY . /go/src/github.com/vmware-tanzu/velero 1.6s => [velero-builder 4/4] RUN mkdir -p /output/usr/bin && export GOARM=$( echo "" | cut -c2-) && go build -o /output/velero -ldflags "-X github.com/vmware 49.7s => [restic-builder 3/3] RUN mkdir -p /output/usr/bin && export GOARM=$(echo "" | cut -c2-) && /go/src/github.com/vmware-tanzu/velero/hack/build-restic.sh && 34.6s => [stage-2 2/3] COPY --from=velero-builder /output / 0.2s => [stage-2 3/3] COPY --from=restic-builder /output / 0.1s => exporting to docker image format 4.1s => => exporting layers 2.7s => => exporting manifest sha256:5387d1a59b10970f9d0800ba960c4eb0f008e0d2bba5a9498b009ac478e5f4d4 0.0s => => exporting config sha256:f29cee5131777c8dc47571fc475f1abb2984ff1657b51c3a720c8aa453b195f2 0.0s => => sending tarball 1.4s => importing to docker 0.5s => => loading layer 4e2166c05ab9 491.52kB / 45.94MB 0.5s => => loading layer dd8fc23784d7 98.30kB / 8.82MB 0.1s 1 warning found (use --debug to expand): - UndefinedVar: Usage of undefined variable '$GOPROXY' (line 58) container: velero/velero:main WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested Client: Version: main Git commit: 303d239819259b54ecd264eef176b9dc55db3871-dirty Go version: go1.22.8 ```

``` make container && docker run --rm velero/velero:main /velero version --client-only [+] Building 60.0s (18/18) FINISHED docker-container:colima-multiplat => [internal] load build definition from Dockerfile 0.0s => => transferring dockerfile: 2.37kB 0.0s => [internal] load metadata for docker.io/paketobuildpacks/run-jammy-tiny:0.2.52 3.6s => [internal] load metadata for docker.io/library/golang:1.22.8-bookworm 3.4s => [auth] paketobuildpacks/run-jammy-tiny:pull token for registry-1.docker.io 0.0s => [auth] library/golang:pull token for registry-1.docker.io 0.0s => [internal] load .dockerignore 0.0s => => transferring context: 60B 0.0s => CACHED [stage-2 1/3] FROM docker.io/paketobuildpacks/run-jammy-tiny:0.2.52@sha256:402b925a81a4c6985438fd37d0b22022ca688e528bbd46a38831a3702067cced 0.0s => => resolve docker.io/paketobuildpacks/run-jammy-tiny:0.2.52@sha256:402b925a81a4c6985438fd37d0b22022ca688e528bbd46a38831a3702067cced 0.0s => CACHED [restic-builder 1/3] FROM docker.io/library/golang:1.22.8-bookworm@sha256:3f0457a0a56a926d93c2baf4cf0057a645e8ff69ff31314080fcc62389643b8e 0.0s => => resolve docker.io/library/golang:1.22.8-bookworm@sha256:3f0457a0a56a926d93c2baf4cf0057a645e8ff69ff31314080fcc62389643b8e 0.0s => [internal] load build context 0.2s => => transferring context: 883.86kB 0.2s => CACHED [velero-builder 2/4] WORKDIR /go/src/github.com/vmware-tanzu/velero 0.0s => [velero-builder 3/4] COPY . /go/src/github.com/vmware-tanzu/velero 1.6s => [restic-builder 2/3] COPY . /go/src/github.com/vmware-tanzu/velero 1.6s => [velero-builder 4/4] RUN mkdir -p /output/usr/bin && export GOARM=$( echo "" | cut -c2-) && go build -o /output/velero -ldflags "-X github.com/vmware 49.7s => [restic-builder 3/3] RUN mkdir -p /output/usr/bin && export GOARM=$(echo "" | cut -c2-) && /go/src/github.com/vmware-tanzu/velero/hack/build-restic.sh && 34.6s => [stage-2 2/3] COPY --from=velero-builder /output / 0.2s => [stage-2 3/3] COPY --from=restic-builder /output / 0.1s => exporting to docker image format 4.1s => => exporting layers 2.7s => => exporting manifest sha256:5387d1a59b10970f9d0800ba960c4eb0f008e0d2bba5a9498b009ac478e5f4d4 0.0s => => exporting config sha256:f29cee5131777c8dc47571fc475f1abb2984ff1657b51c3a720c8aa453b195f2 0.0s => => sending tarball 1.4s => importing to docker 0.5s => => loading layer 4e2166c05ab9 491.52kB / 45.94MB 0.5s => => loading layer dd8fc23784d7 98.30kB / 8.82MB 0.1s 1 warning found (use --debug to expand): - UndefinedVar: Usage of undefined variable '$GOPROXY' (line 58) container: velero/velero:main WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested Client: Version: main Git commit: 303d239819259b54ecd264eef176b9dc55db3871-dirty Go version: go1.22.8 ``` Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

``` make container && docker run --rm velero/velero:main /velero version --client-only [+] Building 60.0s (18/18) FINISHED docker-container:colima-multiplat => [internal] load build definition from Dockerfile 0.0s => => transferring dockerfile: 2.37kB 0.0s => [internal] load metadata for docker.io/paketobuildpacks/run-jammy-tiny:0.2.52 3.6s => [internal] load metadata for docker.io/library/golang:1.22.8-bookworm 3.4s => [auth] paketobuildpacks/run-jammy-tiny:pull token for registry-1.docker.io 0.0s => [auth] library/golang:pull token for registry-1.docker.io 0.0s => [internal] load .dockerignore 0.0s => => transferring context: 60B 0.0s => CACHED [stage-2 1/3] FROM docker.io/paketobuildpacks/run-jammy-tiny:0.2.52@sha256:402b925a81a4c6985438fd37d0b22022ca688e528bbd46a38831a3702067cced 0.0s => => resolve docker.io/paketobuildpacks/run-jammy-tiny:0.2.52@sha256:402b925a81a4c6985438fd37d0b22022ca688e528bbd46a38831a3702067cced 0.0s => CACHED [restic-builder 1/3] FROM docker.io/library/golang:1.22.8-bookworm@sha256:3f0457a0a56a926d93c2baf4cf0057a645e8ff69ff31314080fcc62389643b8e 0.0s => => resolve docker.io/library/golang:1.22.8-bookworm@sha256:3f0457a0a56a926d93c2baf4cf0057a645e8ff69ff31314080fcc62389643b8e 0.0s => [internal] load build context 0.2s => => transferring context: 883.86kB 0.2s => CACHED [velero-builder 2/4] WORKDIR /go/src/github.com/vmware-tanzu/velero 0.0s => [velero-builder 3/4] COPY . /go/src/github.com/vmware-tanzu/velero 1.6s => [restic-builder 2/3] COPY . /go/src/github.com/vmware-tanzu/velero 1.6s => [velero-builder 4/4] RUN mkdir -p /output/usr/bin && export GOARM=$( echo "" | cut -c2-) && go build -o /output/velero -ldflags "-X github.com/vmware 49.7s => [restic-builder 3/3] RUN mkdir -p /output/usr/bin && export GOARM=$(echo "" | cut -c2-) && /go/src/github.com/vmware-tanzu/velero/hack/build-restic.sh && 34.6s => [stage-2 2/3] COPY --from=velero-builder /output / 0.2s => [stage-2 3/3] COPY --from=restic-builder /output / 0.1s => exporting to docker image format 4.1s => => exporting layers 2.7s => => exporting manifest sha256:5387d1a59b10970f9d0800ba960c4eb0f008e0d2bba5a9498b009ac478e5f4d4 0.0s => => exporting config sha256:f29cee5131777c8dc47571fc475f1abb2984ff1657b51c3a720c8aa453b195f2 0.0s => => sending tarball 1.4s => importing to docker 0.5s => => loading layer 4e2166c05ab9 491.52kB / 45.94MB 0.5s => => loading layer dd8fc23784d7 98.30kB / 8.82MB 0.1s 1 warning found (use --debug to expand): - UndefinedVar: Usage of undefined variable '$GOPROXY' (line 58) container: velero/velero:main WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested Client: Version: main Git commit: 303d239819259b54ecd264eef176b9dc55db3871-dirty Go version: go1.22.8 ``` Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

blackpiglet · 2024-11-13T02:51:52Z

/assign @kaovilai

``` make container && docker run --rm velero/velero:main /velero version --client-only [+] Building 60.0s (18/18) FINISHED docker-container:colima-multiplat => [internal] load build definition from Dockerfile 0.0s => => transferring dockerfile: 2.37kB 0.0s => [internal] load metadata for docker.io/paketobuildpacks/run-jammy-tiny:0.2.52 3.6s => [internal] load metadata for docker.io/library/golang:1.22.8-bookworm 3.4s => [auth] paketobuildpacks/run-jammy-tiny:pull token for registry-1.docker.io 0.0s => [auth] library/golang:pull token for registry-1.docker.io 0.0s => [internal] load .dockerignore 0.0s => => transferring context: 60B 0.0s => CACHED [stage-2 1/3] FROM docker.io/paketobuildpacks/run-jammy-tiny:0.2.52@sha256:402b925a81a4c6985438fd37d0b22022ca688e528bbd46a38831a3702067cced 0.0s => => resolve docker.io/paketobuildpacks/run-jammy-tiny:0.2.52@sha256:402b925a81a4c6985438fd37d0b22022ca688e528bbd46a38831a3702067cced 0.0s => CACHED [restic-builder 1/3] FROM docker.io/library/golang:1.22.8-bookworm@sha256:3f0457a0a56a926d93c2baf4cf0057a645e8ff69ff31314080fcc62389643b8e 0.0s => => resolve docker.io/library/golang:1.22.8-bookworm@sha256:3f0457a0a56a926d93c2baf4cf0057a645e8ff69ff31314080fcc62389643b8e 0.0s => [internal] load build context 0.2s => => transferring context: 883.86kB 0.2s => CACHED [velero-builder 2/4] WORKDIR /go/src/github.com/vmware-tanzu/velero 0.0s => [velero-builder 3/4] COPY . /go/src/github.com/vmware-tanzu/velero 1.6s => [restic-builder 2/3] COPY . /go/src/github.com/vmware-tanzu/velero 1.6s => [velero-builder 4/4] RUN mkdir -p /output/usr/bin && export GOARM=$( echo "" | cut -c2-) && go build -o /output/velero -ldflags "-X github.com/vmware 49.7s => [restic-builder 3/3] RUN mkdir -p /output/usr/bin && export GOARM=$(echo "" | cut -c2-) && /go/src/github.com/vmware-tanzu/velero/hack/build-restic.sh && 34.6s => [stage-2 2/3] COPY --from=velero-builder /output / 0.2s => [stage-2 3/3] COPY --from=restic-builder /output / 0.1s => exporting to docker image format 4.1s => => exporting layers 2.7s => => exporting manifest sha256:5387d1a59b10970f9d0800ba960c4eb0f008e0d2bba5a9498b009ac478e5f4d4 0.0s => => exporting config sha256:f29cee5131777c8dc47571fc475f1abb2984ff1657b51c3a720c8aa453b195f2 0.0s => => sending tarball 1.4s => importing to docker 0.5s => => loading layer 4e2166c05ab9 491.52kB / 45.94MB 0.5s => => loading layer dd8fc23784d7 98.30kB / 8.82MB 0.1s 1 warning found (use --debug to expand): - UndefinedVar: Usage of undefined variable '$GOPROXY' (line 58) container: velero/velero:main WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested Client: Version: main Git commit: 303d239819259b54ecd264eef176b9dc55db3871-dirty Go version: go1.22.8 ``` Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

* update golangci-lint to v1.62.x Signed-off-by: cpanato <ctadeu@gmail.com> * update go to min go1.23.4 Signed-off-by: cpanato <ctadeu@gmail.com> --------- Signed-off-by: cpanato <ctadeu@gmail.com>

kaovilai mentioned this issue Nov 12, 2024

Automatically bump go patch version in main branch go.mod #8392

Closed

3 tasks

kaovilai added a commit to kaovilai/velero that referenced this issue Nov 12, 2024

Add go buildinfo to velero to assist with vmware-tanzu#8397

a217d53

kaovilai added a commit to kaovilai/velero that referenced this issue Nov 12, 2024

Add go buildinfo to velero to assist with vmware-tanzu#8397

68e2d43

kaovilai mentioned this issue Nov 12, 2024

release-1.15: Reduce minimum go toolchain in go.mod. #8399

Merged

3 tasks

kaovilai added a commit to kaovilai/velero that referenced this issue Nov 12, 2024

Add go buildinfo to velero to assist with vmware-tanzu#8397

277556a

kaovilai added a commit to kaovilai/velero that referenced this issue Nov 12, 2024

Add go buildinfo to velero to assist with vmware-tanzu#8397

112364f

kaovilai added a commit to kaovilai/velero that referenced this issue Nov 12, 2024

Add go buildinfo to velero to assist with vmware-tanzu#8397

b0a0cc1

Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

kaovilai added a commit to kaovilai/velero that referenced this issue Nov 12, 2024

Add go buildinfo to velero to assist with vmware-tanzu#8397

106d45f

Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

kaovilai added a commit to kaovilai/velero that referenced this issue Nov 12, 2024

Add go buildinfo to velero to assist with vmware-tanzu#8397

7a82ae4

Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

kaovilai added a commit to kaovilai/velero that referenced this issue Nov 12, 2024

Add go buildinfo to velero to assist with vmware-tanzu#8397

6cc53cf

Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

kaovilai added a commit to kaovilai/velero that referenced this issue Nov 12, 2024

Add go buildinfo to velero to assist with vmware-tanzu#8397

35f9f85

Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

blackpiglet assigned kaovilai Nov 13, 2024

kaovilai added a commit to kaovilai/velero that referenced this issue Nov 13, 2024

Add go buildinfo to velero to assist with vmware-tanzu#8397

ff8496c

Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>

kaovilai mentioned this issue Nov 14, 2024

Add go buildinfo to velero to check toolchain version used. #8398

Closed

3 tasks

kaovilai added the Enhancement/Dev Internal or Developer-focused Enhancement to Velero label Nov 19, 2024

kaovilai mentioned this issue Dec 3, 2024

deps: Remove Go patch version from go.mods kopia/kopia#4292

Open

kaovilai mentioned this issue Dec 15, 2024

govulncheck-action: Support using a later patch than go directive patch version in go.mod golang/go#70853

Open

kaovilai mentioned this issue Dec 24, 2024

Stop bumping go directive unless necessitated by other dependencies containers/image#2664

Open

kaovilai mentioned this issue Dec 24, 2024

Stop bumping go directive unless necessitated by other dependencies chainguard-dev/apko#1454

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Do not set minimum go toolchain version in release branches. #8397

Do not set minimum go toolchain version in release branches. #8397

kaovilai commented Nov 12, 2024 •

edited

Loading

blackpiglet commented Nov 13, 2024

Do not set minimum go toolchain version in release branches. #8397

Do not set minimum go toolchain version in release branches. #8397

Comments

kaovilai commented Nov 12, 2024 • edited Loading

blackpiglet commented Nov 13, 2024

kaovilai commented Nov 12, 2024 •

edited

Loading