Closed skriss closed 2 years ago
Yet another PR that would've failed the upgrade test due to https://github.com/projectcontour/contour-operator/issues/438 were it not disabled already.
Merging #464 (51ddbd5) into main (8439647) will decrease coverage by
2.55%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #464 +/- ##
==========================================
- Coverage 80.41% 77.85% -2.56%
==========================================
Files 29 29
Lines 2231 2213 -18
==========================================
- Hits 1794 1723 -71
- Misses 322 355 +33
- Partials 115 135 +20
Impacted Files | Coverage Δ | |
---|---|---|
internal/operator/operator.go | 77.77% <ø> (+25.92%) |
:arrow_up: |
internal/objects/clusterrole/cluster_role.go | 90.17% <100.00%> (ø) |
|
internal/status/status.go | 48.64% <0.00%> (-24.33%) |
:arrow_down: |
internal/objects/contour/finalizer.go | 62.50% <0.00%> (-18.75%) |
:arrow_down: |
internal/objects/rbac.go | 54.76% <0.00%> (-9.53%) |
:arrow_down: |
internal/controller/controller.go | 74.59% <0.00%> (-8.20%) |
:arrow_down: |
internal/objects/configmap/configmap.go | 70.88% <0.00%> (-6.33%) |
:arrow_down: |
internal/objects/job/job.go | 76.52% <0.00%> (-5.22%) |
:arrow_down: |
internal/objects/contour/contour.go | 76.62% <0.00%> (-5.20%) |
:arrow_down: |
internal/objects/service/service.go | 84.34% <0.00%> (-4.35%) |
:arrow_down: |
... and 2 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 8439647...51ddbd5. Read the comment docs.
Yet another PR that would've failed the upgrade test due to #438 were it not disabled already.
Also, yet another set of effort solved by combining the repos. :cry:
Updates operator per Contour changes:
https://github.com/projectcontour/contour/pull/4141 https://github.com/projectcontour/contour/pull/4138 https://github.com/projectcontour/contour/pull/4145
Signed-off-by: Steve Kriss krisss@vmware.com