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): XDS config dump on Global CP #4301
Conversation
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com>
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com>
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #4301 +/- ##
==========================================
- Coverage 55.70% 55.52% -0.19%
==========================================
Files 935 940 +5
Lines 56434 56733 +299
==========================================
+ Hits 31439 31503 +64
- Misses 22518 22743 +225
- Partials 2477 2487 +10
Continue to review full report at Codecov.
|
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com>
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com>
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com>
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.
Phew, that's not an easy code 😅 But that's awesome work and functionality!
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com>
Summary
This PR introduces support for executing XDS Config Dump on Global CP.
It establishes a pattern of exchanging information between Global CP and Zone CP aside of exchanging Kuma resources.
Support of XDSConfig stream inside
pkg/kds/mux
may not be the cleanest solution, but I'd rather postpone refactors to the moment we introduce streaming of resources to the same service (as described here #3789 (comment))Issues resolved
Fix #3789
Documentation
In progress
Testing
Backwards compatibility
- [ ] UpdateUPGRADE.md
with any steps users will need to take when upgrading.- [ ] Addbackport-to-stable
label if the code follows our backporting policy