Skip to content

Commit 13f9ec8

Browse files
committed
fix
Signed-off-by: Yiqing Yan <[email protected]>
1 parent fe9bc47 commit 13f9ec8

File tree

2 files changed

+15
-12
lines changed

2 files changed

+15
-12
lines changed

jenkins/L0_Test.groovy

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ LLM_ROOT = "llm"
2525
ARTIFACT_PATH = env.artifactPath ? env.artifactPath : "sw-tensorrt-generic/llm-artifacts/${JOB_NAME}/${BUILD_NUMBER}"
2626
UPLOAD_PATH = env.uploadPath ? env.uploadPath : "sw-tensorrt-generic/llm-artifacts/${JOB_NAME}/${BUILD_NUMBER}"
2727

28-
REUSE_ARITIFACT_PATH = env.reuseArtifactPath
28+
REUSE_ARTIFACT_PATH = env.reuseArtifactPath
2929

3030
X86_64_TRIPLE = "x86_64-linux-gnu"
3131
AARCH64_TRIPLE = "aarch64-linux-gnu"
@@ -352,8 +352,8 @@ def runLLMTestlistOnSlurm_MultiNodes(pipeline, platform, testList, config=VANILL
352352
def makoOptsJson = transformMakoArgsToJson(["Mako options:"] + makoArgs)
353353
def testListPath = renderTestDB(testList, llmSrcLocal, stageName, makoOptsJson)
354354
// Reuse passed tests
355-
if (REUSE_ARITIFACT_PATH) {
356-
reusePassedTests(REUSE_ARITIFACT_PATH, stageName, testListPath)
355+
if (REUSE_ARTIFACT_PATH) {
356+
reusePassedTests(pipeline, llmSrcLocal, REUSE_ARTIFACT_PATH, stageName, testListPath)
357357
}
358358
Utils.exec(pipeline, script: "sshpass -p '${remote.passwd}' scp -r -p -oStrictHostKeyChecking=no ${testListPath} ${remote.user}@${remote.host}:${testListPathNode}",)
359359

@@ -1069,13 +1069,14 @@ def renderTestDB(testContext, llmSrc, stageName, preDefinedMakoOpts=null) {
10691069
return testList
10701070
}
10711071

1072-
def reusePassedTests(reusedArtifactPath, stageName, testListFile) {
1072+
def reusePassedTests(pipeline, llmSrc, reusedArtifactPath, stageName, testListFile) {
10731073
def reusedPath = "${WORKSPACE}/reused"
10741074
sh "mkdir -p ${reusedPath}"
10751075
def resultsFileName = "results-${stageName}"
10761076
def passedTestsFile = "${reusedPath}/${stageName}/passed_tests.txt"
10771077
try {
1078-
trtllm_utils.llmExecStepWithRetry(pipeline, script: "cd ${reusedPath} && wget -nv ${reusedArtifactPath}/test-results/${resultsFileName}.tar.gz")
1078+
def resultsUrl = "https://urm.nvidia.com/artifactory/${reusedArtifactPath}/test-results/${resultsFileName}.tar.gz"
1079+
trtllm_utils.llmExecStepWithRetry(pipeline, script: "cd ${reusedPath} && wget -nv ${resultsUrl}")
10791080
sh "cd ${reusedPath} && tar -zxf ${resultsFileName}.tar.gz"
10801081
// Get passed tests
10811082
sh """
@@ -1450,8 +1451,8 @@ def runLLMTestlistOnPlatformImpl(pipeline, platform, testList, config=VANILLA_CO
14501451

14511452
def testDBList = renderTestDB(testList, llmSrc, stageName)
14521453
// Reuse passed tests
1453-
if (REUSE_ARITIFACT_PATH) {
1454-
reusePassedTests(REUSE_ARITIFACT_PATH, stageName, testDBList)
1454+
if (REUSE_ARTIFACT_PATH) {
1455+
reusePassedTests(pipeline, llmSrc, REUSE_ARTIFACT_PATH, stageName, testDBList)
14551456
}
14561457

14571458
testList = "${testList}_${splitId}"

jenkins/scripts/delete_passed_tests.py renamed to jenkins/scripts/reuse_passed_tests.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
1+
import argparse
12
import os
23
import sys
3-
import argparse
44
import xml.etree.ElementTree as ET
5-
from test_rerun import parse_name
5+
6+
import test_rerun
7+
68

79
def get_passed_tests(input_file, output_file):
810
if not os.path.exists(input_file):
@@ -25,8 +27,8 @@ def get_passed_tests(input_file, output_file):
2527
classname = testcase.attrib.get('classname', '')
2628
name = testcase.attrib.get('name', '')
2729
filename = testcase.attrib.get('file', '')
28-
testName = parse_name(classname, name, filename)
29-
passed_tests.append(testName)
30+
test_name = test_rerun.parse_name(classname, name, filename)
31+
passed_tests.append(test_name)
3032
except Exception as e:
3133
print(f"Failed to parse {input_file}: {e}")
3234
return
@@ -89,4 +91,4 @@ def remove_passed_tests(input_file, passed_tests_file):
8991
required=True,
9092
help='File containing passed tests')
9193
args = parser.parse_args(sys.argv[2:])
92-
remove_passed_tests(args.input_file, args.passed_tests_file)
94+
remove_passed_tests(args.input_file, args.passed_tests_file)

0 commit comments

Comments
 (0)