Skip to content

Conversation

@kit1980
Copy link
Contributor

@kit1980 kit1980 commented Oct 21, 2024

Trying to loading it caused a big issue with 2.5.0 release - #138324

cufile is not actually used currently by default, see #133489

Trying to loading it caused a big issue with 2.5.0 release - #138324
cufile is not actually used currently by default - #133489
@pytorch-bot
Copy link

pytorch-bot bot commented Oct 21, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit 075efb5 with merge base c1ead6f (image):
💚 Looks good so far! There are no failures yet. 💚

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

Copy link
Contributor

@atalman atalman left a comment

Choose a reason for hiding this comment

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

lgtm

@kit1980
Copy link
Contributor Author

kit1980 commented Oct 21, 2024

@pytorchbot label "topic: not user facing"

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Oct 21, 2024
@kit1980
Copy link
Contributor Author

kit1980 commented Oct 21, 2024

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Oct 21, 2024
@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

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

@kit1980
Copy link
Contributor Author

kit1980 commented Oct 22, 2024

@pytorchbot cherry-pick --onto release/2.5 -c critical

@pytorchbot
Copy link
Collaborator

Cherry picking #138501

The cherry pick PR is at #138539 and it is recommended to link a critical cherry pick PR with an issue.

Details for Dev Infra team Raised by workflow job

kit1980 added a commit that referenced this pull request Oct 22, 2024
Don't try to load cufile (#138501)

Trying to loading it caused a big issue with 2.5.0 release - #138324

cufile is not actually used currently by default, see #133489

Pull Request resolved: #138501
Approved by: https://github.com/atalman, https://github.com/mikaylagawarecki, https://github.com/malfet

(cherry picked from commit 012ff2a)

Co-authored-by: Sergii Dymchenko <[email protected]>
@github-actions github-actions bot deleted the sdym/cufile-main branch November 22, 2024 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk Trigger trunk jobs on your pull request Merged release notes: build release notes category topic: binaries

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants