Closed yanfeng1992 closed 1 month ago
Thanks @yanfeng1992, I've actually had a partial cleanup before. /assign
A ut has been failed, can you help fix it?
Attention: Patch coverage is 14.70588%
with 29 lines
in your changes are missing coverage. Please review.
Project coverage is 51.75%. Comparing base (
a0c0a98
) to head (7c2b3e1
).
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Thanks @yanfeng1992, I've actually had a partial cleanup before. /assign
A ut has been failed, can you help fix it?
already fixed it @XiShanYongYe-Chang
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: XiShanYongYe-Chang
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Signed-off-by: huangyanfeng huangyanfeng1992@gmail.com
What type of PR is this? /kind cleanup
What this PR does / why we need it:
The Controller will requeue the Request to be processed again if an error is non-nil or Result.Requeue is true, an error is non-nil has higher priority. Therefore, there is no need to set Result.Requeue to true when the error is non-nil.
Which issue(s) this PR fixes: Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: