Skip to content

Add smoke test to Windows CI#39941

Closed
guyang3532 wants to merge 1 commit intopytorch:masterfrom
guyang3532:add_win_smoke_test
Closed

Add smoke test to Windows CI#39941
guyang3532 wants to merge 1 commit intopytorch:masterfrom
guyang3532:add_win_smoke_test

Conversation

@guyang3532
Copy link
Copy Markdown
Contributor

@guyang3532 guyang3532 commented Jun 12, 2020 •
edited
Loading

Related pr: pytorch/builder#456
test at:#39943

@guyang3532 guyang3532 requested a review from peterjc123 June 12, 2020 11:27
@guyang3532 guyang3532 mentioned this pull request Jun 12, 2020
Merged
@dr-ci
Copy link
Copy Markdown

dr-ci bot commented Jun 12, 2020 •
edited
Loading

💊 CI failures summary and remediations

As of commit adf218b (more details on the Dr. CI page):

✅ None of the CI failures appear to be your fault 💚

❄️ 1 failure tentatively classified as flaky

but reruns have not yet been triggered to confirm:

See CircleCI build pytorch_linux_xenial_cuda10_2_cudnn7_py3_gcc7_test (1/1)

Step: "Run tests" (full log | diagnosis details | 🔁 rerun) ❄️

Jun 16 04:32:59 ConnectionResetError: [Errno 104] Connection reset by peer 
Jun 16 04:32:59   File "/opt/conda/lib/python3.6/multiprocessing/connection.py", line 455, in accept 
Jun 16 04:32:59     deliver_challenge(c, self._authkey) 
Jun 16 04:32:59   File "/opt/conda/lib/python3.6/multiprocessing/connection.py", line 722, in deliver_challenge 
Jun 16 04:32:59     response = connection.recv_bytes(256)        # reject large message 
Jun 16 04:32:59   File "/opt/conda/lib/python3.6/multiprocessing/connection.py", line 216, in recv_bytes 
Jun 16 04:32:59     buf = self._recv_bytes(maxlength) 
Jun 16 04:32:59   File "/opt/conda/lib/python3.6/multiprocessing/connection.py", line 407, in _recv_bytes 
Jun 16 04:32:59     buf = self._recv(4) 
Jun 16 04:32:59   File "/opt/conda/lib/python3.6/multiprocessing/connection.py", line 379, in _recv 
Jun 16 04:32:59     chunk = read(handle, remaining) 
Jun 16 04:32:59 ConnectionResetError: [Errno 104] Connection reset by peer 
Jun 16 04:32:59 /opt/conda/lib/python3.6/multiprocessing/semaphore_tracker.py:143: UserWarning: semaphore_tracker: There appear to be 14 leaked semaphores to clean up at shutdown 
Jun 16 04:32:59   len(cache)) 
Jun 16 04:33:02 Process ErrorTrackingProcess-126: 
Jun 16 04:33:02 Traceback (most recent call last): 
Jun 16 04:33:02   File "/opt/conda/lib/python3.6/multiprocessing/process.py", line 258, in _bootstrap 
Jun 16 04:33:02     self.run() 
Jun 16 04:33:02   File "/var/lib/jenkins/workspace/test/test_dataloader.py", line 362, in run 
Jun 16 04:33:02     super(ErrorTrackingProcess, self).run() 
Jun 16 04:33:02   File "/opt/conda/lib/python3.6/multiprocessing/process.py", line 93, in run 
Jun 16 04:33:02     self._target(*self._args, **self._kwargs)
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 or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 6 times.

@guyang3532 guyang3532 mentioned this pull request Jun 12, 2020
Closed
peterjc123
peterjc123 reviewed Jun 13, 2020
View reviewed changes
.circleci/cimodel/data/binary_build_definitions.py Outdated
Copy link
Copy Markdown
Collaborator

@peterjc123 peterjc123 Jun 13, 2020

Choose a reason for hiding this comment

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

I think you could also remove the definition of the item.

@guyang3532 guyang3532 force-pushed the add_win_smoke_test branch from 5ddc86c to adf218b Compare June 16, 2020 03:06
@peterjc123
Copy link
Copy Markdown
Collaborator

peterjc123 commented Jun 16, 2020

@pytorchbot merge this please

@pytorchbot pytorchbot added the merge-this-please Was marked for merge with @pytorchbot merge this please label Jun 16, 2020
facebook-github-bot
facebook-github-bot reviewed Jun 16, 2020
View reviewed changes
Copy link
Copy Markdown
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.

@ezyang is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Jun 16, 2020

@ezyang merged this pull request in c065049.

xwang233 pushed a commit to xwang233/pytorch that referenced this pull request Jun 20, 2020
@guyang3532 @xwang233
Add smoke test to Windows CI (pytorch#39941)
c40073a 
Summary:
Related pr: pytorch/builder#456
test at:pytorch#39943
Pull Request resolved: pytorch#39941

Differential Revision: D22068004

Pulled By: ezyang

fbshipit-source-id: 5244f64d842b3a8bf0af720dffb4b1a0370cc178
@guyang3532 guyang3532 mentioned this pull request Jul 28, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@seemethere seemethere seemethere approved these changes

@peterjc123 peterjc123 peterjc123 approved these changes

+1 more reviewer

@facebook-github-bot facebook-github-bot facebook-github-bot left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

merge-this-please Was marked for merge with @pytorchbot merge this please Merged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@guyang3532 @peterjc123 @facebook-github-bot @seemethere @pytorchbot @mruberry