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: output log of each test target #78

Closed
facaiy opened this issue Mar 6, 2019 · 3 comments
Closed

CI: output log of each test target #78

facaiy opened this issue Mar 6, 2019 · 3 comments

Comments

@facaiy
Copy link
Member

facaiy commented Mar 6, 2019

There seems only one target tensorflow_addons/ubuntu/py2/presubmit in the detail page, check out:
https://source.cloud.google.com/results/invocations/bcda13af-4a13-41c6-a7db-76ff8e3ec504/targets/tensorflow_addons%2Fubuntu%2Fpy2%2Fpresubmit/log

@av8ramit
Could we give all the details for each test target, like tensorflow/tensorflow? That would be greatly helpful for review.

@av8ramit
Copy link
Contributor

av8ramit commented Mar 8, 2019

Hi @facaiy unfortunately for reasons decided unbeknownst to me, the way the system internally is set up is that you can only run predefined bazel targets if you want the targets to be split up. I think this is for security reasons. Since the script is modified a lot externally, I've made the internal invocation a simple shell script call. We're discussing on what the best way to move forward is.

@facaiy
Copy link
Member Author

facaiy commented Mar 9, 2019

@av8ramit Thank you for the detailed reply, Amit. I start to understand why we have only one presubmit target here. It has been good enough for me. Thanks again, we wouldn't have gotten here without your work. Please update the issue when new information becomes available :-)

@facaiy facaiy closed this as completed Mar 21, 2019
@av8ramit
Copy link
Contributor

No problem @facaiy glad I was able to help!

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

No branches or pull requests

2 participants