Skip to content

Commit b811339

Browse files
committed
Make CI happy
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
1 parent d2d2c86 commit b811339

File tree

1 file changed

+22
-8
lines changed

1 file changed

+22
-8
lines changed

.github/workflows/vllm_ascend_doctest.yaml

Lines changed: 22 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,15 +15,20 @@
1515
# This file is a part of the vllm-ascend project.
1616
#
1717

18-
name: 'e2e test'
18+
name: 'Doctest'
1919

2020
on:
2121
pull_request:
2222
branches:
2323
- 'main'
2424
- '*-dev'
2525
paths:
26+
# If we are changing the doctest we should do a PR test
2627
- '.github/workflows/vllm_ascend_doctest.yaml'
28+
- 'tests/doctest/**'
29+
schedule:
30+
# Runs at 01:00 UTC (8:00 AM Beijing) every day
31+
- cron: '0 0 * * *'
2732

2833
# Bash shells do not use ~/.profile or ~/.bashrc so these shells need to be explicitly
2934
# declared as "shell: bash -el {0}" on steps that need to be properly activated.
@@ -38,15 +43,14 @@ jobs:
3843
# Each version should be tested
3944
fail-fast: false
4045
matrix:
41-
vllm_verison: [main, v0.7.3-dev, v0.8.4rc2, v0.8.4rc1]
46+
vllm_verison: [main, v0.7.3-dev, main-openeuler, v0.7.3-dev-openeuler]
4247
name: vLLM Ascend test
4348
runs-on: linux-arm64-npu-1
4449
container:
4550
image: m.daocloud.io/quay.io/ascend/vllm-ascend:${{ matrix.vllm_verison }}
4651
steps:
47-
- name: Check npu and CANN info
52+
- name: Check NPU and CANN info
4853
run: |
49-
pwd
5054
npu-smi info
5155
cat /usr/local/Ascend/ascend-toolkit/latest/"$(uname -i)"-linux/ascend_toolkit_install.info
5256
@@ -63,11 +67,21 @@ jobs:
6367

6468
- name: Run vllm-project/vllm test for V0 Engine
6569
run: |
66-
pwd
67-
ls -la
70+
# Prnint hash info
71+
cd /workspace/vllm-ascend
72+
git --no-pager log -1
73+
cd /workspace/vllm
74+
git --no-pager log -1
75+
76+
# PWD: /__w/vllm-ascend/vllm-ascend
6877
if [ ! -d /workspace/vllm-ascend/tests/doctest ]; then
6978
echo "Warning: the doctest path doesn't exists, copy now"
7079
cp -r tests/doctest /workspace/vllm-ascend/tests/
7180
fi
72-
cd /workspace
73-
./vllm-ascend/tests/doctest/run_tests.sh
81+
82+
# Simulate container to enter directory
83+
# TODO(yikun): should change `/` to `/workspace` after #725 relsoved:
84+
# [Bug]: Fail to start vllm under /workspace
85+
# https://github.com/vllm-project/vllm-ascend/issues/725
86+
cd /
87+
./workspace/vllm-ascend/tests/doctest/run_tests.sh

0 commit comments

Comments
 (0)