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

Split ondevice training cpu packaging pipeline to a separated pipeline #21485

Merged
merged 4 commits into from
Jul 25, 2024

Conversation

snnn
Copy link
Member

@snnn snnn commented Jul 24, 2024

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.

@snnn snnn requested a review from a team as a code owner July 24, 2024 19:54
@snnn snnn merged commit 4167b68 into main Jul 25, 2024
99 checks passed
@snnn snnn deleted the snnn/split branch July 25, 2024 17:58
snnn pushed a commit that referenced this pull request Jul 29, 2024
### Description
Since the onedevice training cpu packaging has been a separated
pipeline, it's nuget package publishing step must be moved as well.

### Motivation and Context
Fixes the exception in Nuget Publishing Packaging Pipeline caused by
#21485
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