Skip to content

investigate_ci: show build errors for all failed jobs#3014

Merged
jstarks merged 1 commit intomicrosoft:mainfrom
jstarks:better_investigate_ci
Mar 16, 2026
Merged

investigate_ci: show build errors for all failed jobs#3014
jstarks merged 1 commit intomicrosoft:mainfrom
jstarks:better_investigate_ci

Conversation

@jstarks
Copy link
Member

@jstarks jstarks commented Mar 16, 2026

  • Show build/job failure logs for all failed jobs, not just the first
  • Always show build failures regardless of whether test artifacts exist
  • Extract error-pattern lines (rustc, linker, etc.) from logs instead of dumping the raw tail
  • Fall back to check-run annotations API when logs aren't available (e.g., run still in progress), so failures are visible immediately

- Show build/job failure logs for all failed jobs, not just the first
- Always show build failures regardless of whether test artifacts exist
- Extract error-pattern lines (rustc, linker, etc.) from logs instead
  of dumping the raw tail
- Fall back to check-run annotations API when logs aren't available
  (e.g., run still in progress), so failures are visible immediately
@jstarks jstarks requested a review from a team as a code owner March 16, 2026 22:35
Copilot AI review requested due to automatic review settings March 16, 2026 22:35
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 improves the CI failure investigation script to show build/job failure logs for all failed jobs instead of just the first one, and adds smarter error extraction from logs.

Changes:

  • Iterate over all failed jobs (not just the first) and always show build failure logs regardless of test artifact availability
  • Extract error-pattern lines (rustc, linker, etc.) from logs instead of dumping raw tail, with fallback to check-run annotations API
  • Fix gh() helper to return empty string on failure when check=False (previously fell through to return stdout), and add quiet parameter

You can also share your feedback on Copilot code review. Take the survey.

@jstarks jstarks merged commit 2711b3f into microsoft:main Mar 16, 2026
59 checks passed
@jstarks jstarks deleted the better_investigate_ci branch March 16, 2026 23:06
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.

3 participants