fix review suggestions

Signed-off-by: MengqingCao <cmq0113@163.com>

Author: MengqingCao

.github/workflows/vllm_ascend_test_pd.yaml

@@ -32,7 +32,7 @@ defaults:
 
 jobs:
   test:
-    if: ${{ github.event.label.name == 'module:pd' }}
+    if: github.event_name == 'schedule' || github.event.label.name == 'module:pd'
     strategy:
       matrix:
         vllm_verison: [v0.8.5.post1]
@@ -96,6 +96,5 @@ jobs:
           pip install -v -e .
 
       - name: Run vllm-project/vllm-ascend PD Disaggregation test
-        if: github.event_name == 'schedule' || github.event.label.name == 'module:pd'
         run: |
           pytest -sv tests/e2e/pd_disaggreate/test_pd_e2e.py

tests/e2e/pd_disaggreate/setup_pd.sh

@@ -17,15 +17,6 @@
 # This file is a part of the vllm-ascend project.
 #
 
-
-function cleanup_instances() {
-  VLLM_PID=$(pgrep -f "vllm serve")
-  _info "===> Try kill -2 ${VLLM_PID} to exit."
-  kill -2 "$VLLM_PID"
-  wait_for_exit "$VLLM_PID"
-}
-
-
 function run_prefill_instance() {
   local model_name=$1
   local tp_size=$2
@@ -140,7 +131,4 @@ function run_proxy_server() {
   # Start the proxy server
   echo "Starting proxy server with command: $PROXY_CMD"
   $PROXY_CMD &
-
-  # Wait for the proxy to start
-  sleep 3
 }

tests/e2e/pd_disaggreate/test_pd_e2e.py

@@ -1,5 +1,3 @@
-#!/bin/bash
-
 #
 # Copyright (c) 2025 Huawei Technologies Co., Ltd. All Rights Reserved.
 #
@@ -75,8 +73,8 @@ def start_and_test_pipeline():
     proc = subprocess.Popen(["bash", SCRIPT_PATH])
     try:
         print("Waiting for proxy port to be available...")
-        wait_for_port(PROXY_PORT, 1200)
-        wait_for_port(DECODE_PORT, 1200)
+        wait_for_port(PROXY_PORT, 180)
+        wait_for_port(DECODE_PORT, 180)
 
         # request
         payload = {