-
Notifications
You must be signed in to change notification settings - Fork 338
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(kuma-cp): validate DP compat on kuma backend #4236
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4236 +/- ##
==========================================
- Coverage 55.65% 55.63% -0.03%
==========================================
Files 933 934 +1
Lines 56257 56288 +31
==========================================
+ Hits 31312 31315 +3
- Misses 22457 22481 +24
- Partials 2488 2492 +4
Continue to review full report at Codecov.
|
/format |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, feel free to ignore suggestion
1f5d4cd
to
f7178d9
Compare
Signed-off-by: Paul Parkanzky <[email protected]>
fix(kuma-cp): simplify min version check Co-authored-by: Mike Beaumont <[email protected]> Signed-off-by: Paul Parkanzky <[email protected]>
f7178d9
to
f4cc0cc
Compare
Signed-off-by: Paul Parkanzky <[email protected]>
* feat(kuma-cp): validate DP compat on kuma backend Signed-off-by: Paul Parkanzky <[email protected]>
Signed-off-by: Paul Parkanzky [email protected]
Summary
Kuma-cp side validation of Kuma DP and Zone CP versions. Result stored as boolean within DataplaneInsight and ZoneInsight. Will require a GUI change to consume the bool.
Need to decide if this should be backported to stable.
Full changelog
Issues resolved
Fix #4203
Documentation
Testing
Backwards compatibility
UPGRADE.md
with any steps users will need to take when upgrading.backport-to-stable
label if the code follows our backporting policy