Skip to content

Correctly exclude the Helix Job Monitor stage in coreclr outerloop - #132882

Merged
premun merged 1 commit into
dotnet:mainfrom
premun:prvysoky/disable
Aug 28, 2026
Merged

Correctly exclude the Helix Job Monitor stage in coreclr outerloop#132882
premun merged 1 commit into
dotnet:mainfrom
premun:prvysoky/disable

Conversation

@premun

@premunpremun commented Aug 28, 2026

Copy link
Copy Markdown
Member

No description provided.

CopilotAI lite review requested due to automatic review settings August 28, 2026 10:43
@github-actionsgithub-actionsBot added the area-Infrastructure-coreclr Only use for closed issues label Aug 28, 2026
@azure-pipelines

azure-pipelinesBot commented Aug 28, 2026

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

@premun

premun commented Aug 28, 2026

Copy link
Copy Markdown
MemberAuthor

/ba-g Killing the monitor asap to reduce load on AzDO

@premun
premun enabled auto-merge (squash) August 28, 2026 10:44
CopilotAI reviewed Aug 28, 2026
View reviewed changes

CopilotAI 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 CoreCLR outerloop Azure Pipelines definition to conditionally include the Helix Job Monitor job template only when enableHelixJobMonitor is enabled, instead of always injecting the monitor job.

Changes:

  • Wraps the helix-job-monitor.yml job template inclusion in a compile-time conditional (${{ if ... }}) driven by enableHelixJobMonitor.
  • Leaves the existing enableHelixJobMonitor: false default in this pipeline, so the monitor job is excluded by default for this definition.

@premun
premun merged commit 11b8941 into dotnet:mainAug 28, 2026
20 of 103 checks passed
@premun
premun deleted the prvysoky/disable branch August 28, 2026 10:55
@premun

premun commented Aug 28, 2026

Copy link
Copy Markdown
MemberAuthor

/backport to release/11.0

@github-actions

github-actionsBot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Started backporting to release/11.0 (link to workflow run)

@github-actionsgithub-actionsBot mentioned this pull request Aug 28, 2026
Merged
akoeplinger pushed a commit that referenced this pull request Aug 28, 2026
@github-actions@premun
[release/11.0] Correctly exclude the Helix Job Monitor stage in `core…
efe877f
…clr outerloop` (#132883)
Backport of #132882 to release/11.0
/cc @premun
Co-authored-by: Přemek Vysoký <premek.vysoky@microsoft.com>
@build-analysisbuild-analysisBot mentioned this pull request Aug 28, 2026
Open
@dotnet-maestrodotnet-maestroBot mentioned this pull request Aug 29, 2026
Merged
@dotnet-milestone-botdotnet-milestone-botBot added this to the 12.0-preview1 milestone Aug 29, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@akoeplingerakoeplingerakoeplinger approved these changes
+1 more reviewer
@virekumarvirekumarvirekumar approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@premunpremun

Labels

area-Infrastructure-coreclrOnly use for closed issues

Projects

Runtime Infra
Status: Done

Milestone

12.0-preview1

Development

Successfully merging this pull request may close these issues.

4 participants

@premun@akoeplinger@virekumar