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

ci: pin to non-broken estimator nightlies #2668

Merged
merged 1 commit into from
Sep 18, 2019

Conversation

wchargin
Copy link
Contributor

Summary:
Due to tensorflow/tensorflow#32621.

This unblocks our CI until tomorrow’s nightlies are pushed.

Test Plan:
Ran this 5-line shell command (with pip install changed to echo) once with TF_VERSION_IDset to each value from theenv` section, and noted
that in all cases it succeeds and that it only prints output in the
correct case.

wchargin-branch: pin-estimator-20190917-nightly

Summary:
Due to <tensorflow/tensorflow#32621>.

This unblocks our CI until tomorrow’s nightlies are pushed.

Test Plan:
Ran this 5-line shell command (with `pip install` changed to `echo) once
with `TF_VERSION_ID` set to each value from the `env` section, and noted
that in all cases it succeeds and that it only prints output in the
correct case.

wchargin-branch: pin-estimator-20190917-nightly
Copy link
Contributor

@stephanwlee stephanwlee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:(

@wchargin
Copy link
Contributor Author

Note that the current build on master will happen to pass because the
tests that catch this (core_plugin_test and scalars_plugin_test) are
cached and not re-run at that commit. But this is brittle, and if we
were to push a release branch then the tests would suddenly fail.

@nfelt
Copy link
Contributor

nfelt commented Sep 18, 2019

I'm not even sure that's true for a new release branch, since it inherits the cache from master by default: https://docs.travis-ci.com/user/caching/

@wchargin wchargin merged commit e4a7049 into master Sep 18, 2019
@wchargin wchargin deleted the wchargin-pin-estimator-20190917-nightly branch September 18, 2019 16:46
@wchargin
Copy link
Contributor Author

I'm not even sure that's true for a new release branch, since it
inherits the cache from master by default:
https://docs.travis-ci.com/user/caching/

That’s… unfortunate.

In any case, this way we don’t have to worry about it. :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants