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
Set py3.x build-environment name consistently #122247
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/122247
Note: Links to docs will display an error until the docs builds have been completed. ✅ You can merge normally! (3 Unrelated Failures)As of commit f40dd66 with merge base a843bbd (): FLAKY - The following jobs failed but were likely due to flakiness present on trunk:
BROKEN TRUNK - The following job failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the fix
@pytorchbot merge -f "It's broken in trunk already, let's try to fwd fix it" |
@jeanschmidt FYI we still have a forked-PR problem, see https://github.com/pytorch/pytorch/actions/runs/8351590342/job/22860190399?pr=122247 |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Look like I need to revert an additional change here. This is a new PR coming from @clee2000 IIRC |
@pytorchbot drci |
@clee2000 I assume that failing llm retrieval job will not stop subsequent CI test jobs to continue, so I have added a flaky rule for that here https://github.com/pytorch/test-infra/blob/generated-stats/stats/flaky-rules.json#L63-L67 in case we don't want to revert #121836 |
Thanks, I've changed the bucket settings details in #122279 so the aws credentials aren't needed anymore, anyone experiencing the issue can resolve it by rebasing as well now |
@pytorchbot cherry-pick --onto release/2.3 -c critical |
#122157 checks for the Python version using `"$BUILD_ENVIRONMENT" != *py3.8*`, but some build environment uses a different style with `py3_8` instead causing numpy 2.x to be installed there wrongly, i.e. https://hud.pytorch.org/pytorch/pytorch/commit/03b987fe3fa93f398c0af5b40e512950c39a7cb6 Pull Request resolved: #122247 Approved by: https://github.com/malfet (cherry picked from commit 6fefc52)
Cherry picking #122247The cherry pick PR is at #123446 and it is recommended to link a critical cherry pick PR with an issue Details for Dev Infra teamRaised by workflow job |
#122157 checks for the Python version using `"$BUILD_ENVIRONMENT" != *py3.8*`, but some build environment uses a different style with `py3_8` instead causing numpy 2.x to be installed there wrongly, i.e. https://hud.pytorch.org/pytorch/pytorch/commit/03b987fe3fa93f398c0af5b40e512950c39a7cb6 Pull Request resolved: #122247 Approved by: https://github.com/malfet (cherry picked from commit 6fefc52) Co-authored-by: Huy Do <huydhn@gmail.com>
#122157 checks for the Python version using `"$BUILD_ENVIRONMENT" != *py3.8*`, but some build environment uses a different style with `py3_8` instead causing numpy 2.x to be installed there wrongly, i.e. https://hud.pytorch.org/pytorch/pytorch/commit/03b987fe3fa93f398c0af5b40e512950c39a7cb6 Pull Request resolved: #122247 Approved by: https://github.com/malfet
#122157 checks for the Python version using
"$BUILD_ENVIRONMENT" != *py3.8*
, but some build environment uses a different style withpy3_8
instead causing numpy 2.x to be installed there wrongly, i.e. https://hud.pytorch.org/pytorch/pytorch/commit/03b987fe3fa93f398c0af5b40e512950c39a7cb6