Skip to content

test pr prehook - #79577

Open
Ray961123 wants to merge 7 commits into
PaddlePaddle:developfrom
Ray961123:test-prehook
Open

test pr prehook#79577
Ray961123 wants to merge 7 commits into
PaddlePaddle:developfrom
Ray961123:test-prehook

Conversation

@Ray961123

Copy link
Copy Markdown
Collaborator

PR Category

Environment Adaptation

PR Types

Security

Description

pr提交测试

是否引起精度变化

@risemeup1111 risemeup1111 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

序号 位置 优先级 状态
1 PR #79577(非行级:标题/描述过于泛化) P3 🚧
2 .github/workflows/_Mac.yml:62(新 runner 缺少 workflow 依赖的下载工具) P0
  1. 请将标题和 Description 改为能说明 Mac/Windows runner group 迁移的目的、影响范围和验证方式;当前标题和描述无法支撑审查与后续追溯。
点击展开 Review 规则

PR 评审规则:

  • P0、P1 级别的评审必须提交新的 commit 进行修改;
  • P2、P3 级别的评审可以通过评论进行修改。

状态说明:

已解决 无需修复 待修复
🟡 🚧
Powered by Nyanpasu with gpt-5.5 xhigh, please check the suggestions carefully.

@Ray961123

Copy link
Copy Markdown
Collaborator Author

/re-run all-failed

needs: [check-bypass]
runs-on:
group: Mac-CI
group: mac-test

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P0 切换到这个 runner group 后,Mac CI 无法通过首个环境检查:本 PR 的 Mac-CPU / Build and test 已由 mac-test 的 runner 接管,但 workflow 在调用现有的 wget 下载步骤时报告 wget: command not found 并以 127 退出,后续构建和测试均被跳过。

请先让 mac-test 提供该 workflow 依赖的下载工具,或将下载步骤改为该组已预置的工具,并在当前 head 上验证 Mac 作业能够进入构建阶段。

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

当前 head 的 Mac-CPU / Build and test 已由 mac-test/mac-m4-4 接管,Check environment 成功完成并进入构建阶段,原 wget 阻断已消失。本问题已解决。

@Ray961123

Copy link
Copy Markdown
Collaborator Author

/re-run

@Ray961123

Copy link
Copy Markdown
Collaborator Author

/re-run CI-Windows

@Ray961123

Copy link
Copy Markdown
Collaborator Author

/re-run all-failed

4 similar comments
@Ray961123

Copy link
Copy Markdown
Collaborator Author

/re-run all-failed

@Ray961123

Copy link
Copy Markdown
Collaborator Author

/re-run all-failed

@Ray961123

Copy link
Copy Markdown
Collaborator Author

/re-run all-failed

@Ray961123

Copy link
Copy Markdown
Collaborator Author

/re-run all-failed

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.

2 participants