Closed huiwq1990 closed 2 years ago
@huiwq1990: GitHub didn't allow me to assign the following users: your_reviewer.
Note that only openyurtio members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. For more information please see the contributor guide
Merging #88 (32953f7) into master (8d865d2) will increase coverage by
7.90%
. The diff coverage is49.50%
.:exclamation: Current head 32953f7 differs from pull request most recent head 7f76e05. Consider uploading reports for the commit 7f76e05 to get more accurate results
@@ Coverage Diff @@
## master #88 +/- ##
==========================================
+ Coverage 7.78% 15.69% +7.90%
==========================================
Files 17 20 +3
Lines 1901 2498 +597
==========================================
+ Hits 148 392 +244
- Misses 1724 2037 +313
- Partials 29 69 +40
Flag | Coverage Δ | |
---|---|---|
unittests | 15.69% <49.50%> (+7.90%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
.../webhook/yurtappdaemon/yurtappdaemon_validation.go | 47.36% <ø> (ø) |
|
...anager/webhook/yurtappset/yurtappset_validation.go | 44.64% <ø> (ø) |
|
...ger/webhook/yurtappdaemon/yurtappdaemon_webhook.go | 46.00% <46.00%> (ø) |
|
...ppmanager/webhook/yurtappset/yurtappset_webhook.go | 52.94% <52.94%> (ø) |
|
...r/controller/yurtingress/yurtingress_controller.go | 17.64% <0.00%> (ø) |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
Fixes #78
@Congrool @rambohe-ch
Thanks for your contribution. But I'm not familiar with how to generate webhook with controller-runtime. It will take some time. Anyone else that can help review this pr is also welcome.
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: huiwq1990, rambohe-ch
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?
What this PR does / why we need it:
1) upgrade controller-runtime version 2) webhook code testable
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
other Note