Skip to content

Align GHA workflows in the scope of report uploads#853

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

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

Conversation

@mawilk90
Copy link

@mawilk90 mawilk90 commented Feb 5, 2026

📝 Description

GHA workflows upload test results to TOD on push, PR and manual (workflow_dispatch) - aligning integration test workflows to upload results on push and/or PR only.

https://track.akamai.com/jira/browse/TPT-4190

✔️ How to Test

Manual run of workflows should not upload test results to TOD.

@mawilk90 mawilk90 requested a review from a team as a code owner February 5, 2026 09:14
@mawilk90 mawilk90 added the improvement for improvements in existing functionality in the changelog. label Feb 5, 2026
@mawilk90 mawilk90 requested review from a team, Copilot, jriddle-linode, lgarber-akamai, vshanthe and zliang-akamai and removed request for a team February 5, 2026 09:14
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 restricts test result uploads in the e2e-suite workflow to only occur on push events, removing uploads for pull requests and manual workflow dispatches. This aligns the integration test workflows with the project's test reporting policy.

Changes:

  • Modified the process-upload-report job condition to only run on push events
  • Removed an extraneous blank line

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

@mawilk90
Copy link
Author

mawilk90 commented Feb 6, 2026

Hi, I decided to modify a bit the condition for test report uploads because sometimes it may be reasonable to push results even on manual run - in my opinion TOD is more readable and, especially in case of some regression runs (like yesterday), it would be helpful to have reports in TOD on demand

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants