KEMBAR78
Do not run CI on forks by clee2000 · Pull Request #138714 · pytorch/pytorch · GitHub
Skip to content

Conversation

@clee2000
Copy link
Contributor

@clee2000 clee2000 commented Oct 23, 2024

Add if: github.repository_owner == 'pytorch' for some jobs that were missing it

Fixes #138564

@pytorch-bot
Copy link

pytorch-bot bot commented Oct 23, 2024

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ No Failures

As of commit 72c9c9b with merge base c272526 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Oct 23, 2024
@clee2000 clee2000 marked this pull request as ready for review October 23, 2024 16:49
@clee2000 clee2000 requested a review from a team as a code owner October 23, 2024 16:49
@clee2000
Copy link
Contributor Author

@pytorchbot merge -f "I only needed to make sure the workflow files were well formatted"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

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

@Rajveer100
Copy link

Rajveer100 commented Oct 23, 2024

Thanks for addressing this, hopefully this should be it!

@malfet
Copy link
Contributor

malfet commented Oct 23, 2024

Thank you for the fix, but I'm unsure how it will fix slow workflows.. Don't we need to add something like that to trunk/slow and pull?

@clee2000
Copy link
Contributor Author

Thank you for the fix, but I'm unsure how it will fix slow workflows.. Don't we need to add something like that to trunk/slow and pull?

linux tests and builds already have the check, so those should be ok, and these make up the most of the jobs in slow/trunk etc. Unfortunately I don't think the syntax allows for add the check at the workflow level

SamGinzburg pushed a commit that referenced this pull request Oct 28, 2024
Add `if: github.repository_owner == 'pytorch'` for some jobs that were missing it

Fixes #138564
Pull Request resolved: #138714
Approved by: https://github.com/huydhn, https://github.com/kit1980
@github-actions github-actions bot deleted the csl/exclude_from_forks branch November 23, 2024 02:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] Continuous spam of PyTorch CI build status notifications in fork

6 participants