Skip to content

[CI] Rename tests/ce to tests/ci_validation to avoid CE naming confusion#7733

Merged
EmmonsCurse merged 2 commits intoPaddlePaddle:developfrom
EmmonsCurse:rename_ce_to_ci_validation
May 7, 2026
Merged

[CI] Rename tests/ce to tests/ci_validation to avoid CE naming confusion#7733
EmmonsCurse merged 2 commits intoPaddlePaddle:developfrom
EmmonsCurse:rename_ce_to_ci_validation

Conversation

@EmmonsCurse
Copy link
Copy Markdown
Collaborator

Motivation

The FastDeploy/tests/ce path is misleading during CI execution, as some developers mistakenly interpret it as CE-related execution rather than CI validation logic.

To reduce ambiguity and improve maintainability, the directory naming should better reflect its actual purpose in the CI workflow.

Modifications

  • Renamed FastDeploy/tests/ce to FastDeploy/tests/ci_validation.
  • Deleted useless dir tests/ce/deterministic.
  • Updated related references and paths accordingly.
  • Improved clarity of CI-related test organization and reduced confusion around CE naming.

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 ci_iluvatar
/skip-ci ci_hpu
/skip-ci build_xpu

@paddle-bot
Copy link
Copy Markdown

paddle-bot Bot commented May 7, 2026

Thanks for your contribution!

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (develop@78b5462). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #7733   +/-   ##
==========================================
  Coverage           ?   71.61%           
==========================================
  Files              ?      396           
  Lines              ?    55569           
  Branches           ?     8688           
==========================================
  Hits               ?    39793           
  Misses             ?    13038           
  Partials           ?     2738           
Flag Coverage Δ
GPU 71.61% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown

@PaddlePaddle-bot PaddlePaddle-bot left a comment

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-05-07 16:55:12

📋 Review 摘要

PR 概述:将 tests/ce 目录重命名为 tests/ci_validation,并删除 tests/ce/deterministic/ 下的测试脚本
变更范围:CI workflow 配置、文档目录说明、代码风格检查脚本
影响面 Tag[CI] [Docs]

📝 PR 规范检查

PR 标题包含合规 [CI] Tag,描述结构完整(Motivation / Modifications / Usage or Command / Accuracy Tests / Checklist 均存在且有效填写),规范合规。

问题

级别 文件 概述
❓ 疑问 tests/ce/deterministic/test_determinism_verification.py 删除 470 行确定性验证测试,PR 描述称其为"无用目录",是否已确认无需迁移至 tests/ci_validation/ 或其他测试目录?

总体评价

纯目录重命名 PR,CI workflow / 文档 / pytest 配置 / pre-commit 脚本均已同步更新,整体完整。唯一疑问是被删除的 tests/ce/deterministic/ 包含一个功能完整的确定性验证测试(470 行),建议作者在 PR 描述中说明该测试已废弃或已有其他测试覆盖,便于后续维护者了解背景。

@PaddlePaddle-bot
Copy link
Copy Markdown

🤖 Paddle-CI-Agent | ci_status_monitor | 2026-05-07 17:31:43

CI报告基于以下代码生成(30分钟更新一次):


1 任务总览

所有 Required 任务均已通过 ✅,建议合并。

总执行(rerun次数) 总任务 ✅ 通过 ❌ 失败 ⏳ 运行中 ⏸️ 等待中 跳过
36(0) 36 31 0 0 0 5

2 任务状态汇总

2.1 Required任务 : 8/10 通过

必选任务阻塞合并,失败需优先处理。

状态 任务 耗时 根因 修复建议 日志 重跑
其余 8 个必选任务通过 - - - - -

注:另有 2 个 Required 任务处于跳过(skipped)状态,不影响合并判定。

2.2 可选任务 — 23/26 通过

可选任务不阻塞合并,失败仅供参考。

状态 任务 耗时 日志 重跑
其余 23 个可选任务通过 - - -
⏭️ 5 个可选任务已跳过(CI_HPU、XPU 相关等) - - -

3 失败详情(仅 required)

无 required 失败任务。

@EmmonsCurse EmmonsCurse merged commit 24d5213 into PaddlePaddle:develop May 7, 2026
37 checks passed
@EmmonsCurse EmmonsCurse deleted the rename_ce_to_ci_validation branch May 7, 2026 12:46
sunlei1024 pushed a commit to sunlei1024/FastDeploy that referenced this pull request May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants