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

[pytorch] Split maven publish into two parts #3273

Merged
merged 2 commits into from
Jun 27, 2024

Conversation

frankfliu
Copy link
Contributor

@frankfliu frankfliu commented Jun 24, 2024

The pytorch native jar file become two large, and sonatype auto staging break the artifacts into multiple staging repo and can not close properly.

@frankfliu frankfliu requested review from zachgk and a team as code owners June 24, 2024 16:27
Copy link
Contributor

@zachgk zachgk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some optional comments. Overall LGTM

@@ -7,6 +7,10 @@ on:
description: 'staging/snapshot, default is snapshot'
required: true
default: 'snapshot'
package_type:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than making this an input, it might be better to run as a matrix and have both package builds in one run?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The problem is on sonatype side, in gradle we are publishing each package sequentially, ideally, each package should landed in the same repo or each with its own repo. We don't have control the close the repo for each package. Even we manually run two publish command, it may or may not in the same repo. Currently, I manually publish one, login to sonatype, close the repo and run another publish.

.github/workflows/native_publish_pytorch.yml Outdated Show resolved Hide resolved
Co-authored-by: Zach Kimberg <zachary@kimberg.com>
@frankfliu frankfliu merged commit 020012d into deepjavalibrary:master Jun 27, 2024
5 checks passed
@frankfliu frankfliu deleted the split branch June 27, 2024 00:48
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.

2 participants