Skip to content

Conversation

@mattkur
Copy link
Contributor

@mattkur mattkur commented Oct 16, 2025

Looks like we aren't getting logs in our test results repo. The pipeline that uploads the logs matches on workflow name, which has changed.

Copilot AI review requested due to automatic review settings October 16, 2025 17:19
@mattkur mattkur requested a review from a team as a code owner October 16, 2025 17:19
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 updates the workflow triggers in the upload-petri-results.yml workflow to match the new pipeline naming convention. The workflow names were previously prefixed with [flowey] but have since been changed, preventing test results from being uploaded to the test results repository.

Key Changes:

  • Removed the [flowey] prefix from existing workflow names
  • Added a new workflow trigger for Release PR tests

@smalis-msft
Copy link
Contributor

Thank you, I was just looking into this too hahah.

Copy link
Contributor

@smalis-msft smalis-msft left a comment

Choose a reason for hiding this comment

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

Is there no way more stable than name?

@mattkur
Copy link
Contributor Author

mattkur commented Oct 16, 2025

Is there no way more stable than name?

Good question, but I can't chase that rabbit right now. Maybe @benhillis knows?

@smalis-msft
Copy link
Contributor

Doesn't look like there's any option other than name. Something to keep in mind in the future
https://docs.github.com/en/actions/reference/workflows-and-actions/events-that-trigger-workflows#workflow_run

@mattkur mattkur enabled auto-merge (squash) October 16, 2025 17:23
@mattkur mattkur disabled auto-merge October 16, 2025 18:37
@mattkur mattkur merged commit c85e6e7 into microsoft:main Oct 16, 2025
48 of 49 checks passed
@mattkur mattkur deleted the upload-logs branch October 16, 2025 18:46
@github-actions
Copy link

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