Repository navigation
Prevent workflow telemetry failures from failing the workflow - #2395
Merged
Alexander Holstrup (aholstrup1) merged 1 commit intoOct 6, 2026
Merged
Conversation
Allow the telemetry-only composite step to fail without failing the workflow, while preserving actual PR status checks. Update YAML regression coverage and documentation. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Alexander Holstrup (aholstrup1)
requested a review
from a team
as a code owner
October 6, 2026 12:47
Copilot started reviewing on behalf of
Alexander Holstrup (aholstrup1)
October 6, 2026 12:48
View session
Alexander Holstrup (aholstrup1)
enabled auto-merge (squash)
October 6, 2026 12:48
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The implementation is focused, documented, and covered by the existing YAML regression test.
Review effort: Balanced
Findings: None
What changed in this PR
Prevents telemetry-only finalization failures from failing otherwise successful workflows.
Changes:
- Makes
WorkflowPostProcessnon-blocking. - Extends YAML regression coverage.
- Updates documentation and release notes.
| File | Description |
|---|---|
Actions/WorkflowPostProcess/action.yaml |
Allows telemetry-step failures. |
Actions/WorkflowPostProcess/README.md |
Documents non-blocking behavior. |
Tests/WorkflowPostProcess.Test.ps1 |
Verifies the new YAML setting. |
Tests/TestActionsHelper.psm1 |
Adds support for asserting continue-on-error. |
RELEASENOTES.md |
Announces the reliability fix. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Maria Zhelezova (mazhelez)
approved these changes
Oct 6, 2026
Jonas Blunck (jonasblunck)
approved these changes
Oct 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
❔What, Why & How
WorkflowPostProcessonly collects telemetry, but a GitHub API 502 left a nonzeroghexit code and failed an otherwise successful BCApps run during finalization.Set
continue-on-error: trueon the composite action's telemetry step so these failures remain visible in logs without failing the workflow. Actual build/test status checks remain unchanged. Update the YAML regression check, action documentation, and release notes.Validation: all 9 targeted WorkflowPostProcess and PullRequestStatusCheck tests passed on Windows PowerShell 5 and PowerShell 7.
Related to issue: N/A. Observed in this BCApps workflow run.
✅ Checklist