-
Notifications
You must be signed in to change notification settings - Fork 327
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) give CA managers all backends at once #2956
Conversation
Signed-off-by: Mike Beaumont <mjboamail@gmail.com>
Signed-off-by: Mike Beaumont <mjboamail@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #2956 +/- ##
==========================================
+ Coverage 52.37% 52.40% +0.03%
==========================================
Files 912 912
Lines 52517 52520 +3
==========================================
+ Hits 27505 27525 +20
+ Misses 22819 22800 -19
- Partials 2193 2195 +2
Continue to review full report at Codecov.
|
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.
This change looks fine.
I would suggest expanding the commit message to explain why this change is needed. I'm assuming that it makes some subsequent change easier. If so, including the explanation in the commit message makes life easier for PR review, but also for people who will look at and try to understand the context of this change in a year from now.
11a9dac
to
c5fb860
Compare
Co-authored-by: James Peach <james.peach@konghq.com> Signed-off-by: Mike Beaumont <mjboamail@gmail.com>
Signed-off-by: Mike Beaumont <mjboamail@gmail.com>
714d308
to
f269bb8
Compare
This reverts commit c586219.
Summary
This allows CA managers to know all of the relevant backends during the
Enable
step . A manager is then able to, for example, remove state used to maintain previous backends that may no longer exist.