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.

Sign up for GitHub

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

Bazel CI: //tests/integration/reproducibility:reproducibility_test broken with Bazel@HEAD #3024

Closed
meteorcloudy opened this issue Dec 6, 2021 · 4 comments · Fixed by #3026
Closed

Bazel CI: //tests/integration/reproducibility:reproducibility_test broken with Bazel@HEAD #3024

meteorcloudy opened this issue Dec 6, 2021 · 4 comments · Fixed by #3026

Comments

@meteorcloudy
Copy link
Contributor

https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/2268#cf713d45-245a-437a-bb0f-85a10498d523

exec ${PAGER:-/usr/bin/less} "$0" || exit 1
Executing tests from //tests/integration/reproducibility:reproducibility_test
-----------------------------------------------------------------------------
--- FAIL: Test (35.40s)
    reproducibility_test.go:226: open /private/var/tmp/_bazel_buildkite/151f56892d8b39575ff82475bb0e1262/bazel_testing/bazel_go_test/main0/bazel-out/host/bin/external/go_sdk/builder: no such file or directory
FAIL

A bisect points to bazelbuild/bazel@4b47d6f, which looks like an intentional change.
@meteorcloudy
Copy link
Contributor Author

FYI @katre

@meteorcloudy
Copy link
Contributor Author

meteorcloudy commented Dec 6, 2021 •
edited
Loading

@robfig Can you also take a look at this?
To reproduce

export USE_BAZEL_VERSION=4b47d6f849bb2eca669f48cfdb5d1796100920f3
bazelisk test //tests/integration/reproducibility:reproducibility_test

Latest version of Bazelisk is available at https://github.com/bazelbuild/bazelisk/releases

@katre
Copy link
Contributor

katre commented Dec 6, 2021

See #2967, this is known and needs to be fixed in rules_go's tests.

@meteorcloudy
Copy link
Contributor Author

@katre Thanks!

@robfig @jayconrod If this isn't going to be fixed soon, can you disable the test to make downstream green again?

robfig added a commit that referenced this issue Dec 8, 2021
@robfig
reproducibility_test: explicitly build the go_sdk builder
5e32fb9 
This allows us to find it at a known path.

It is no longer possible to access genfiles at a known path:
bazelbuild/bazel#14049

Fixes #2967
Fixes #3024
@robfig robfig mentioned this issue Dec 8, 2021
Merged
robfig added a commit that referenced this issue Dec 8, 2021
@robfig
reproducibility_test: explicitly build the go_sdk builder (#3026)
b851b94 
This allows us to find it at a known path.

It is no longer possible to access genfiles at a known path:
bazelbuild/bazel#14049

Fixes #2967
Fixes #3024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

reproducibility_test: explicitly build the go_sdk builder bazel-contrib/rules_go
2 participants
@katre @meteorcloudy