Skip to content

Align GHA workflows in the scope of report uploads#754

Open
mawilk90 wants to merge 4 commits intolinode:devfrom
mawilk90:hotfix/align_report_uploads
Open

Align GHA workflows in the scope of report uploads#754
mawilk90 wants to merge 4 commits intolinode:devfrom
mawilk90:hotfix/align_report_uploads

Conversation

@mawilk90
Copy link

@mawilk90 mawilk90 commented Feb 6, 2026

📝 Description

Added workflow logic to upload test results for manual runs on demand only

✔️ How to Test

Manual run of the workflow should not upload test results to TOD by default.

@mawilk90 mawilk90 requested a review from Copilot February 6, 2026 13:43
@mawilk90 mawilk90 requested a review from a team as a code owner February 6, 2026 13:43
@mawilk90 mawilk90 added the improvement for improvements in existing functionality in the changelog. label Feb 6, 2026
@mawilk90 mawilk90 requested review from ezilber-akamai and mgwoj and removed request for a team February 6, 2026 13:43
@mawilk90 mawilk90 added the testing for updates to the testing suite in the changelog. label Feb 6, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refines the test report upload behavior in the integration tests workflow by adding conditional logic to control when test results are uploaded to object storage. The change ensures that manual workflow runs (workflow_dispatch) only upload test reports when explicitly requested via a new input parameter, while automatic runs triggered by pushes to main continue to upload reports by default.

Changes:

  • Added a new boolean input parameter test_report_upload for manual workflow triggers with a default value of false
  • Updated the upload job's conditional logic to require explicit opt-in for report uploads during manual runs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement for improvements in existing functionality in the changelog. testing for updates to the testing suite in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant