-
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
chore(kumactl) improve flags description, add extra validation #2352
Conversation
Signed-off-by: lobkovilya <ilya.lobkov@konghq.com>
Signed-off-by: Ilya Lobkov <lobkovilya@yandex.ru>
Codecov Report
@@ Coverage Diff @@
## master #2352 +/- ##
==========================================
+ Coverage 52.14% 52.51% +0.37%
==========================================
Files 875 879 +4
Lines 47801 47841 +40
==========================================
+ Hits 24925 25125 +200
+ Misses 20855 20669 -186
- Partials 2021 2047 +26
Continue to review full report at Codecov.
|
return errors.Errorf("--kuma-dp-user or --kuma-dp-uid should be supplied") | ||
} | ||
|
||
if args.RedirectAllDNSTraffic && args.RedirectDNS { | ||
return errors.Errorf("one of --redirect-dns or --redirect-all-dns-traffic should be specified") |
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.
nit: I'd phrase this as
return errors.Errorf("one of --redirect-dns or --redirect-all-dns-traffic should be specified") | |
return errors.Errorf("exactly one of --redirect-dns or --redirect-all-dns-traffic should be specified") |
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.
None of them is also a valid case, I think I probably should do only one of ...
?
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
(cherry picked from commit 4aa8ec3)
Summary
Some minor chores for
kumactl install transparent-proxy
command.Full changelog
--modify-iptables
, correct me if I'm wrong but what's the point runningkumactl install transparent-proxy
with--modify-iptables=false
--redirect-dns
and--redirect-all-dns-traffic
passed at the same timeIssues resolved
Fix #XXX
Documentation
Testing
Backwards compatibility
backport-to-stable
label if the code is backwards compatible. Otherwise, list breaking changes.