Closed ghost closed 3 years ago
Kudos, SonarCloud Quality Gate passed!
0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells
No Coverage information
0.0% Duplication
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: fdupont-redhat, mansam
To complete the pull request process, please assign
You can assign the PR to them by writing /assign
in a comment when ready.
The full list of commands accepted by this bot can be found here.
/retest
/lgtm
/test pull-vm-import-operator-e2e-k8s-1.18
/retest
/test pull-vm-import-operator-e2e-k8s-1.18
It's Good Friday today, so I'll merge my own PR since it's been approved.
It seems that the formula should be
roundup(size/(1-overhead), 512)
. Rounding integers doesn't work withmath.Round
. I've also changed the overhead values to use real numbers that have been calculated separately.