Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[browser] Add Helix Job Name for Wasm.Build.Tests jobs - #120536

#120536
Merged
maraf merged 12 commits into
maindotnet/runtime:mainfrom
maraf/HelixJobMarkerdotnet/runtime:maraf/HelixJobMarkerCopy head branch name to clipboard
Dec 18, 2025
Merged

[browser] Add Helix Job Name for Wasm.Build.Tests jobs#120536
maraf merged 12 commits into
maindotnet/runtime:mainfrom
maraf/HelixJobMarkerdotnet/runtime:maraf/HelixJobMarkerCopy head branch name to clipboard

Conversation

@maraf

@maraf maraf commented Oct 8, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
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

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 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

Comment thread src/libraries/sendtohelixhelp.proj
@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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.