Skip to content

Conversation

@maraf
Copy link
Member

@maraf maraf commented Oct 8, 2025

Prefix messages produced by Helix SDK in the AzDO text log with job prefix. Example:

Sent Helix Job 'NoFingerprint-ST'; see work items at https://helix.dot.net...
Sent Helix Job 'JavascriptBundler-ST'; see work items at https://helix.dot.net...

Based on dotnet/arcade#16204

@maraf maraf self-assigned this Oct 8, 2025
@maraf maraf added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Oct 8, 2025
@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Oct 8, 2025
@maraf maraf changed the title Identify HelixJob URLs with human readable marker [browser] Add Helix Job Name for Wasm.Build.Tests jobs Oct 9, 2025
@maraf maraf added area-Infrastructure-mono and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Oct 9, 2025
@maraf maraf added this to the 11.0.0 milestone Oct 9, 2025
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @akoeplinger, @matouskozak, @simonrozsival
See info in area-owners.md if you want to be subscribed.

@maraf maraf removed the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Dec 17, 2025
@maraf maraf requested a review from akoeplinger December 17, 2025 14:23
@maraf maraf marked this pull request as ready for review December 17, 2025 14:23
Copilot AI review requested due to automatic review settings December 17, 2025 14:23
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 adds support for naming Helix jobs in Wasm.Build.Tests jobs by propagating a WorkItemPrefix property to set both the HelixJobName property and the jobName value in HelixProperties. This enables prefixed job names to appear in AzDO text logs (e.g., "Sent Helix Job 'NoFingerprint-ST'").

Key changes:

  • Adds a new HelixProperties item with jobName value derived from WorkItemPrefix (with trailing dash trimmed)
  • Introduces a new SetHelixJobName target to set the HelixJobName property before test execution

@maraf maraf merged commit a61c6f8 into main Dec 18, 2025
93 checks passed
@maraf maraf deleted the maraf/HelixJobMarker branch December 18, 2025 16:28
@github-actions github-actions bot locked and limited conversation to collaborators Jan 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants