Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update workflows that use cu116 to cu117 #5361

Merged
merged 5 commits into from Apr 4, 2024
Merged

Update workflows that use cu116 to cu117 #5361

merged 5 commits into from Apr 4, 2024

Conversation

loadams
Copy link
Contributor

@loadams loadams commented Apr 3, 2024 •
edited

The following workflows were specifying runners with cu116, we are updating these to cu117.

Workflows impacted:

  • nv-accelerate-v100
  • new build: 22 passed, 5 skipped, 11 warnings in 129.04s (0:02:09)
  • old build: 22 passed, 5 skipped, 11 warnings in 318.84s (0:05:18)
  • nv-ds-chat
  • new build: 15 passed, 1 skipped in 2729.91s (0:45:29)
  • old build: 15 passed, 1 skipped in 3511.82s (0:58:31)
  • nv-inference - recently failing and disabled, needs fixes.
  • new build: 36 failed, 74 passed, 95 skipped, 4 warnings in 877.45s (0:14:37)
  • old build: 36 failed, 74 passed, 95 skipped, 4 warnings in 3633.34s (1:00:33)
  • nv-mii
  • new build: 4 passed, 23 deselected, 3 warnings in 116.28s (0:01:56)
  • old build: 4 passed, 23 deselected, 3 warnings in 196.79s (0:03:16)
  • nv-nightly
  • new build: 3 passed, 3 skipped, 4713 deselected, 1 warning in 1831.83s (0:30:31)
  • old build: 3 passed, 3 skipped, 4713 deselected, 1 warning in 2459.06s (0:40:59)
  • nv-torch-latest-v100
  • new build: 947 passed, 169 skipped, 4 warnings in 2550.25s (0:42:30) and 61 passed, 4 skipped, 4643 deselected, 1 warning in 563.34s (0:09:23)
  • old build: 947 passed, 169 skipped, 4 warnings in 3216.47s (0:53:36) and 61 passed, 4 skipped, 4643 deselected, 1 warning in 611.17s (0:10:11)
  • nv-torch-nightly-v100
  • new build: 13 failed, 982 passed, 121 skipped, 4 warnings in 2691.26s (0:44:51)
  • old build: 13 failed, 982 passed, 121 skipped, 4 warnings in 3117.03s (0:51:57)
  • nv-transformers-v100 - disabled for 4 months, needs work regardless.

@loadams loadams requested a review from mrwyattii as a code owner April 3, 2024 19:28
@loadams loadams added this pull request to the merge queue Apr 4, 2024
Merged via the queue into master with commit 4621ba4 Apr 4, 2024
13 of 14 checks passed
@loadams loadams deleted the loadams/update-cu117 branch April 4, 2024 19:57
rraminen pushed a commit to ROCm/DeepSpeed that referenced this pull request May 9, 2024
@loadams @rraminen
Update workflows that use cu116 to cu117 (microsoft#5361)
bbf9df4 
The following workflows were specifying runners with cu116, we are
updating these to cu117.

Workflows impacted:
- [x] nv-accelerate-v100
- [new
build](https://github.com/microsoft/DeepSpeed/actions/runs/8557768042/job/23450811816?pr=5361):
22 passed, 5 skipped, 11 warnings in 129.04s (0:02:09)
- [old
build](https://github.com/microsoft/DeepSpeed/actions/runs/8547131990/job/23418750315):
22 passed, 5 skipped, 11 warnings in 318.84s (0:05:18)
- [x] nv-ds-chat
- [new
build](https://github.com/microsoft/DeepSpeed/actions/runs/8546543733/job/23417119129):
15 passed, 1 skipped in 2729.91s (0:45:29)
- [old
build](https://github.com/microsoft/DeepSpeed/actions/runs/8531148226/job/23370268262):
15 passed, 1 skipped in 3511.82s (0:58:31)
- [x] nv-inference - recently failing and disabled, needs fixes.
- [new
build](https://github.com/microsoft/DeepSpeed/actions/runs/8558749560):
36 failed, 74 passed, 95 skipped, 4 warnings in 877.45s (0:14:37)
- [old
build](https://github.com/microsoft/DeepSpeed/actions/runs/8546382497/job/23416626521):
36 failed, 74 passed, 95 skipped, 4 warnings in 3633.34s (1:00:33)
- [x] nv-mii
- [new
build](https://github.com/microsoft/DeepSpeed/actions/runs/8557768075/job/23450812054?pr=5361):
4 passed, 23 deselected, 3 warnings in 116.28s (0:01:56)
- [old
build](https://github.com/microsoft/DeepSpeed/actions/runs/8547246351/job/23419064526):
4 passed, 23 deselected, 3 warnings in 196.79s (0:03:16)
- [x] nv-nightly
- [new
build](https://github.com/microsoft/DeepSpeed/actions/runs/8557763671/job/23450792634):
3 passed, 3 skipped, 4713 deselected, 1 warning in 1831.83s (0:30:31)
- [old
build](https://github.com/microsoft/DeepSpeed/actions/runs/8547230983/job/23419020962):
3 passed, 3 skipped, 4713 deselected, 1 warning in 2459.06s (0:40:59)
- [x] nv-torch-latest-v100
- [new
build](https://github.com/microsoft/DeepSpeed/actions/runs/8557768039/job/23450811779):
947 passed, 169 skipped, 4 warnings in 2550.25s (0:42:30) and 61 passed,
4 skipped, 4643 deselected, 1 warning in 563.34s (0:09:23)
- [old
build](https://github.com/microsoft/DeepSpeed/actions/runs/8547232496/job/23419024966):
947 passed, 169 skipped, 4 warnings in 3216.47s (0:53:36) and 61 passed,
4 skipped, 4643 deselected, 1 warning in 611.17s (0:10:11)
- [x] nv-torch-nightly-v100
- [new
build](https://github.com/microsoft/DeepSpeed/actions/runs/8558930744):
13 failed, 982 passed, 121 skipped, 4 warnings in 2691.26s (0:44:51)
- [old
build](https://github.com/microsoft/DeepSpeed/actions/runs/8558895638):
13 failed, 982 passed, 121 skipped, 4 warnings in 3117.03s (0:51:57)
- [x] nv-transformers-v100 - disabled for 4 months, needs work
regardless.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lekurile lekurile lekurile approved these changes

@mrwyattii mrwyattii Awaiting requested review from mrwyattii mrwyattii is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@loadams @lekurile