Closed hzxuzhonghu closed 3 weeks ago
Attention: Patch coverage is 26.08696%
with 17 lines
in your changes missing coverage. Please review.
Project coverage is 53.23%. Comparing base (
9bef054
) to head (c5b6f8d
). Report is 96 commits behind head on main.
Files with missing lines | Patch % | Lines |
---|---|---|
pkg/controller/workload/workload_processor.go | 26.08% | 15 Missing and 2 partials :warning: |
Files with missing lines | Coverage Δ | |
---|---|---|
pkg/controller/workload/workload_processor.go | 66.83% <26.08%> (-1.97%) |
:arrow_down: |
... and 9 files with indirect coverage changes
Continue to review full report in Codecov by Sentry.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update b7ebf49...c5b6f8d. Read the comment docs.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED
Approval requirements bypassed by manually added approval.
This pull-request has been approved by:
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@LiZhenCheng9527 Can you take another look
/lgtm
What type of PR is this?
/kind feature
What this PR does / why we need it:
Which issue(s) this PR fixes: Fixes #693
Special notes for your reviewer:
Does this PR introduce a user-facing change?: