Skip to content

Conversation

@SigureMo
Copy link
Member

@SigureMo SigureMo commented Jan 8, 2026

PR Category

Execute Infrastructure

PR Types

Bug fixes

Description

修复 #77174 后在 make 后添加 ccache -s 命令导致编译退出码无法正确传递的问题,这会导致 H-Coverage 编译失败无法正确退出

Copilot AI review requested due to automatic review settings January 8, 2026 02:55
@paddle-bot
Copy link

paddle-bot bot commented Jan 8, 2026

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a bug in the H-Coverage CI workflow where the exit code from the make command was not properly propagated after adding a ccache -s command. The fix ensures that compilation failures are correctly detected and cause the workflow to fail as expected.

Key Changes

  • Captures the exit code from make -j20 before running ccache -s
  • Explicitly exits with the captured exit code to ensure proper failure detection

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SigureMo SigureMo merged commit 1978096 into PaddlePaddle:develop Jan 8, 2026
106 of 107 checks passed
@SigureMo SigureMo deleted the ci/ensure-preper-exit-code-handling-in-coverage-build-step branch January 8, 2026 07:50
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