Skip to content

Commit 66c718d

Browse files
authored
Update the artifact names to include a prefix when running the minimum base package tests (DataDog#21240)
1 parent cd0f4c0 commit 66c718d

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

.github/workflows/pr-test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ jobs:
7272

7373
uses: ./.github/workflows/test-target.yml
7474
with:
75-
job-name: "${{ matrix.name }}"
75+
job-name: "minimum-base-package-${{ matrix.name }}"
7676
target: "${{ matrix.target }}"
7777
target-env: "${{ matrix.target-env }}"
7878
platform: "${{ matrix.platform }}"

.github/workflows/test-target.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,8 @@ jobs:
112112
# Capture traces for a separate job to do the submission
113113
TRACE_CAPTURE_BASE_DIR: "trace-captures"
114114
TRACE_CAPTURE_LOG: "trace-captures/output.log"
115+
# Prefix for artifact names when using minimum base package
116+
MINIMUM_BASE_PACKAGE_PREFIX: "${{ inputs.minimum-base-package && 'minimum-base-package-' || '' }}"
115117

116118
permissions:
117119
# needed for compute-matrix in test-target.yml
@@ -374,7 +376,7 @@ jobs:
374376
if: inputs.repo == 'core' && always()
375377
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
376378
with:
377-
name: "${{ inputs.traces-artifact-name }}-${{ inputs.target }}${{ inputs.target-env && format('-{0}', inputs.target-env) || '' }}-${{ inputs.platform }}"
379+
name: "${{ env.MINIMUM_BASE_PACKAGE_PREFIX }}${{ inputs.traces-artifact-name }}-${{ inputs.target }}${{ inputs.target-env && format('-{0}', inputs.target-env) || '' }}-${{ inputs.platform }}"
378380
path: "${{ env.TRACE_CAPTURE_FILE }}"
379381

380382
- name: Finalize test results
@@ -389,7 +391,7 @@ jobs:
389391
if: always()
390392
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
391393
with:
392-
name: "test-results-${{ inputs.target }}${{ inputs.target-env && format('-{0}', inputs.target-env) || '' }}-${{ inputs.platform }}"
394+
name: "${{ env.MINIMUM_BASE_PACKAGE_PREFIX }}test-results-${{ inputs.target }}${{ inputs.target-env && format('-{0}', inputs.target-env) || '' }}-${{ inputs.platform }}"
393395
path: "${{ env.TEST_RESULTS_BASE_DIR }}"
394396

395397
- name: Upload coverage data as artifact
@@ -400,6 +402,6 @@ jobs:
400402
# Flaky tests will have low coverage, don't include in artifacts to avoid pipeline issues
401403
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
402404
with:
403-
name: "coverage-${{ inputs.target }}${{ inputs.target-env && format('-{0}', inputs.target-env) || '' }}-${{ inputs.platform }}"
405+
name: "${{ env.MINIMUM_BASE_PACKAGE_PREFIX }}coverage-${{ inputs.target }}${{ inputs.target-env && format('-{0}', inputs.target-env) || '' }}-${{ inputs.platform }}"
404406
path: "${{ inputs.target }}/coverage.xml"
405407
if-no-files-found: ignore

0 commit comments

Comments
 (0)