Skip to content

Use go.dev/dl for downloading Go#23

Merged
kcp-ci-bot merged 1 commit intokcp-dev:masterfrom
xmudrii:fix-go-dl
Dec 2, 2025
Merged

Use go.dev/dl for downloading Go#23
kcp-ci-bot merged 1 commit intokcp-dev:masterfrom
xmudrii:fix-go-dl

Conversation

@xmudrii
Copy link
Member

@xmudrii xmudrii commented Dec 2, 2025

The original URL is throwing HTTP error 401:

I1202 13:25:00.921222       1 install.go:92] Installing go 1.24.9 to /go-workspace/go-1.24.9
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0   298    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 403

gzip: stdin: unexpected end of file
tar: Child returned status 1
tar: Error is not recoverable: exiting now
E1202 13:25:01.100987       1 publisher.go:460] command "/bin/bash -c curl -SLf https://storage.googleapis.com/golang/go1.24.9.linux-amd64.tar.gz | tar -xz --strip 1 -C /go-workspace/go-tmp-265313062" failed: exit status 2
I1202 13:25:01.110173       1 main.go:185] Failed to run publisher: command "/bin/bash -c curl -SLf https://storage.googleapis.com/golang/go1.24.9.linux-amd64.tar.gz | tar -xz --strip 1 -C /go-workspace/go-tmp-265313062" failed: exit status 2

I don't know if Google changed anything, but go.dev/dl is the official URL on their website. The original URL is coming from the upstream publishing-bot, I don't know where did they find that URL. It could be that it has been used in the past, but that Google has stopped updating the bucket.

/assign @xrstf

On-behalf-of: @SAP marko.mudrinic@sap.com
Signed-off-by: Marko Mudrinić <mudrinic.mare@gmail.com>
@kcp-ci-bot kcp-ci-bot added dco-signoff: yes Indicates the PR's author has signed the DCO. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Dec 2, 2025
@xmudrii xmudrii changed the title Use go.dev/dl for downloading Go Use go.dev/dl for downloading Go Dec 2, 2025
Copy link

@xrstf xrstf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label Dec 2, 2025
@kcp-ci-bot
Copy link

LGTM label has been added.

DetailsGit tree hash: 8d832b6fb1c1bf69a79af5ec1a75c3a93d098e67

@kcp-ci-bot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: xrstf

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kcp-ci-bot kcp-ci-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 2, 2025
@kcp-ci-bot kcp-ci-bot merged commit 2deefa6 into kcp-dev:master Dec 2, 2025
6 checks passed
@xmudrii xmudrii deleted the fix-go-dl branch December 2, 2025 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has signed the DCO. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants