Skip to content

[ci] ci: remove nemotronh 4b L2 ckpt jobs until hybrid pattern fixed (PR #2628) - #2693

Merged
yaoyu-33 merged 1 commit into
avavre/hybrid_pattern_fixfrom
yuya/skip-nemotronh-4b-l2
Mar 7, 2026
Merged

[ci] ci: remove nemotronh 4b L2 ckpt jobs until hybrid pattern fixed (PR #2628)#2693
yaoyu-33 merged 1 commit into
avavre/hybrid_pattern_fixfrom
yuya/skip-nemotronh-4b-l2

Conversation

@yaoyu-33

@yaoyu-33 yaoyu-33 commented Mar 7, 2026

Copy link
Copy Markdown
Contributor

Comment out L2_Launch_ckpts_mbridge_to_mlm_nemotronh_4b and L2_Launch_ckpts_mlm_to_mbridge_nemotronh_4b in the functional test matrix.

Why: The previous skip via @pytest.mark.pleasefixme on the test class caused pytest to collect 0 tests (all deselected by -m "not pleasefixme"), so pytest exited with code 5 and the L2 jobs still failed. Removing the jobs from the matrix is the reliable fix until hybrid_override_pattern → hybrid_layer_pattern is resolved in PR #2628.

Made with Cursor

… fixed (PR #2628)

- Comment out L2_Launch_ckpts_mbridge_to_mlm_nemotronh_4b and
  L2_Launch_ckpts_mlm_to_mbridge_nemotronh_4b in cicd-main.yml.
- pleasefixme mark causes pytest to collect 0 tests (all skipped), leading to
  exit code 5 and job failure; removing jobs from matrix is the reliable fix.

Signed-off-by: yaoyu-33 <yaoyu.094@gmail.com>
Made-with: Cursor
@yaoyu-33
yaoyu-33 requested a review from a team as a code owner March 7, 2026 20:57
@copy-pr-bot

copy-pr-bot Bot commented Mar 7, 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.

@yaoyu-33

yaoyu-33 commented Mar 7, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test 0d29922

@yaoyu-33
yaoyu-33 merged commit b787d84 into avavre/hybrid_pattern_fix Mar 7, 2026
17 checks passed
@yaoyu-33
yaoyu-33 deleted the yuya/skip-nemotronh-4b-l2 branch March 7, 2026 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant