Closed XiShanYongYe-Chang closed 2 weeks ago
:warning: Please install the to ensure uploads and comments are reliably processed by Codecov.
Attention: Patch coverage is 80.64516%
with 6 lines
in your changes missing coverage. Please review.
Project coverage is 29.66%. Comparing base (
51b2e10
) to head (b59761e
).
Files | Patch % | Lines |
---|---|---|
...kg/controllers/status/cluster_status_controller.go | 80.64% | 4 Missing and 2 partials :warning: |
: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.
/retest
/cc @zhzhuang-zju @RainbowMango
/retest
/retest
/retest
/hold
/hold cancel /kind bug
/assign @RainbowMango
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: RainbowMango
The full list of commands accepted by this bot can be found here.
The pull request process is described here
What type of PR is this?
/kind bug /kind failing-test
What this PR does / why we need it:
During the cluster status update, due to the presence of an old Condition in the previous Cluster status, this Condition is incorrectly updated to the Cluster object during the status refresh. The expected Cluster status conditions should no longer include the old condition.
Which issue(s) this PR fixes: Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: