Skip to content

Commit e834f04

Browse files
authored
[TRTLLM-9579][infra] Set mergeWaiveList stage UNSTABLE when there is any issue (#9692)
Signed-off-by: Yiqing Yan <yiqingy@nvidia.com>
1 parent 5d6edc3 commit e834f04

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

jenkins/L0_MergeRequest.groovy

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -371,8 +371,11 @@ def mergeWaiveList(pipeline, globalVars)
371371
} catch (InterruptedException e) {
372372
throw e
373373
} catch (Exception e) {
374-
echo "Merge test waive list failed. Error: ${e.toString()}"
375-
echo "Fallback to use the default test waive list from the PR"
374+
catchError(
375+
buildResult: 'SUCCESS',
376+
stageResult: 'UNSTABLE') {
377+
error "Merge test waive list failed. Fallback to use the default test waive list from the PR. Error: ${e.toString()}"
378+
}
376379
}
377380
}
378381

jenkins/L0_Test.groovy

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2207,6 +2207,8 @@ ${reusedTestsContent}
22072207
REUSED_TESTS_EOF
22082208
""")
22092209
echo "Appended ${reusedTests.size()} reused tests to ${waivesTxt}"
2210+
} else {
2211+
echo "No reused tests found"
22102212
}
22112213
} catch (InterruptedException e) {
22122214
throw e

0 commit comments

Comments
 (0)