coveralls
commented
6 months ago Pull Request Test Coverage Report for Build 8543739403
Details
- 19 of 45 (42.22%) changed or added relevant lines in 3 files are covered.
- 2 unchanged lines in 2 files lost coverage.
- Overall coverage increased (+0.05%) to 65.427%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
|---|---|---|---|---|---|
| pkg/controllers/backend/backend.go | 3 | 5 | 60.0% | ||
| pkg/controllers/nodeBackend/nodeBackend.go | 5 | 7 | 71.43% | ||
| pkg/util/util.go | 11 | 33 | 33.33% | ||
| <!-- | Total: | 19 | 45 | 42.22% | --> |
| Files with Coverage Reduction | New Missed Lines | % | ||
|---|---|---|---|---|
| pkg/util/util.go | 1 | 53.85% | ||
| pkg/controllers/nodeBackend/nodeBackend.go | 1 | 65.98% | ||
| <!-- | Total: | 2 | --> |
| Totals | |
|---|---|
| Change from base Build 8448533111: | 0.05% |
| Covered Lines: | 2269 |
| Relevant Lines: | 3468 |
In scenarios, in Flannel cni cluster with Service having multiple nodeports assigned, Native ingress controller is picking the first nodeport instead of picking the targetport based on the port in the ingress.
Fix: We fetch nodeport from the respective service definition.