Revert "[XPU][CI] xpu add ci test for pd"#5645
Merged
plusNew001 merged 1 commit intodevelopfrom Dec 18, 2025
Merged
Conversation
This reverts commit 80fb530.
|
Thanks for your contribution! |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR reverts a previous change (#5610) by replacing a larger-scale PD separation test with a simpler one. The changes swap out a test for a 300B model with TP4 configuration for a test using a smaller 0.3B model with TP1 configuration, and removes some configuration code from the shared test utilities.
Key changes:
- Replaces test for ERNIE-4.5-300B-A47B model (with wint4 quantization, TP4) with ERNIE-4.5-0.3B model (TP1)
- Removes 5-second stabilization sleep after health check in
start_server - Removes dynamic RDMA NIC configuration from
setup_ep_envfunction
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| tests/xpu_ci/test_pd_separation.py | Adds new PD separation test for smaller 0.3B model with TP1 configuration |
| tests/xpu_ci/test_pd_300b_tp4.py | Removes PD separation test for larger 300B model with TP4 configuration |
| tests/xpu_ci/conftest.py | Removes post-health-check sleep and dynamic RDMA setup from EP environment configuration |
EmmonsCurse
approved these changes
Dec 18, 2025
chang-wenbin
pushed a commit
to chang-wenbin/FastDeploy
that referenced
this pull request
Mar 2, 2026
…Paddle#5645) This reverts commit 80fb530.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reverts #5610