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

Jump to bottom

x/build/cmd/coordinator: subrepo trybots testing against security branch #29510

Closed
bradfitz opened this issue Jan 2, 2019 · 3 comments
Closed

x/build/cmd/coordinator: subrepo trybots testing against security branch #29510

bradfitz opened this issue Jan 2, 2019 · 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
Unreleased

Comments

@bradfitz
Copy link
Contributor

bradfitz commented Jan 2, 2019

Seen on a trybot run:

https://go-review.googlesource.com/c/tools/+/156078

Build is still in progress...
This change failed on linux-amd64 (Go 1.10-security.x):
See https://storage.googleapis.com/go-build-log/25ca8f49/linux-amd64_578e3eba.log

I suspect there's a bug in the fixes for #17626, or old assumptions that no longer apply now that we have a security branch.

Maybe just filter out the security branch before we find the last N=2 releases?

Or delete the security branch altogether?
@bradfitz bradfitz added the NeedsFix The path to resolution is known, but the work has not been done. label Jan 2, 2019
@gopherbot gopherbot added this to the Unreleased milestone Jan 2, 2019
@gopherbot gopherbot added the Builders x/build issues (builders, bots, dashboards) label Jan 2, 2019
@dmitshur
Copy link
Contributor

dmitshur commented Jan 2, 2019 •
edited

Or delete the security branch altogether?

We did delete them in the canonical repository (i.e., https://go.googlesource.com/go/+refs). However, it continues to show up in the GitHub mirror repository (even after a manual deletion) due to a bug in gitmirror, see #23099 (comment).

I suspect we'll want to fix this with a quick trybot-specific workaround first, because the underlying issues in gitmirror (and/or in maintner) may take significantly longer to investigate and fix (and can be done independently afterwards).

@dmitshur dmitshur mentioned this issue Jan 2, 2019
Closed
@dmitshur
Copy link
Contributor

dmitshur commented Jan 4, 2019

This is worse than I originally understood, because trybots are not just testing the security release-branches in addition to normal release-branches, but rather only the security release-branches.

It's not too bad since the security release branches are pretty representative of Go 1.11 and Go 1.10, which is the goal of #17626, but we should still fix it so that the latest release-branches get tested. I'll start looking into this soon.

@gopherbot
Copy link

Change https://golang.org/cl/157443 mentions this issue: maintner/maintnerd/maintapi: ignore unexpected release branches

@bradfitz bradfitz assigned bradfitz and unassigned dmitshur Jan 11, 2019
@golang golang locked and limited conversation to collaborators Jan 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
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
Milestone
Unreleased
Development

No branches or pull requests
3 participants
@bradfitz @dmitshur @gopherbot