Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

x/build/cmd/gomote: linux-loong64 gomote creation fails with "HTTP Error 403: Forbidden" #72869

Closed
dmitshur opened this issue Mar 14, 2025 · 3 comments
Labels
arch-loong64 Issues solely affecting the loongson architecture. Builders x/build issues (builders, bots, dashboards) FixPending Issues that have a fix which has not yet been reviewed or submitted. NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@dmitshur
Copy link
Contributor

Create a gotip-linux-loong64 gomote results in an error. Filing this as a tracking issue.

A swarming task is successfully created, but the task fails. https://chromium-swarm.appspot.com/task?id=6fadfc2f6a116710 is an example of a failed swarming task. It fails with "HTTP Error 403: Forbidden".

The task attempts to fetch the latest version of the buildlet.linux-loong64 binary from a GCS bucket (https://storage.googleapis.com/go-builder-data) using the script visible here:

https://cs.opensource.google/go/x/build/+/master:internal/gomote/swarming.go;l=1024-1053;drc=3ff389173c43ad07048563029b575ab7ec0df5da

But it appears that requests results in the 403 response, even though that file is set to be publicly readable:

$ curl -O 'https://storage.googleapis.com/go-builder-data/buildlet.linux-loong64'
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   298  100   298    0     0   2284      0 --:--:-- --:--:-- --:--:--  2292
$ echo $?
0

CC @golang/loong64, @golang/release.

@dmitshur dmitshur added arch-loong64 Issues solely affecting the loongson architecture. Builders x/build issues (builders, bots, dashboards) NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Mar 14, 2025
@dmitshur dmitshur modified the milestones: Backlog, Unreleased Mar 14, 2025
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/657975 mentions this issue: DashboardBuilders: document required connectivity to Google Cloud Storage

gopherbot pushed a commit to golang/wiki that referenced this issue Mar 14, 2025
…rage

Gomote creation cannot work without, so include it in the list.

For golang/go#72869.

Change-Id: Id54f3aa32259c3b298b26eb51c881d90dc759ae9
Reviewed-on: https://go-review.googlesource.com/c/wiki/+/657975
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/657995 mentions this issue: cmd/buildlet: add linux-loong64 targets

@dmitshur
Copy link
Contributor Author

dmitshur commented Mar 14, 2025

Whoops, that file wasn't present in the GCS bucket. (Thanks @cagedmantis for spotting the issue!) I've sent CL 657995 that adds it as a target and uploaded it now.

Gomote creation and SSHing into the gomote works successfully in my testing after the above.


$ gomote create gotip-linux-loong64                       
# still creating gotip-linux-loong64 (1) after 6s; 0 requests ahead of you
# still creating gotip-linux-loong64 (1) after 11s; 0 requests ahead of you
[...]
# still creating gotip-linux-loong64 (1) after 36s; 0 requests ahead of you
# still creating gotip-linux-loong64 (1) after 41s; 0 requests ahead of you
$USER-gotip-linux-loong64-0
$ gomote ssh $USER-gotip-linux-loong64-0
# Welcome to the gomote ssh proxy.
[...]
$ go version
go version go1.24.1 linux/loong64
$ which unzip
/usr/bin/unzip
$ ^D
exit
Connection to localhost closed.
Connection to gomotessh.golang.org closed.
$ gomote destroy $USER-gotip-linux-loong64-0
# Destroying $USER-gotip-linux-loong64-0

@dmitshur dmitshur added NeedsFix The path to resolution is known, but the work has not been done. FixPending Issues that have a fix which has not yet been reviewed or submitted. and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Mar 14, 2025
@dmitshur dmitshur moved this to In Progress in Go Release Mar 14, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Go Release Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arch-loong64 Issues solely affecting the loongson architecture. Builders x/build issues (builders, bots, dashboards) FixPending Issues that have a fix which has not yet been reviewed or submitted. NeedsFix The path to resolution is known, but the work has not been done.
Projects
Status: Done
Development

No branches or pull requests

2 participants