Kong / kubernetes-ingress-controller

:gorilla: Kong for Kubernetes: The official Ingress Controller for Kubernetes.
https://docs.konghq.com/kubernetes-ingress-controller/
Apache License 2.0
2.2k stars 590 forks source link

[Backport release/2.12.x] fix(controller): Do not update existing Programmed condition to Unknown #6403

Closed randmonkey closed 1 month ago

randmonkey commented 1 month ago

Backport https://github.com/Kong/kubernetes-ingress-controller/commit/4b0a44a4a7cfc158719a739eb78e8ca486e4fbaa from https://github.com/Kong/kubernetes-ingress-controller/pull/6395 to 2.12.x.

What this PR does / why we need it:

Which issue this PR fixes:

Special notes for your reviewer:

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 50.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 36.2%. Comparing base (b929f67) to head (732faa7). Report is 12 commits behind head on release/2.12.x.

Files Patch % Lines
internal/controllers/utils/conditions.go 25.0% 2 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## release/2.12.x #6403 +/- ## ================================================= - Coverage 68.0% 36.2% -31.9% ================================================= Files 165 164 -1 Lines 19344 19268 -76 ================================================= - Hits 13159 6978 -6181 - Misses 5397 11504 +6107 + Partials 788 786 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.