-
Notifications
You must be signed in to change notification settings - Fork 838
[bug] (v4) Unable to upload to same artifact name from multiple jobs #478
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
Comments
Actually, it seems this is called out here:
It just wasn't included in the "What's changed" section of the Dependabot release notes because it just has a summary saying "Lots has changed". I should've followed the link through. Seems like this is certainly intended though. |
Well, this is a bad news for me. I find convenient to use the like in https://github.com/Neoteroi/BlackSheep/actions/runs/7370452109/job/20056867940 Now if I want to upgrade my workflow, I need to publish to different folders and download artifacts from multiple sources - making the workflow look like a mess compared to how clean it used to look like. For now I stay with |
Yeah, I rolled back to v3 too. Until I'm forced to upgrade, the old way is much more convenient for me. |
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3 to 4. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v3...v4) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Possibly fixes actions/upload-artifact#478
Reverts #5830. It is breaking the release pipeline. actions/upload-artifact#478
I had to roll back also |
Same here, I also use a matrix to build multi platform releases in the same directory, and then zipping them all together, rolling back to v3 :( |
Seems like a lot of people have been bitten by this, so although it appears to have been deliberate I'm re-opening for better visibility to see if the authors want to chime in (of course, it's very possible it may just be closed as WAI). |
…a breaking change in actions/upload-artifact@v4 (see actions/upload-artifact#478)
v4 requires to have every uploaded artifact in a separate directory. - actions/upload-artifact#478 - https://github.com/actions/upload-artifact/blob/main/docs/MIGRATION.md
@awhisler: your matrix appears to have an overlap. That means that one of the other items in the matrix has the same value -- and that is the one that managed to produce an artifact. If you want help, you'd need to provide the workflow. |
## Describe your changes Update our GitHub actions versions to fix the deprecation warning. With `actions/upload-artifact@v4`, multiple jobs cannot upload to the same folder anymore. Hence, I have updated the flows according to [this migration guide](https://github.com/actions/upload-artifact/blob/main/docs/MIGRATION.md#multiple-uploads-to-the-same-named-artifact). See also this discussion: actions/upload-artifact#478 The workflows for cypress are kept on `v3` according to [this comment](streamlit#8689 (review)) to keep the usage of it simple until we retire it soonish. ## GitHub Issue Link (if applicable) ## Testing Plan - if the CI workflows run through, we should be all good --- **Contribution License Agreement** By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.
ci: adjust coverage report name for v4 artifact bug actions/upload-artifact#478
ci: adjust coverage report name for v4 artifact bug actions/upload-artifact#478
For whatever reason, v4 breaks the upload job now. See actions/upload-artifact#478. I'm just going to roll it back for the time being and retry the 1.4.0 release.
Complete rework of our workflows to account for actions/upload-artifact#478. Signed-off-by: Louis Poier <[email protected]>
Unfortunately, we cannot use v3 anymore at all... See https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/ You'd see the following error:
|
This works around a breaking change introduced in actions/upload-artifact@v4. See actions/upload-artifact#478
This works around a breaking change introduced in actions/upload-artifact@v4. See actions/upload-artifact#478
Somehow can't write to the same folder: actions/upload-artifact#478 Signed-off-by: Ryan Harper <[email protected]>
…ique names (which this did have, but one-level down in DIST) actions/upload-artifact#478
(Edit: There are solutions posted in this thread such as at #478 (comment), #478 (comment), #478 (comment))
What happened?
The PR from dependabot to upgrade to v4 is failing on my project with this error:
It seems like this is a breaking change that wasn't mentioned in the changelog and I'm not sure if it was deliberate.
There's some discussion about this behaviour in #279 and it suggests that it was fine to do this and there wouldn't be issues as long as the filenames within the artifact are unique. This was convenient to bundle the logs from several shards together into a single artifact rather than having lots of individual zip files to download.
What did you expect to happen?
I expected everything to work the same as in v3 unless it was noted as a deliberate breaking change.
How can we reproduce it?
Create multiple jobs that upload artifacts with the same name (but the files from each job are uniquely named).
Anything else we need to know?
No response
What version of the action are you using?
v4.0.0
What are your runner environments?
linux, window, macos
Are you on GitHub Enterprise Server? If so, what version?
No response
The text was updated successfully, but these errors were encountered: