[Metax][CI] fix ci bugs#7929
Merged
Merged
Conversation
Removed cleanup step from CI workflow.
|
Thanks for your contribution! |
Collaborator
Author
|
/skip-ci all |
Contributor
There was a problem hiding this comment.
Pull request overview
该 PR 聚焦于修复/稳定 Metax 相关 CI:调整 CI_METAX 工作流,避免在触发 Jenkins 任务前执行高风险/易失败的工作区清理操作,从而降低自托管 runner 上的 CI 失败概率。
Changes:
- 移除
.github/workflows/ci_metax.yml中的Cleanup步骤(rm -rf * .[^.]*)。 - 保持其余流程不变:在自托管
XPU-P800runner 上通过MetaX-MACA/simple-jenkins-githubaction@v1.2触发 Jenkins 的paddle_fastdeploy_metax_smoketest任务。
备注:PR 描述的 Motivation/Modifications/Usage/Tests 目前未填写,建议补充该变更要解决的具体 CI 报错现象与复现/验证方式,方便后续维护与回溯。
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 Paddle-CI-Agent | pr_review |
2026-05-26 17:28:12
📋 Review 摘要
PR 概述:移除 Metax CI workflow 中的 Cleanup 步骤以修复 CI 问题
变更范围:.github/workflows/ci_metax.yml
影响面 Tag:CI Metax
问题
| 级别 | 文件 | 概述 |
|---|---|---|
| 📝 PR 规范 | PR 描述 | Motivation 和 Modifications 章节内容为空,仅保留了 HTML 注释占位符 |
📝 PR 规范检查
PR 标题格式符合规范(含两个官方 Tag [Metax] 和 [CI])。PR 描述中 ## Motivation 和 ## Modifications 章节内容为空(仅有 HTML 注释占位符),需补充实际内容。
标题建议(可直接复制):
[Metax][CI] fix ci bugs✓ 已符合规范,无需修改
PR 描述建议(可直接复制,必须复刻 checklist §D2 模板的完整结构):
## Motivation
移除 `.github/workflows/ci_metax.yml` 中过于激进的 Cleanup 步骤(`rm -rf * .[^.]*`),该步骤会在触发 Jenkins job 前删除工作目录下所有文件(含隐藏文件),可能导致 GitHub Actions runner 运行环境异常,从而引发 CI 失败。
## Modifications
- 删除 `.github/workflows/ci_metax.yml` 中的 `Cleanup` 步骤,该步骤执行 `rm -rf * .[^.]*` 清理工作目录
## Usage or Command
N/A
## Accuracy Tests
N/A
## Checklist
- [x] 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.
- [x] Format your code, run `pre-commit` before commit.
- [x] 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.总体评价
变更内容简洁明确,删除有潜在风险的 Cleanup 步骤是合理的修复。建议补充 PR 描述中的 Motivation 和 Modifications 内容。
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.
Motivation
Modifications
Usage or Command
Accuracy Tests
Checklist
[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]]pre-commitbefore commit.releasebranch, make sure the PR has been submitted to thedevelopbranch, then cherry-pick it to thereleasebranch with the[Cherry-Pick]PR tag.