-
Notifications
You must be signed in to change notification settings - Fork 332
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(kumactl): add --api-timeout flag #3723
Conversation
|
Codecov Report
@@ Coverage Diff @@
## master #3723 +/- ##
=======================================
Coverage 50.85% 50.85%
=======================================
Files 925 925
Lines 56789 56790 +1
=======================================
+ Hits 28878 28880 +2
+ Misses 25619 25616 -3
- Partials 2292 2294 +2
Continue to review full report at Codecov.
|
tests were flaky because we were modifying a global variable. Extracted this global var in a configuration parameter as it may be useful anyway Signed-off-by: Charly Molter <charly.molter@konghq.com> Actually do api-timeout for the whole kumactl Signed-off-by: Charly Molter <charly.molter@konghq.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.
thanks for iterating on this
Signed-off-by: Charly Molter <charly.molter@konghq.com>
Signed-off-by: Charly Molter <charly.molter@konghq.com>
Summary
tests were flaky because we were modifying a global variable.
Extracted this global var in a configuration parameter as it may be
useful anyway.
Documentation
[ ] Link to the website documentation PRTesting
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