projectcontour / contour-operator

Experimental repository to explore an operator for deploying Contour
Apache License 2.0
43 stars 34 forks source link

sync CRDs from projectcontour/contour #573

Closed skriss closed 1 year ago

skriss commented 1 year ago

Signed-off-by: Steve Kriss krisss@vmware.com

skriss commented 1 year ago

Hmm, not sure why this is still showing a CRD error when running envtest-based tests, but not when running in kind clusters..

skriss commented 1 year ago

In light of #574, I opted to just disable the envtest-based tests rather than mucking around with kubebuilder versions/etc. since the tests are likely pretty redundant with E2E's anyway

codecov[bot] commented 1 year ago

Codecov Report

Base: 75.21% // Head: 49.63% // Decreases project coverage by -25.57% :warning:

Coverage data is based on head (7189e7a) compared to base (e991cb9). Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #573 +/- ## =========================================== - Coverage 75.21% 49.63% -25.58% =========================================== Files 29 29 Lines 2635 2635 =========================================== - Hits 1982 1308 -674 - Misses 512 1307 +795 + Partials 141 20 -121 ``` | [Impacted Files](https://codecov.io/gh/projectcontour/contour-operator/pull/573?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour) | Coverage Δ | | |---|---|---| | [internal/operator/config.go](https://codecov.io/gh/projectcontour/contour-operator/pull/573/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour#diff-aW50ZXJuYWwvb3BlcmF0b3IvY29uZmlnLmdv) | `0.00% <0.00%> (-100.00%)` | :arrow_down: | | [internal/controller/controller.go](https://codecov.io/gh/projectcontour/contour-operator/pull/573/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour#diff-aW50ZXJuYWwvY29udHJvbGxlci9jb250cm9sbGVyLmdv) | `0.00% <0.00%> (-72.67%)` | :arrow_down: | | [internal/operator/operator.go](https://codecov.io/gh/projectcontour/contour-operator/pull/573/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour#diff-aW50ZXJuYWwvb3BlcmF0b3Ivb3BlcmF0b3IuZ28=) | `0.00% <0.00%> (-72.50%)` | :arrow_down: | | [internal/objects/serviceaccount/service\_account.go](https://codecov.io/gh/projectcontour/contour-operator/pull/573/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour#diff-aW50ZXJuYWwvb2JqZWN0cy9zZXJ2aWNlYWNjb3VudC9zZXJ2aWNlX2FjY291bnQuZ28=) | `0.00% <0.00%> (-71.16%)` | :arrow_down: | | [internal/objects/contour/finalizer.go](https://codecov.io/gh/projectcontour/contour-operator/pull/573/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour#diff-aW50ZXJuYWwvb2JqZWN0cy9jb250b3VyL2ZpbmFsaXplci5nbw==) | `0.00% <0.00%> (-55.56%)` | :arrow_down: | | [internal/objects/rbac.go](https://codecov.io/gh/projectcontour/contour-operator/pull/573/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour#diff-aW50ZXJuYWwvb2JqZWN0cy9yYmFjLmdv) | `0.00% <0.00%> (-51.82%)` | :arrow_down: | | [internal/status/status.go](https://codecov.io/gh/projectcontour/contour-operator/pull/573/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour#diff-aW50ZXJuYWwvc3RhdHVzL3N0YXR1cy5nbw==) | `0.00% <0.00%> (-50.00%)` | :arrow_down: | | [internal/objects/namespace/namespace.go](https://codecov.io/gh/projectcontour/contour-operator/pull/573/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour#diff-aW50ZXJuYWwvb2JqZWN0cy9uYW1lc3BhY2UvbmFtZXNwYWNlLmdv) | `10.29% <0.00%> (-48.53%)` | :arrow_down: | | [pkg/slice/slice.go](https://codecov.io/gh/projectcontour/contour-operator/pull/573/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour#diff-cGtnL3NsaWNlL3NsaWNlLmdv) | `50.00% <0.00%> (-45.84%)` | :arrow_down: | | [internal/objects/configmap/configmap.go](https://codecov.io/gh/projectcontour/contour-operator/pull/573/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour#diff-aW50ZXJuYWwvb2JqZWN0cy9jb25maWdtYXAvY29uZmlnbWFwLmdv) | `28.86% <0.00%> (-39.18%)` | :arrow_down: | | ... and [13 more](https://codecov.io/gh/projectcontour/contour-operator/pull/573/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour) | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=projectcontour)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

sunjayBhatia commented 1 year ago

could the issue be a tools version? envtest etc. is probably pretty out of date here and on my dev machine currently

cool with dropping the tests since yeah we're planning on stopping supporting this anyways (and code hasn't changed tbh)

skriss commented 1 year ago

could the issue be a tools version? envtest etc. is probably pretty out of date here and on my dev machine currently

cool with dropping the tests since yeah we're planning on stopping supporting this anyways (and code hasn't changed tbh)

Yeah my best guess is it's due to an old envtest/associated binaries, but updating to kubebuilder v3 to get newer versions is not super-straightforward and I didn't see the point of going down that road. I think as long as we have the E2E's working we still have decent confidence here.