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.

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

Make ROCm packaging stages to a single workflow #21235

Merged
merged 8 commits into from
Jul 4, 2024
Merged

Conversation

mszhanyi
Copy link
Contributor

@mszhanyi mszhanyi commented Jul 3, 2024

Description

Make current ROCm packaging stages to a single workflow.
Reduce the possibility of all nightly packages can't be generated by one failed stage

Motivation and Context

Our plan is to reduce the complexity of the current zip-nuget pipeline to improve the stability and performance of nightly packages generation.
ROCm packaging stages has no dependencies with other packaging jobs and it's the most time-consuming route.
After this change, the most used CPU/CUDA/Mobile packaging workflow duration can be reduced roughly from 3h20m to 2h30m.

@mszhanyi mszhanyi requested a review from a team as a code owner July 3, 2024 01:35
@mszhanyi mszhanyi changed the title Make ROCm packaging as a single workflow Make ROCm packaging to a single workflow Jul 3, 2024
@mszhanyi mszhanyi requested review from PeixuanZuo and cloudhan July 3, 2024 04:26
@mszhanyi mszhanyi changed the title Make ROCm packaging to a single workflow Make ROCm packaging stages to a single workflow Jul 3, 2024
@mszhanyi mszhanyi merged commit 30b6e82 into main Jul 4, 2024
98 of 104 checks passed
@mszhanyi mszhanyi deleted the zhanyi/rcomwl branch July 4, 2024 03:07
snnn added a commit that referenced this pull request Jul 25, 2024
#21485)

### Description
Right now our "Zip-Nuget-Java-Nodejs Packaging Pipeline" is too big.
This OnDevice training part is independent of the others, so it can be
split out. Then our NPM Packaging pipeline will not depends on this
training stuff.

### Motivation and Context
Similar to #21235 

Also, this PR fixed a problem that: "NuGet_Test_Linux_Training_CPU" job
downloads artifacts from "onnxruntime-linux-x64" for getting customop
shared libs, but the job forget to declare it depends on the
"Linux_C_API_Packaging_CPU_x64" which produces the artifact. Such
problems can be hard to find when a pipeline goes big.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants