Skip to content

Commit a95b422

Browse files
authored
fix: Update buildpack-integration-test.yml (#165)
* Update buildpack-integration-test.yml Add ${{ inputs.builder-runtime }} to the `name` for artifact upload. This will avoid the conflict in names, where e.g. php81, php82 and php83 will try to upload the artifacts with the same name and the second and later attempts will fail: ``` Run actions/upload-artifact@v4 with: name: cloudevent_buildpack_integ_logs path: /tmp/ff_* retention-days: 5 if-no-files-found: warn compression-level: 6 overwrite: false include-hidden-files: false With the provided path, there will be 2 files uploaded Artifact name is valid! Root directory input is valid! Error: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run ``` https://github.com/GoogleCloudPlatform/functions-framework-php/actions/runs/13191006742/job/36867920300 * fix: Update buildpack-integration-test.yml change uploaded artifact name based on whether a builder-url is specified or not
1 parent ab51e35 commit a95b422

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

.github/workflows/buildpack-integration-test.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -175,6 +175,7 @@ jobs:
175175
uses: actions/upload-artifact@v4
176176
if: always()
177177
with:
178-
name: ${{ matrix.type }}_buildpack_integ_logs
178+
name: ${{ matrix.type }}_${{ inputs.builder-runtime }}_${{ inputs.builder-url == '' && 'no_builder_url' || 'with_builder_url' }}_buildpack_integ_logs
179179
path: /tmp/ff_*
180180
retention-days: 5
181+

0 commit comments

Comments
 (0)