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: reconsider NaCL builders on subrepos #28582

Closed
dmitshur opened this issue Nov 3, 2018 · 3 comments
Closed

x/build: reconsider NaCL builders on subrepos #28582

dmitshur opened this issue Nov 3, 2018 · 3 comments
Labels
Builders x/build issues (builders, bots, dashboards) FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@dmitshur
Copy link
Contributor

dmitshur commented Nov 3, 2018

I'm not seeing an issue about this, so filing one to track this.

NaCL builders got added to subrepo trybots as part of changes in CL 145157, which got deployed a few days ago. Brad noticed it earlier and told me:

also I need to remove nacl trybots from x/net etc
that was a mistake I think

They seem to be failing on many x/tools trybot runs, e.g., https://go-review.googlesource.com/c/tools/+/147317#message-0b65ea2ab17f9e28f0519d84b8bb646345441b6a, making the real failures harder to see.

Adding NeedsDecision because I'm not 100% confident what's the best resolution here, and looking for confirmation.

/cc @bradfitz @heschik

@dmitshur dmitshur added Builders x/build issues (builders, bots, dashboards) NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. labels Nov 3, 2018
@gopherbot gopherbot added this to the Unreleased milestone Nov 3, 2018
@bradfitz
Copy link
Contributor

bradfitz commented Nov 4, 2018

Decision: don't run nacl for subrepos. There's no mechanism to package their testdata files up anyway.

@bradfitz bradfitz added the NeedsFix The path to resolution is known, but the work has not been done. label Nov 4, 2018
@gopherbot gopherbot removed the NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. label Nov 4, 2018
@gopherbot
Copy link
Contributor

Change https://golang.org/cl/147337 mentions this issue: dashboard: revert accidental inclusion of nacl subrepo trybots

@bradfitz
Copy link
Contributor

bradfitz commented Nov 4, 2018

Deployed.

Farmer was:

21h27m35s. Version 906dcfd58886df4b3156fd9fa9d51f9767bd66f6.

Now:

Uptime 1s. Version 9f6998b87f0ce9198930a0a7eb5e5b86ae8b7802

@golang golang locked and limited conversation to collaborators Nov 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Builders x/build issues (builders, bots, dashboards) FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Development

No branches or pull requests

3 participants