Skip to content

ci: add test results artifact uploads to CI workflows#15577

Merged
Joibel merged 1 commit intoargoproj:mainfrom
Joibel:claude/add-test-artifacts-KJc3M
Feb 17, 2026
Merged

ci: add test results artifact uploads to CI workflows#15577
Joibel merged 1 commit intoargoproj:mainfrom
Joibel:claude/add-test-artifacts-KJc3M

Conversation

@Joibel
Copy link
Member

@Joibel Joibel commented Feb 17, 2026

Motivation

Test results are currently not being persisted as artifacts in the CI pipeline, making it difficult to review test outcomes after workflow completion. Adding artifact uploads will improve visibility into test results and aid in debugging failures.

Modifications

Added test results artifact uploads to two CI jobs:

  1. Unit tests job: Uploads test/reports/json/unit.json as test-results-unit artifact
  2. E2E tests job: Uploads test/reports/json/{test}.json as test-results-{test}-{profile}-k8s-{version} artifact

Both uploads:

  • Run on all outcomes (if: ${{ always() }})
  • Use actions/upload-artifact@v6.0.0
  • Warn if no files are found (non-blocking)
  • Retain artifacts for 10 days

Verification

CI workflow validation - the changes are configuration-only and will be validated by GitHub Actions when the workflow runs.

Documentation

N/A - internal CI configuration change

Upload gotestsum JSON output as artifacts for unit tests and all E2E
test matrix entries. This makes test results available for download
and analysis after CI runs. Artifacts use `if: always()` to ensure
results are captured even on test failure.

https://claude.ai/code/session_01XiSAWwFATArzJr4nUXiRdZ
Signed-off-by: Claude <noreply@anthropic.com>
@Joibel Joibel force-pushed the claude/add-test-artifacts-KJc3M branch from 885a598 to 960a615 Compare February 17, 2026 10:12
@Joibel Joibel enabled auto-merge (squash) February 17, 2026 13:13
@Joibel Joibel merged commit 465af68 into argoproj:main Feb 17, 2026
39 checks passed
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.

2 participants