Skip to content

[cDAC] Disable Helix job monitor on XPlat dump generation - #132043

Merged
rcj1 merged 2 commits into
dotnet:mainfrom
rcj1:draft
Aug 9, 2026
Merged

[cDAC] Disable Helix job monitor on XPlat dump generation#132043
rcj1 merged 2 commits into
dotnet:mainfrom
rcj1:draft

Conversation

@rcj1

@rcj1 rcj1 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Disable the Helix Job Monitor for cDAC cross-platform dump generation.

This restores the previous synchronous behavior where the submitting job:

  1. Waits for Helix work to complete.
  2. Downloads dumps.tar.gz into artifacts/helixresults.
  3. Publishes the dumps for the dependent cross-platform test jobs.

Root cause

After #129690 enabled the Helix Job Monitor globally, SendHelixJob began returning immediately after submission. The cDAC dump-generation flow requires DownloadFilesFromResults, but the monitor does not download those artifacts for the submitting job.

Consequently, Publish Dump Artifacts runs before artifacts/helixresults exists, although the Helix work items later complete successfully.

Fixes  #131979 

Note

This content was created with assistance from AI.

@rcj1 rcj1 self-assigned this Aug 8, 2026
@rcj1 rcj1 added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Aug 8, 2026
Copilot AI lite review requested due to automatic review settings August 8, 2026 19:18
@rcj1 rcj1 added the NO-REVIEW Experimental/testing PR, do NOT review it label Aug 8, 2026
@rcj1
rcj1 temporarily deployed to copilot-pat-pool August 8, 2026 19:18 — with GitHub Actions Inactive
@rcj1

rcj1 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

/azp run runtime-diagnostics

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 5 pipeline(s).
11 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@rcj1
rcj1 temporarily deployed to copilot-pat-pool August 8, 2026 19:18 — with GitHub Actions Inactive

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 updates the runtime-diagnostics Azure Pipelines definition to change the default mode for cDAC dump testing and to adjust the Helix submission parameters for the x-plat dump generation leg.

Changes:

  • Change the default cdacDumpTestMode parameter from single-leg to xplat.
  • Add /p:EnableHelixJobMonitor=false to the Helix submission used for x-plat dump generation.
Show a summary per file
File Description
eng/pipelines/runtime-diagnostics.yml Changes cDAC dump test mode default and modifies Helix send parameters for dump generation.

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment thread eng/pipelines/runtime-diagnostics.yml
Copilot AI review requested due to automatic review settings August 8, 2026 22:56

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.

Review details

Suppressed comments (1)

eng/pipelines/runtime-diagnostics.yml:64

  • Changing cdacDumpTestMode default to xplat will cause PR-triggered runs of this pipeline to execute the cross-platform dump generation + fan-in dump test legs by default (see the if ... cdacDumpTestMode == 'xplat' conditions later in the file). That substantially increases CI time/cost and can raise reliability risks (large dump artifacts / fan-in dependencies) for routine PR validation; nightly Schedule already forces xplat regardless of this parameter.
  default: xplat
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

Copilot AI review requested due to automatic review settings August 9, 2026 01:39
@rcj1 rcj1 changed the title Draft pr to test runtime-diagnostics pipeline [cDAC] Disable Helix job monitor on XPlat dump generation Aug 9, 2026
@rcj1 rcj1 added area-Diagnostics-cdac and removed NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it area-Diagnostics-coreclr labels Aug 9, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @steveisok, @tommcdon, @dotnet/dotnet-diag
See info in area-owners.md if you want to be subscribed.

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.

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@rcj1
rcj1 marked this pull request as ready for review August 9, 2026 01:45
@rcj1
rcj1 requested a review from hoyosjs August 9, 2026 01:45
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 5 pipeline(s).
11 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@rcj1

rcj1 commented Aug 9, 2026

Copy link
Copy Markdown
Contributor Author

/ba-g unrelated sos dump failure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants