Skip to content

Commit 7d38c7a

Browse files
committed
revert: drop AWS/Azure workflow comment-only edits from #1644
Those files do not change serve behavior. #1644 is the GKE inference cell; AWS/Azure steps stay disabled as on main. Signed-off-by: Rohit Rajani <rorajani@nvidia.com>
1 parent 73f23ba commit 7d38c7a

2 files changed

Lines changed: 24 additions & 18 deletions

File tree

.github/workflows/uat-aws.yaml

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -813,15 +813,18 @@ jobs:
813813
RUN_ID: ${{ github.run_id }}
814814
run: ./tests/uat/aws/run train "${TEST_CONFIG}"
815815

816-
# STILL DISABLED (step commented out below). phase_serve already selects the
817-
# GPU pool for both graph components (#1644), so the Frontend no longer
818-
# cold-pulls the ~12GB vllm-runtime image on a small CPU-pool node. Only
819-
# uat-gcp.yaml re-enables the serve *step* for now (GKE x inference is what
820-
# #1644 tracks); re-enable this one after that lane has a served completion,
821-
# and restore the Serve summary row below to `${{ steps.serve.outcome }}`.
822-
# Until then the inference STACK (dynamo + KAI scheduler + DRA driver) is
823-
# still stood up and validated by prep/install/conformance/verify; only the
824-
# served-workload run is skipped (Serve summary row is a static `disabled`).
816+
# TEMPORARILY DISABLED (step commented out below). The vLLM workload run is
817+
# excluded from UAT for now because the Frontend component pulls the full
818+
# ~12GB vllm-runtime image on a fresh (non-GPU) node and never finishes
819+
# within the 30m readiness budget: the decode worker reuses the GPU node's
820+
# cached image (`already present on machine`), but the Frontend lands on a
821+
# different node and pulls from scratch, wedging in ContainerCreating until
822+
# the phase times out. The inference STACK (dynamo + KAI scheduler + DRA
823+
# driver) is still stood up and validated by prep/install/conformance/verify;
824+
# only the served-workload run is skipped (the Serve summary row is a static
825+
# `disabled`). Re-enable by uncommenting the step once the pull is addressed
826+
# (pre-pull/image cache, or pin the Frontend to the GPU node so it reuses
827+
# the cached image). Kept in lockstep with uat-gcp.yaml. Tracked by #1644.
825828
#
826829
# - name: UAT - serve (DynamoGraphDeployment + endpoint)
827830
# id: serve

.github/workflows/uat-azure.yaml

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -797,15 +797,18 @@ jobs:
797797
RUN_ID: ${{ github.run_id }}
798798
run: ./tests/uat/azure/run train "${TEST_CONFIG}"
799799

800-
# STILL DISABLED (step commented out below). phase_serve already selects the
801-
# GPU pool for both graph components (#1644), so the Frontend no longer
802-
# cold-pulls the ~12GB vllm-runtime image on a small CPU-pool node. Only
803-
# uat-gcp.yaml re-enables the serve *step* for now (GKE x inference is what
804-
# #1644 tracks); re-enable this one after that lane has a served completion,
805-
# and restore the Serve summary row below to `${{ steps.serve.outcome }}`.
806-
# Until then the inference STACK (dynamo + KAI scheduler + DRA driver) is
807-
# still stood up and validated by prep/install/conformance/verify; only the
808-
# served-workload run is skipped (Serve summary row is a static `disabled`).
800+
# TEMPORARILY DISABLED (step commented out below). The vLLM workload run is
801+
# excluded from UAT for now because the Frontend component pulls the full
802+
# ~12GB vllm-runtime image on a fresh (non-GPU) node and never finishes
803+
# within the 30m readiness budget: the decode worker reuses the GPU node's
804+
# cached image (`already present on machine`), but the Frontend lands on a
805+
# different node and pulls from scratch, wedging in ContainerCreating until
806+
# the phase times out. The inference STACK (dynamo + KAI scheduler + DRA
807+
# driver) is still stood up and validated by prep/install/conformance/verify;
808+
# only the served-workload run is skipped (the Serve summary row is a static
809+
# `disabled`). Re-enable by uncommenting the step once the pull is addressed
810+
# (pre-pull/image cache, or pin the Frontend to the GPU node so it reuses
811+
# the cached image).
809812
#
810813
# - name: UAT - serve (DynamoGraphDeployment + endpoint)
811814
# id: serve

0 commit comments

Comments
 (0)