[Benchmark] Fix error msg upload in performance benchmark (#1559)
### What this PR does / why we need it? Make sure that None parameters are not passed in for `--error` ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? CI passed locally Signed-off-by: wangli <wangli858794774@gmail.com>
This commit is contained in:
2
.github/workflows/nightly_benchmarks.yaml
vendored
2
.github/workflows/nightly_benchmarks.yaml
vendored
@@ -198,7 +198,7 @@ jobs:
|
|||||||
--commit_title "$commit_title" \
|
--commit_title "$commit_title" \
|
||||||
--created_at "$commit_time_no_tz" \
|
--created_at "$commit_time_no_tz" \
|
||||||
--res_dir ./benchmarks/results \
|
--res_dir ./benchmarks/results \
|
||||||
--error $ERROR_MSG \
|
--error "$ERROR_MSG" \
|
||||||
--extra_feat '{"VLLM_USE_V1": "${{ matrix.vllm_use_v1 }}"}'
|
--extra_feat '{"VLLM_USE_V1": "${{ matrix.vllm_use_v1 }}"}'
|
||||||
rm -rf ./benchmarks/results
|
rm -rf ./benchmarks/results
|
||||||
cd -
|
cd -
|
||||||
|
|||||||
Reference in New Issue
Block a user