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

Tensorflow and Tensorflow Serving are broken at HEAD #1880

Closed
damienmg opened this issue Oct 4, 2016 · 6 comments
Closed

Tensorflow and Tensorflow Serving are broken at HEAD #1880

damienmg opened this issue Oct 4, 2016 · 6 comments
Assignees
@damienmg
Labels
breakage P1 I'll work on this now. (Assignee required) type: bug under investigation

Comments

@damienmg
Copy link
Contributor

damienmg commented Oct 4, 2016

For the same causes, due to something in Android.
@damienmg damienmg added type: bug under investigation P1 I'll work on this now. (Assignee required) category: rules > android labels Oct 4, 2016
@damienmg damienmg self-assigned this Oct 4, 2016
@damienmg
Copy link
Contributor Author

damienmg commented Oct 4, 2016

Forgot to add the log: http://ci.bazel.io/job/TensorFlow/BAZEL_VERSION=HEAD,PLATFORM_NAME=linux-x86_64/437/console

@damienmg
Copy link
Contributor Author

damienmg commented Oct 5, 2016

And the repro is simply bazel fetch tensorflow/...

@damienmg
Copy link
Contributor Author

damienmg commented Oct 5, 2016

This is in the release candidate, we will break TensorFlow...

/cc @katre

@damienmg
Copy link
Contributor Author

damienmg commented Oct 5, 2016

Ouf tested with the release branch it is not in \o/, bisecting now.

@damienmg
Copy link
Contributor Author

damienmg commented Oct 5, 2016

culprit af8ebca! rollback!

@damienmg
Copy link
Contributor Author

damienmg commented Oct 5, 2016

So reason this was not caught before: no test for that /cc @kchodorow and TensorFlow just started to do it.

@damienmg damienmg mentioned this issue Oct 5, 2016
Closed
@aj-michael aj-michael mentioned this issue Dec 12, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@damienmg damienmg

Labels
breakage P1 I'll work on this now. (Assignee required) type: bug under investigation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@damienmg