Skip to content

fix: AUT-2228 route partial reruns to the current attempt - #296

Merged
chtruong814 merged 1 commit into
NVIDIA-NeMo:mainfrom
svcnemo-autobot:fix/partial-rerun-runner-labels
Sep 8, 2026
Merged

chtruong814 merged 1 commit into
NVIDIA-NeMo:mainfrom
svcnemo-autobot:fix/partial-rerun-runner-labels

Conversation

@svcnemo-autobot

Copy link
Copy Markdown
Contributor

Background

Partial reruns must target runners allocated for the current workflow attempt.

What changed

Added the v1.9.2 ephemeral routing integration, passed its attempt-independent base through preflight, and reconstructed attempt-qualified labels at 5 scheduling boundaries.

Details

flowchart LR
  R[Router base] --> P[Preflight]
  P --> J[Current-attempt jobs]
Loading

The existing static legacy label remains the fallback.

Tested

Passed YAML parsing, routing/output wiring and boundary-count checks, whitespace, size, conflict-marker, and private-key scans. Full pre-commit could not start because the typed checkout does not expose Git metadata; CI remains authoritative.

Signed-off-by: svcnemo-autobot <svcnemo-autobot@nvidia.com>
@svcnemo-autobot
svcnemo-autobot requested a review from a team as a code owner September 8, 2026 17:32
@copy-pr-bot

copy-pr-bot Bot commented Sep 8, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@greptile-apps

greptile-apps Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge; no concrete correctness, security, or repository-rule issue was identified.

Summary

  • Adds the pinned v1.9.2 routing workflow with legacy-label fallback configuration.
  • Passes the attempt-independent runner-label base through preflight.
  • Qualifies container-build, unit-test, and nightly-test runner labels with the current workflow attempt.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Ephemeral runner routing] -->|runner_label_base| B[Preflight]
  B -->|runner_prefix| C{Prefix starts with ephe-v2-?}
  C -->|Yes| D[Append current run_attempt]
  C -->|No| E[Keep legacy/static label]
  D --> F[Five scheduling boundaries]
  E --> F
Loading

@svcnvidia-nemo-ci

Copy link
Copy Markdown

/ok to test 6b7c286

@chtruong814
chtruong814 merged commit a6f393a into NVIDIA-NeMo:main Sep 8, 2026
32 checks passed

This branch was successfully deployed

2 active deployments
public — 6b7c2863 Deployed Sep 8, 2026 by copy-pr-bot[bot] via release / finalize / notify #251
test — 6b7c2863 Deployed Sep 8, 2026 by copy-pr-bot[bot] via cicd-wait-in-queue #1020
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