Skip to content

Conversation

@ailzhang
Copy link
Contributor

@ailzhang ailzhang commented Feb 6, 2020

Bazelisk automatically reads .bazelversion file and install the required version of Bazel. This saves us from updating CI script everytime we need a Bazel upgrade.
Use clang-8 for consistency with pytorch/xla repo.

@dr-ci
Copy link

dr-ci bot commented Feb 6, 2020

💊 CircleCI build failures summary and remediations

As of commit 0cc1802:

  • 1/1 failures introduced in this PR

Detailed failure analysis

One may explore the probable reasons each build failed interactively on the Dr. CI website.

🕵️ 1 new failure recognized by patterns

The following build failures do not appear to be due to upstream breakage:

See CircleCI build pytorch_linux_xenial_py2_7_9_test (1/1)

Step: "Test" (full log | pattern match details)

Feb 10 22:05:51 RuntimeError: test_nn failed!
Feb 10 22:05:51   File "/opt/python/2.7.9/lib/python2.7/site-packages/xmlrunner/runner.py", line 7, in <module> 
Feb 10 22:05:51     from .result import _XMLTestResult 
Feb 10 22:05:51   File "/opt/python/2.7.9/lib/python2.7/site-packages/xmlrunner/result.py", line 42, in <module> 
Feb 10 22:05:51     for (low, high) in _illegal_unichrs 
Feb 10 22:05:51 ValueError: chr() arg not in range(256) 
Feb 10 22:05:51 Traceback (most recent call last): 
Feb 10 22:05:51   File "test/run_test.py", line 457, in <module> 
Feb 10 22:05:51     main() 
Feb 10 22:05:51   File "test/run_test.py", line 450, in main 
Feb 10 22:05:51     raise RuntimeError(message) 
Feb 10 22:05:51 RuntimeError: test_nn failed! 
Feb 10 22:05:51 + cleanup 
Feb 10 22:05:51 + retcode=1 
Feb 10 22:05:51 + set +x 
Feb 10 22:05:51 =================== sccache compilation log =================== 
Feb 10 22:05:51 =========== If your build fails, please take a look at the log above for possible reasons =========== 
Feb 10 22:05:51 Compile requests                 10 
Feb 10 22:05:51 Compile requests executed         8 
Feb 10 22:05:51 Cache hits                        2 
Feb 10 22:05:51 Cache misses                      6 
Feb 10 22:05:51 Cache timeouts                    0 

This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker.

This comment has been revised 9 times.

@ailzhang ailzhang force-pushed the bazelisk branch 2 times, most recently from 00f5042 to 0cb459e Compare February 6, 2020 02:30
@ailzhang ailzhang changed the title Use bazelisk instead of specifying bazel version manually. [TEST]Use bazelisk instead of specifying bazel version manually. Feb 6, 2020
@ailzhang ailzhang force-pushed the bazelisk branch 2 times, most recently from 4e6d277 to 91df6d1 Compare February 10, 2020 19:46
@ailzhang ailzhang changed the title [TEST]Use bazelisk instead of specifying bazel version manually. Use bazelisk instead of specifying bazel version manually. Feb 10, 2020
@ailzhang ailzhang requested a review from mruberry February 10, 2020 21:23
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@ailzhang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Collaborator

@mruberry mruberry left a comment

Choose a reason for hiding this comment

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

Cool!

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@ailzhang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@ailzhang merged this pull request in a3e69d3.

ttumiel pushed a commit to ttumiel/pytorch that referenced this pull request Mar 4, 2020
…3036)

Summary:
Bazelisk automatically reads `.bazelversion` file and install the required version of Bazel. This saves us from updating CI script everytime we need a Bazel upgrade.
Use clang-8 for consistency with pytorch/xla repo.
Pull Request resolved: pytorch#33036

Differential Revision: D19820819

Pulled By: ailzhang

fbshipit-source-id: 1560ec225cd037a811769a509a704b0df77ea183
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants