Skip to content

[TRTLLMINF-579][infra] Rename Check Test List stage to Check-Test-List and mark it skipped when not run #21497

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
EmmaQiaoCh merged 3 commits into from
Nov 6, 2026
Merged

[TRTLLMINF-579][infra] Rename Check Test List stage to Check-Test-List and mark it skipped when not run #21497

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
01384a4
[None][infra] Rename Check Test List stage to Check-Test-List and mar…
EmmaQiaoCh Nov 3, 2026
378ae88
[None][infra] Mark Check-Test-List skip on a nested Run stage so Blue…
EmmaQiaoCh Nov 3, 2026
ea196ec
[None][infra] Generalize the stage-skip helper and drop unreachable r…
EmmaQiaoCh Nov 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 24 additions & 14 deletions jenkins/L0_MergeRequest.groovy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -931,9 +931,19 @@ def launchReleaseCheck(pipeline, globalVars)
})
}

def skipTestListCheck(String message)
Comment thread
EmmaQiaoCh marked this conversation as resolved.
Resolved
{
// Mark a nested stage so Blue Ocean renders the skip; a mark on the parallel branch's own stage shows as NOT_BUILT.
def runStage = "[Check-Test-List] Run"
stage(runStage) {
echo message
jUtils.markStageSkippedForConditional(runStage)
}
}

def launchTestListCheck(pipeline, globalVars)
{
def key = "Check Test List"
def key = "Check-Test-List"
def image = globalVars["LLM_DOCKER_IMAGE"]
trtllm_utils.launchKubernetesPod(pipeline, createKubernetesPodConfig(image, "package"), "trt-llm", {
stage("[${key}] Run") {
Expand Down Expand Up @@ -2557,22 +2567,22 @@ def launchStages(pipeline, reuseBuild, testFilter, enableFailFast, globalVars)
launchReleaseCheck(this, globalVars)
}
},
"Check Test List": {
"Check-Test-List": {
script {
if (testFilter[INFRA_DRY_RUN]) {
echo "Skipping Check Test List for the infrastructure dry run."
skipTestListCheck("Skipping Check Test List for the infrastructure dry run.")
return
} else if (GEN_POST_MERGE_BUILDS_ONLY) {
echo "Skipping Check Test List (GenPostMergeBuilds mode: builds only)"
skipTestListCheck("Skipping Check Test List (GenPostMergeBuilds mode: builds only)")
return
} else if (runMode == "nightly_release") {
echo "Skipping Check Test List for nightly_release."
skipTestListCheck("Skipping Check Test List for nightly_release.")
return
} else if (testFilter[(ONLY_ONE_GROUP_CHANGED)] == "Docs") {
echo "Skipping Check Test List for Docs-only changes."
skipTestListCheck("Skipping Check Test List for Docs-only changes.")
return
} else if (env.JOB_NAME ==~ /.*BuildDockerImageSanityTest.*/) {
echo "Skipping Check Test List for BuildDockerImageSanityTest."
skipTestListCheck("Skipping Check Test List for BuildDockerImageSanityTest.")
return
}
launchTestListCheck(this, globalVars)
Expand Down Expand Up @@ -3235,7 +3245,7 @@ def launchStages(pipeline, reuseBuild, testFilter, enableFailFast, globalVars)
stages += dockerBuildJob
}

def alwaysFailFastStages = ["Release-Check", "Check Test List"] as Set
def alwaysFailFastStages = ["Release-Check", "Check-Test-List"] as Set
parallelJobs = stages.collectEntries{key, value -> [key, {
script {
stage(key) {
Expand Down Expand Up @@ -3342,16 +3352,16 @@ pipeline {
}
}
},
"Check Test List": {
stage("Check Test List") {
"Check-Test-List": {
stage("Check-Test-List") {
if (testFilter[INFRA_DRY_RUN]) {
echo "Skipping Check Test List for the infrastructure dry run."
skipTestListCheck("Skipping Check Test List for the infrastructure dry run.")
} else if (runMode == "nightly_release") {
echo "Skipping Check Test List for nightly_release."
skipTestListCheck("Skipping Check Test List for nightly_release.")
} else if (testFilter[(ONLY_ONE_GROUP_CHANGED)] == "Docs") {
echo "Skipping Check Test List for Docs-only changes."
skipTestListCheck("Skipping Check Test List for Docs-only changes.")
} else if (env.JOB_NAME ==~ /.*BuildDockerImageSanityTest.*/) {
echo "Skipping Check Test List for BuildDockerImageSanityTest."
skipTestListCheck("Skipping Check Test List for BuildDockerImageSanityTest.")
Comment thread
EmmaQiaoCh marked this conversation as resolved.
Resolved
} else {
launchTestListCheck(this, globalVars)
}
Expand Down
Loading