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

Reenabling Nuget Cuda Packaging Pipeline #20688

Merged
merged 6 commits into from
May 20, 2024
Merged

Reenabling Nuget Cuda Packaging Pipeline #20688

merged 6 commits into from
May 20, 2024

Conversation

jchen351
Copy link
Contributor

Description

Motivation and Context

@jchen351 jchen351 requested a review from a team as a code owner May 15, 2024 20:01
@snnn
Copy link
Member

snnn commented May 16, 2024

Also make a copy of https://github.com/microsoft/onnxruntime/blob/main/tools/ci_build/github/azure-pipelines/publish-nuget.yml .
You may move the original content into a template, the reference it from two places.
Then we will have two nightly nuget publish pipelines. One for CUDA 11, one for CUDA 12. They all should be auto-triggered every day.

@jchen351
Copy link
Contributor Author

Also make a copy of https://github.com/microsoft/onnxruntime/blob/main/tools/ci_build/github/azure-pipelines/publish-nuget.yml . You may move the original content into a template, the reference it from two places. Then we will have two nightly nuget publish pipelines. One for CUDA 11, one for CUDA 12. They all should be auto-triggered every day.

Can we merge this PR first to make the CG issue pass? I will create a separate PR to trigger, nightly.

Also, should the nightly CUDA 12 triggered from "Zip-nuget-java pipeline" or just "Nuge-cuda-packaging-pipline"?

@jchen351 jchen351 merged commit 11df22b into main May 20, 2024
170 of 172 checks passed
@jchen351 jchen351 deleted the Cjian/cuda12 branch May 20, 2024 17:37
@sophies927 sophies927 added the triage:approved Approved for cherrypicks for release label Jun 11, 2024
yf711 pushed a commit that referenced this pull request Jun 18, 2024
<!-- Describe your changes. -->

<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
yf711 pushed a commit that referenced this pull request Jun 18, 2024
### Description
<!-- Describe your changes. -->

### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
baijumeswani pushed a commit that referenced this pull request Jun 20, 2024
<!-- Describe your changes. -->

<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:1.18.1 triage:approved Approved for cherrypicks for release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants