fix update strategy default handling#1179
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1179 +/- ##
===========================================
- Coverage 62.36% 30.88% -31.48%
===========================================
Files 197 239 +42
Lines 4854 6550 +1696
===========================================
- Hits 3027 2023 -1004
- Misses 1439 4351 +2912
+ Partials 388 176 -212
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
72295c7 to
60bbb71
Compare
60bbb71 to
59c6b27
Compare
Signed-off-by: Zakhar Dvurechensky <72825626+Zakharden@users.noreply.github.com>
|
I see this as code-complete on the branch and currently blocked only by maintainer flow. Please re-run checks/review and let me know if anything else is required. |
59c6b27 to
6dd3f0f
Compare
|
I confirmed this branch is code-complete and tests pass; patch coverage is green but Please rerun maintainer-side checks/coverage step when convenient; no further local code changes are pending. |
|
FYI branch is code-complete; and tests are green, while only remains failing. Please re-run maintainer-side coverage/review flow so we can finalize. |
|
Branch is code-complete and tests pass. |
Fixes false positive for omitted rolling update strategy. Validation: go test ./pkg/templates/updateconfig, git diff --check.