Skip to content

feat: enhance telemetry for action download resolution and failures - #4536

Merged
TingluoHuang merged 1 commit into
mainfrom
users/tihaung/resolveactionslo
Jul 1, 2026
Merged

feat: enhance telemetry for action download resolution and failures#4536
TingluoHuang merged 1 commit into
mainfrom
users/tihaung/resolveactionslo

Conversation

@TingluoHuang

Copy link
Copy Markdown
Member

https://github.com/github/actions-dispatch/issues/764

Capture client side experience about resolving and downloading action during setup_job step

Copilot AI review requested due to automatic review settings July 1, 2026 16:41
@TingluoHuang
TingluoHuang requested a review from a team as a code owner July 1, 2026 16:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 adds job-level telemetry to capture the runner’s client-side experience when resolving action download info and when downloading actions during the setup_job step, including success/failure outcomes.

Changes:

  • Record resolve_actions telemetry around action download-info resolution, capturing either "succeeded" or the thrown exception type.
  • Record download_action telemetry around action downloads, capturing either "succeeded" or the thrown exception type.
  • Extend L0 tests to validate telemetry is emitted for success, download failure, and resolution failure scenarios.
Show a summary per file
File Description
src/Runner.Worker/ActionManager.cs Adds job telemetry emission for action resolution and download operations (success/failure).
src/Test/L0/Worker/ActionManagerL0.cs Adds/updates L0 assertions to validate the new telemetry is emitted in key scenarios.

Review details

  • Files reviewed: 2/2 changed files
  • Comments generated: 4
  • Review effort level: Low

Comment thread src/Test/L0/Worker/ActionManagerL0.cs
Comment thread src/Test/L0/Worker/ActionManagerL0.cs
Comment thread src/Runner.Worker/ActionManager.cs
Comment thread src/Runner.Worker/ActionManager.cs
@TingluoHuang
TingluoHuang merged commit 4c6d85c into main Jul 1, 2026
13 checks passed
@TingluoHuang
TingluoHuang deleted the users/tihaung/resolveactionslo branch July 1, 2026 21:29
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