Skip to content

[CI] Remove deprecated IPC unit tests - #7974

Merged
EmmonsCurse merged 1 commit into
PaddlePaddle:developfrom
EmmonsCurse:remove_useless_ipc_unittest
Jun 2, 2026
Merged

[CI] Remove deprecated IPC unit tests#7974
EmmonsCurse merged 1 commit into
PaddlePaddle:developfrom
EmmonsCurse:remove_useless_ipc_unittest

Conversation

@EmmonsCurse

Copy link
Copy Markdown
Collaborator

Motivation

The IPC-related unit tests are no longer applicable and have been deprecated. Keeping these tests in CI introduces unnecessary maintenance overhead and execution time without providing meaningful validation coverage.

Modifications

  • Removed deprecated IPC-related unit tests.
  • Cleaned up obsolete test coverage associated with IPC functionality.
  • Reduced unnecessary CI execution workload and maintenance cost.

Usage or Command

N/A

Accuracy Tests

N/A

Checklist

  • Add at least a tag in the PR title.
    • Tag list: [[FDConfig],[APIServer],[Engine], [Scheduler], [PD Disaggregation], [Executor], [Graph Optimization], [Speculative Decoding], [RL], [Models], [Quantization], [Loader], [OP], [KVCache], [DataProcessor], [BugFix], [Docs], [CI], [Optimization], [Feature], [Benchmark], [Others], [XPU], [HPU], [GCU], [DCU], [Iluvatar], [Metax]]
    • You can add new tags based on the PR content, but the semantics must be clear.
  • Format your code, run pre-commit before commit.
  • Add unit tests. Please write the reason in this PR if no unit tests.
  • Provide accuracy results.
  • If the current PR is submitting to the release branch, make sure the PR has been submitted to the develop branch, then cherry-pick it to the release branch with the [Cherry-Pick] PR tag.

@EmmonsCurse

Copy link
Copy Markdown
Collaborator Author

/skip-ci all

@EmmonsCurse
EmmonsCurse requested a review from juncaipeng June 2, 2026 07:16
@EmmonsCurse
EmmonsCurse merged commit d778309 into PaddlePaddle:develop Jun 2, 2026
43 checks passed
@EmmonsCurse
EmmonsCurse deleted the remove_useless_ipc_unittest branch June 2, 2026 07:19

@PaddlePaddle-bot PaddlePaddle-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Paddle-CI-Agent | pr_review | 2026-06-02 15:23:15

📋 Review 摘要

PR 概述:在 CI 覆盖率测试配置中忽略已废弃的 IPC 相关 e2e 测试
变更范围:tests/cov_pytest.ini
影响面 Tag[CI]

问题

未发现阻塞性问题。

📝 PR 规范检查

✓ 标题格式合规([CI] Tag 正确),描述结构完整,包含所有必填段落。

轻微备注:PR 描述中 Modifications 提到 "Removed deprecated IPC-related unit tests",实际变更仅是在 cov_pytest.ini 中添加 --ignore 指令跳过这些测试,测试文件本身并未从仓库中删除。描述措辞可更精确为"在覆盖率测试中排除了废弃的 IPC 测试",但不影响合入。

总体评价

变更简单且风险极低,仅在 pytest 覆盖率配置中排除两个已废弃的 IPC/splitwise 相关 e2e 测试文件。两个测试文件经确认均为 IPC 传输缓存相关的 splitwise 部署测试,排除合理。

@EmmonsCurse

Copy link
Copy Markdown
Collaborator Author

✅ Cherry-pick successful! Created PR: #8029

@paddle-bot

paddle-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

Thanks for your contribution!

@paddle-bot paddle-bot Bot added the contributor External developers label Jul 13, 2026
@EmmonsCurse

Copy link
Copy Markdown
Collaborator Author

❌ Cherry-pick failed: Conflicts detected when cherry-picking to release/2.6. Please resolve manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants