-
Notifications
You must be signed in to change notification settings - Fork 613
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
Fix GPU CI Test - Remove bazelversion file in docker image #2318
Conversation
Maybe try to bump our bazel version to 3.7.2. TF bumped bazel version to 3.7.2 four days ago. I think they also updated their CI enviornment accordingly. |
Hmmm no luck. Checking the container we build with 3.1.0 is the default bazel installation anyway:
And switching the version in
Will look into this when time allows later today |
Is our image still derived from the published custom_ops image? |
Cause we find bazel 3.7.2 only on nightly custom ops images. |
@WindQAQ @bhack
Ultimately we don't need a
Currently we're using the no-SLA image that builds the TF pip package (AFAIK) and is also being used by tensorflow-io. The custom-op image for TF2.4 should work but that was built after we we're already testing release candidates. I've set time aside tomorrow to create an issue on the current state of those containers and see if we can get the TF build team to consolidate on a modular set of Dockerfiles. |
e74a991
to
e56acc5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for investigation!
Description
GPU test is currently failing on master for what looks like a frivolous error. Appears to be related to bazelbuild/bazel#10356