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

misc-vet-vetall build failure #31210

Closed
aarzilli opened this issue Apr 2, 2019 · 7 comments
Closed

misc-vet-vetall build failure #31210

aarzilli opened this issue Apr 2, 2019 · 7 comments
Labels
Builders x/build issues (builders, bots, dashboards) FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.

Comments

@aarzilli
Copy link
Contributor

aarzilli commented Apr 2, 2019

Just got a weird build failure from misc-vet-vetall builder:

##### cmd/vet/all
can't load package: package golang.org/x/tools/go/analysis/cmd/vet: cannot find package "golang.org/x/tools/go/analysis/cmd/vet" in any of:
	/workdir/go/src/golang.org/x/tools/go/analysis/cmd/vet (from $GOROOT)
	/workdir/gopath/src/golang.org/x/tools/go/analysis/cmd/vet (from $GOPATH)
vet/all: exit status 1
exit status 1
2019/04/02 16:37:12 Failed: exit status 1
2019/04/02 16:37:12 FAILED

Full log: https://storage.googleapis.com/go-build-log/c98fc4a0/misc-vet-vetall_7099ef79.log

@dmitshur dmitshur added 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 Apr 2, 2019
@dmitshur
Copy link
Contributor

dmitshur commented Apr 2, 2019

The log is from CL 163677.

/cc @bradfitz Have we done anything recently (today-ish) that would affect the misc-vet-vetall builder?

@bradfitz
Copy link
Contributor

bradfitz commented Apr 2, 2019

Nope.

And my earlier test CL https://go-review.googlesource.com/c/go/+/170399 passed on the misc-vet-vetall builder.

The coordinator's been up 17 hours and 43 minutes (https://farmer.golang.org/), and that test CL is more recent than that.

@aarzilli
Copy link
Contributor Author

aarzilli commented Apr 2, 2019

Maybe it's because the parent commit was old? Do they build the exact commit or do they cherry pick?

@dmitshur
Copy link
Contributor

dmitshur commented Apr 2, 2019

They build the exact commit. However, your CL's parent commit is not very old at all.

This builder is passing for the latest master commit at https://build.golang.org/.

@dmitshur
Copy link
Contributor

dmitshur commented Apr 2, 2019

Looks like it was some sort of flakiness. Retrying the trybots at https://go-review.googlesource.com/c/go/+/163677 made the failing builder pass.

@bradfitz
Copy link
Contributor

bradfitz commented Apr 2, 2019

No, the git parent was old.

Look at the diff from patchset 2 (which failed) to 3 (which passed) and its git parent change: https://go-review.googlesource.com/c/go/+/163677/2..3//COMMIT_MSG#b1

That's nearly 600 commits behind:

$ git log --format=oneline 73b803ee..4ebc6514 | wc -l
599

Dup of #19664, #9858, #30807

@bradfitz bradfitz closed this as completed Apr 2, 2019
@dmitshur
Copy link
Contributor

dmitshur commented Apr 2, 2019

No, the git parent was old.

Ah, indeed. I made a mistake and looked at the wrong patch set. I'm glad it's a simple explanation.

@golang golang locked and limited conversation to collaborators Apr 1, 2020
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 NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Development

No branches or pull requests

4 participants