-
Notifications
You must be signed in to change notification settings - Fork 105
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
Updates the CI files. #5394
Updates the CI files. #5394
Conversation
.github/workflows/publish.yml
Outdated
uses: actions/upload-artifact/merge@v4 | ||
with: | ||
name: python-client.tar | ||
pattern: *-python-client.tar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe you need to escape the dot?
.github/workflows/publish.yml
Outdated
name: "python-client.tar" | ||
name: "core-python-client.tar" | ||
path: "pulpcore/core-python-client.tar" | ||
if-no-files-found: "error" | ||
overwrite: true | ||
|
||
- name: "Merge Python client packages" | ||
uses: actions/upload-artifact/merge@v4 | ||
with: | ||
name: python-client.tar | ||
pattern: "*-python-client.tar" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are we even trying to solve here? I only see a single artifact uploaded. Why would merging it be an improvement?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In v3, multiple uploads from multiple jobs could be done to the same Artifact. This would result in a single archive, which could be useful for sending to upstream systems outside of Actions via API or UI downloads.1
We can't do that anymore with v4.
Footnotes
-
Merging multiple artifacts https://github.com/actions/upload-artifact/blob/main/docs/MIGRATION.md#merging-multiple-artifacts ↩
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think, we can:
pulp/plugin_template#866
[noissue]
[noissue]