Skip to content

Turn off Windows Defender in temp folder on binary build workflow#99389

Closed
huydhn wants to merge 2 commits intopytorch:mainfrom
huydhn:windows-defender-off
Closed

Turn off Windows Defender in temp folder on binary build workflow#99389
huydhn wants to merge 2 commits intopytorch:mainfrom
huydhn:windows-defender-off

Conversation

@huydhn
Copy link
Copy Markdown
Contributor

@huydhn huydhn commented Apr 18, 2023

This issue starts to show up recently https://github.com/pytorch/pytorch/actions/runs/4724983231/jobs/8385139626 and I'm pretty sure that the root cause is Windows Defender as I did a similar fix on Windows CI a while ago #96931. Without this, Windows binary build could fail flakily when Windows Defender chooses to delete/quarantine a file in the temp folder.

@huydhn huydhn added ciflow/binaries Trigger all binary build and upload jobs on the PR test-config/default labels Apr 18, 2023
@huydhn huydhn requested a review from weiwangmeta April 18, 2023 01:22
@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Apr 18, 2023
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot bot commented Apr 18, 2023

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/99389

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit b78e677:
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@huydhn huydhn marked this pull request as ready for review April 18, 2023 01:24
@huydhn huydhn requested a review from a team as a code owner April 18, 2023 01:24
@huydhn huydhn changed the title Turn off Windows Defender on temp folder in binary build workflow Turn off Windows Defender in temp folder on binary build workflow Apr 18, 2023
Copy link
Copy Markdown
Contributor

@weiwangmeta weiwangmeta left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@huydhn
Copy link
Copy Markdown
Contributor Author

huydhn commented Apr 18, 2023

@pytorchbot merge -f 'Binary build and test jobs have passed. No need to run trunk'

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

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

Labels

ciflow/binaries Trigger all binary build and upload jobs on the PR Merged merging test-config/default topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants