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

Add Linux ROCm CI Pipeline #21798

Merged
merged 5 commits into from
Aug 30, 2024
Merged

Add Linux ROCm CI Pipeline #21798

merged 5 commits into from
Aug 30, 2024

Conversation

mindest
Copy link
Contributor

@mindest mindest commented Aug 20, 2024 •
edited
Loading

Description

  • Add new ROCm CI pipeline (Linux ROCm CI Pipeline) focusing on inference.
  • Resolve test errors; disable flaky tests.

based on test PR #21614.

@mindest mindest marked this pull request as ready for review August 20, 2024 06:21
@mindest mindest requested a review from a team as a code owner August 20, 2024 06:21
@mindest mindest requested review from cloudhan and PeixuanZuo August 20, 2024 06:22
@snnn
Copy link
Member

snnn commented Aug 27, 2024

It takes about 3 hours. Could it be faster? Why the code compilation takes about 2 hours?

@mindest
Copy link
Contributor Author

mindest commented Aug 28, 2024

It takes about 3 hours. Could it be faster? Why the code compilation takes about 2 hours?

Compiling composable kernel takes most of the time.
I specified the arch as MI200's gfx90a to reduce compile time (on the orttraing-amd-ci pipeline build time is ~3 hrs.). And some tests (like kernel explorer) are copied here from the training pipeline. Should this one be enabled, I will try removing duplicates between them.

@mindest
Copy link
Contributor Author

mindest commented Aug 30, 2024

Thanks @snnn, @PeixuanZuo!

@mindest mindest merged commit bfa4da4 into main Aug 30, 2024
97 checks passed
@mindest mindest deleted the linmin/new_amd_ci branch August 30, 2024 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PeixuanZuo PeixuanZuo PeixuanZuo left review comments

@snnn snnn snnn approved these changes

@cloudhan cloudhan Awaiting requested review from cloudhan

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

Successfully merging this pull request may close these issues.
3 participants
@mindest @snnn @PeixuanZuo