Skip to content

Commit 8c0b8ed

Browse files
committed
Catch error in merge waive list stage
Signed-off-by: Yiqing Yan <[email protected]>
1 parent f167b1f commit 8c0b8ed

File tree

1 file changed

+32
-26
lines changed

1 file changed

+32
-26
lines changed

jenkins/L0_MergeRequest.groovy

Lines changed: 32 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -335,33 +335,39 @@ def mergeWaiveList(pipeline, globalVars)
335335
sh "cp ${LLM_ROOT}/tests/integration/test_lists/waives.txt ./waives_CUR_${env.gitlabCommit}.txt"
336336
sh "cp ${LLM_ROOT}/jenkins/scripts/mergeWaiveList.py ./"
337337

338-
// Get TOT waive list
339-
LLM_TOT_ROOT = "llm-tot"
340-
targetBranch = env.gitlabTargetBranch ? env.gitlabTargetBranch : globalVars[TARGET_BRANCH]
341-
echo "Target branch: ${targetBranch}"
342-
withCredentials([string(credentialsId: 'default-sync-llm-repo', variable: 'DEFAULT_SYNC_LLM_REPO')]) {
343-
trtllm_utils.checkoutSource(DEFAULT_SYNC_LLM_REPO, targetBranch, LLM_TOT_ROOT, false, false)
338+
try {
339+
// Get TOT waive list
340+
LLM_TOT_ROOT = "llm-tot"
341+
targetBranch = env.gitlabTargetBranch ? env.gitlabTargetBranch : globalVars[TARGET_BRANCH]
342+
echo "Target branch: ${targetBranch}"
343+
withCredentials([string(credentialsId: 'default-sync-llm-repo', variable: 'DEFAULT_SYNC_LLM_REPO')]) {
344+
trtllm_utils.checkoutSource(DEFAULT_SYNC_LLM_REPO, targetBranch, LLM_TOT_ROOT, false, false)
345+
}
346+
targetBranchTOTCommit = sh (script: "cd ${LLM_TOT_ROOT} && git rev-parse HEAD", returnStdout: true).trim()
347+
echo "Target branch TOT commit: ${targetBranchTOTCommit}"
348+
sh "cp ${LLM_TOT_ROOT}/tests/integration/test_lists/waives.txt ./waives_TOT_${targetBranchTOTCommit}.txt"
349+
350+
// Get waive list diff in current MR
351+
def diff = getMergeRequestOneFileChanges(pipeline, globalVars, "tests/integration/test_lists/waives.txt")
352+
353+
// Write diff to a temporary file to avoid shell escaping issues
354+
writeFile file: 'diff_content.txt', text: diff
355+
356+
// Merge waive lists
357+
sh """
358+
python3 mergeWaiveList.py \
359+
--cur-waive-list=waives_CUR_${env.gitlabCommit}.txt \
360+
--latest-waive-list=waives_TOT_${targetBranchTOTCommit}.txt \
361+
--diff-file=diff_content.txt \
362+
--output-file=waives.txt
363+
"""
364+
trtllm_utils.uploadArtifacts("waives*.txt", "${UPLOAD_PATH}/waive_list/")
365+
echo "New merged test waive list: https://urm.nvidia.com/artifactory/${UPLOAD_PATH}/waive_list/waives.txt"
366+
} catch (InterruptedException e) {
367+
throw e
368+
} catch (Exception e) {
369+
echo "Merge waive list failed. Error: ${e.toString()}"
344370
}
345-
targetBranchTOTCommit = sh (script: "cd ${LLM_TOT_ROOT} && git rev-parse HEAD", returnStdout: true).trim()
346-
echo "Target branch TOT commit: ${targetBranchTOTCommit}"
347-
sh "cp ${LLM_TOT_ROOT}/tests/integration/test_lists/waives.txt ./waives_TOT_${targetBranchTOTCommit}.txt"
348-
349-
// Get waive list diff in current MR
350-
def diff = getMergeRequestOneFileChanges(pipeline, globalVars, "tests/integration/test_lists/waives.txt")
351-
352-
// Write diff to a temporary file to avoid shell escaping issues
353-
writeFile file: 'diff_content.txt', text: diff
354-
355-
// Merge waive lists
356-
sh """
357-
python3 mergeWaiveList.py \
358-
--cur-waive-list=waives_CUR_${env.gitlabCommit}.txt \
359-
--latest-waive-list=waives_TOT_${targetBranchTOTCommit}.txt \
360-
--diff-file=diff_content.txt \
361-
--output-file=waives.txt
362-
"""
363-
trtllm_utils.uploadArtifacts("waives*.txt", "${UPLOAD_PATH}/waive_list/")
364-
echo "New merged test waive list: https://urm.nvidia.com/artifactory/${UPLOAD_PATH}/waive_list/waives.txt"
365371
}
366372

367373
def preparation(pipeline, testFilter, globalVars)

0 commit comments

Comments
 (0)