-
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
fix(kuma-cp) disable zone #2884
Conversation
Signed-off-by: Ilya Lobkov <ilya.lobkov@konghq.com>
consecutive = 0 | ||
} | ||
return consecutive == 20 | ||
}, "30s", "10ms").Should(BeTrue()) |
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.
I think you should use CollectResponsesByInstance
instead of rebuilding the logic.
Signed-off-by: Ilya Lobkov <ilya.lobkov@konghq.com>
Signed-off-by: Ilya Lobkov <ilya.lobkov@konghq.com>
Codecov Report
@@ Coverage Diff @@
## master #2884 +/- ##
=======================================
Coverage 52.31% 52.31%
=======================================
Files 900 900
Lines 52295 52315 +20
=======================================
+ Hits 27356 27369 +13
- Misses 22761 22767 +6
- Partials 2178 2179 +1
Continue to review full report at Codecov.
|
Signed-off-by: Ilya Lobkov <ilya.lobkov@konghq.com> (cherry picked from commit 9fffa21)
Summary
A bug that didn't allow users to disable a zone.
Full changelog
Issues resolved
N/A
Documentation
N/A
Testing
Backwards compatibility
backport-to-stable
label if the code is backwards compatible. Otherwise, list breaking changes.