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

Avoid name colision when uploading artifact #42

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

niconosenzo
Copy link
Member

@niconosenzo niconosenzo commented Oct 16, 2024

We're seeing the following error when building multi-platform images during upload artifact step:

Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run

This should be fixed by using the correct platform pair var names.

@niconosenzo niconosenzo requested review from a team and graveland October 16, 2024 15:53
@graveland
Copy link
Contributor

We're seeing the following error when building multi-platform images during upload artifact step:

Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run

This PR fixes the issue by adding a built-in matrix enumeration to the artifact name.

Context: actions/upload-artifact#480

Was this due to retries? There should only be 2 jobs from the matrix, and they're differentiated by the platform name already? If it's due to retries, you might get > 2 digests to try to create a manifest with, which would also be a problem.

@niconosenzo
Copy link
Member Author

niconosenzo commented Oct 17, 2024

Was this due to retries? There should only be 2 jobs from the matrix, and they're differentiated by the platform name already? If it's due to retries, you might get > 2 digests to try to create a manifest with, which would also be a problem.

😶‍🌫️ Just realized the problem is that I updated the var names and didn't really update them within the upload step:

          platform_slash_pair=linux/${{ matrix.platform }}
          echo "PLATFORM_DASH_PAIR=${platform_slash_pair//\//-}" >> $GITHUB_ENV   

@niconosenzo niconosenzo force-pushed the nico/avoid-artifact-name-conflict branch from 0882521 to 99bca0a Compare October 17, 2024 11:01
We're seeing the following error when building multi-platform images
during upload artifact step:

"""
Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
"""

This should be fixed by using the correct platform pair var names.
@niconosenzo niconosenzo force-pushed the nico/avoid-artifact-name-conflict branch from 99bca0a to e352996 Compare October 17, 2024 11:02
@niconosenzo niconosenzo merged commit 1500803 into main Oct 17, 2024
3 checks passed
@niconosenzo niconosenzo deleted the nico/avoid-artifact-name-conflict branch October 17, 2024 11:50
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