Skip to content
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(kumactl) added shorthand command name for mesh #664

Merged
merged 2 commits into from
Apr 6, 2020

Conversation

tharun208
Copy link
Contributor

Summary

added the shorthand name for mesh argument

SUMMARY_GOES_HERE

Full changelog

  • Added shorthand name for mesh argument in kumactl

Issues resolved

Fix #662

Open Question:
We don't have a test for inspect dataplane with mesh name other than the default name. Is it worth adding?

* added shorthand message for mesh argument in kumactl
Fix kumahq#662
@tharun208 tharun208 requested a review from a team April 4, 2020 07:47
@jakubdyszkiewicz
Copy link
Contributor

Add a CHANGELOG entry please

@tharun208
Copy link
Contributor Author

added changelog entry

@jakubdyszkiewicz jakubdyszkiewicz merged commit 0fd16ab into kumahq:master Apr 6, 2020
@jakubdyszkiewicz
Copy link
Contributor

Thank you, merging. Just for future PRs. This is a feature not a fix. Fix is for fixing bugs.

@tharun208
Copy link
Contributor Author

Thank you, merging. Just for future PRs. This is a feature not a fix. Fix is for fixing bugs.

thanks and noted it.

@tharun208 tharun208 deleted the fix/#662 branch April 6, 2020 09:35
tharun208 added a commit to tharun208/kuma that referenced this pull request Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support -m in addition to --mesh in kumactl
2 participants