Skip to content

Commit 2be6f90

Browse files
authored
ci: Fix error-masking bug and improve debugability in L0_trace (#7930)
1 parent b84cc26 commit 2be6f90

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

qa/L0_trace/test.sh

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
#!/bin/bash
2-
# Copyright 2022-2024, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
2+
# Copyright 2022-2025, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
33
#
44
# Redistribution and use in source and binary forms, with or without
55
# modification, are permitted provided that the following conditions
@@ -220,6 +220,7 @@ fi
220220
if [ `grep -c "\"trace_mode\":\"triton\"" ./curl.out` != "1" ]; then
221221
RET=1
222222
fi
223+
cat ./curl.out
223224

224225
send_inference_requests "client_min.log" 10
225226

@@ -267,9 +268,7 @@ assert_curl_failure "trace_file updated through network protocol expects an erro
267268
if [ `grep -c "\"error\":\"trace file location can not be updated through network protocol\"" ./curl.out` != "1" ]; then
268269
RET=1
269270
fi
270-
if [ `grep -c "\"trace_mode\":\"triton\"" ./curl.out` != "1" ]; then
271-
RET=1
272-
fi
271+
cat ./curl.out
273272

274273
# Use a different name
275274
update_trace_setting "simple" '{"log_frequency":"2"}'
@@ -294,6 +293,7 @@ fi
294293
if [ `grep -c "\"trace_mode\":\"triton\"" ./curl.out` != "1" ]; then
295294
RET=1
296295
fi
296+
cat ./curl.out
297297

298298
send_inference_requests "client_simple.log" 10
299299

@@ -377,6 +377,7 @@ fi
377377
if [ `grep -c "\"trace_mode\":\"triton\"" ./curl.out` != "1" ]; then
378378
RET=1
379379
fi
380+
cat ./curl.out
380381

381382
# Send requests to simple where trace is explicitly disabled
382383
send_inference_requests "client_update.log" 10
@@ -407,6 +408,7 @@ fi
407408
if [ `grep -c "\"trace_mode\":\"triton\"" ./curl.out` != "1" ]; then
408409
RET=1
409410
fi
411+
cat ./curl.out
410412

411413
# Send requests to simple where now uses global setting
412414
send_inference_requests "client_clear.log" 5
@@ -478,6 +480,7 @@ fi
478480
if [ `grep -c "\"trace_mode\":\"triton\"" ./curl.out` != "1" ]; then
479481
RET=1
480482
fi
483+
cat ./curl.out
481484

482485
# Set trace count
483486
update_global_trace_setting '{"trace_count":"5"}'
@@ -502,6 +505,7 @@ fi
502505
if [ `grep -c "\"trace_mode\":\"triton\"" ./curl.out` != "1" ]; then
503506
RET=1
504507
fi
508+
cat ./curl.out
505509

506510
# Send requests to simple where trace is explicitly disabled
507511
send_inference_requests "client_update.log" 10
@@ -528,6 +532,7 @@ fi
528532
if [ `grep -c "\"trace_mode\":\"triton\"" ./curl.out` != "1" ]; then
529533
RET=1
530534
fi
535+
cat ./curl.out
531536

532537
# Check if the indexed file has been generated when trace count reaches 0
533538
if [ ! -f ./global_count.log.0 ]; then
@@ -592,8 +597,6 @@ if [ "$SERVER_PID" == "0" ]; then
592597
exit 1
593598
fi
594599

595-
RET=0
596-
597600
set +e
598601

599602
python $CLIENT_TEST >>$CLIENT_LOG 2>&1
@@ -647,6 +650,7 @@ fi
647650
if [ `grep -c "\"trace_mode\":\"triton\"" ./curl.out` != "1" ]; then
648651
RET=1
649652
fi
653+
cat ./curl.out
650654

651655
set +e
652656
# Send bls requests to make sure simple model is traced
@@ -941,6 +945,7 @@ fi
941945
if [ `grep -c "\"log_frequency\":" ./curl.out` != "0" ]; then
942946
RET=1
943947
fi
948+
cat ./curl.out
944949

945950

946951
set +e
@@ -1026,6 +1031,7 @@ fi
10261031
if [ `grep -c "\"log_frequency\":" ./curl.out` != "0" ]; then
10271032
RET=1
10281033
fi
1034+
cat ./curl.out
10291035

10301036
set +e
10311037
# Send bls requests to make sure bls_simple model is NOT traced

0 commit comments

Comments
 (0)