Closed aamoyel closed 1 year ago
/lgtm
please add testing details to the PR description, and, ideally, a unit test
/ok-to-test
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
controllers/kubevirtcluster_controller.go | 1 | 3 | 33.33% | ||
<!-- | Total: | 1 | 3 | 33.33% | --> |
Totals | |
---|---|
Change from base Build 4542574610: | -0.03% |
Covered Lines: | 947 |
Relevant Lines: | 1805 |
What can i do to make this PR approved ? All tests are green to me.
Re open for merge. Sorry for the miss.
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: aamoyel Once this PR has been reviewed and has the lgtm label, please ask for approval from agradouski. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
the fix looks good - let's make sure we have it tested, and ideally integrated in our automated testing process. @aamoyel - can we add an integration test for this?
Fix #219