Skip to content

Commit f0daaf8

Browse files
authored
[CI][BugFix]Change failure analysis secret keys names in pr_test.yaml (vllm-project#13067)
Updated secret keys for failure analysis in the workflow. ### What this PR does / why we need it? Update the name of obs secrets in pr_test.yaml to fix the problem : unable to upload Update analyze_failure comments ### Does this PR introduce _any_ user-facing change? N/A ### How was this patch tested? - vLLM version: v0.25.1 - vLLM main: vllm-project/vllm@d02df74 --------- Signed-off-by: Xuyzhen <958522639@qq.com>
1 parent 9000fa1 commit f0daaf8

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

.github/workflows/pr_test.yaml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -487,7 +487,9 @@ jobs:
487487
488488
echo "=== CI Gate: PASSED ==="
489489
490-
#this job collects failure from uploaded logs
490+
# this job collects failure from uploaded logs
491+
# failure analyze & upload reports
492+
491493
analyze-failure-report:
492494
name: Analyze failure report
493495
needs: [run-selected-tests, recommend-tests-from-coverage]
@@ -498,5 +500,5 @@ jobs:
498500
recommendations_file: .github/workflows/scripts/recommended_pytest_paths.txt
499501
recommendations_content: ${{ needs.recommend-tests-from-coverage.outputs.coverage_paths || '' }}
500502
secrets:
501-
OBS_ACCESS_KEY_FAILURE_ANALYSIS: ${{ secrets.OBS_ACCESS_KEY_FAILURE_ANALYSIS }}
502-
OBS_SECRET_ACCESS_KEY_FAILURE_ANALYSIS: ${{ secrets.OBS_SECRET_ACCESS_KEY_FAILURE_ANALYSIS }}
503+
OBS_ACCESS_KEY_FAILURE_ANALYSIS: ${{ secrets.OBS_ACCESS_KEY_PRECISION }}
504+
OBS_SECRET_ACCESS_KEY_FAILURE_ANALYSIS: ${{ secrets.OBS_SECRET_ACCESS_KEY_PRECISION }}

0 commit comments

Comments
 (0)