-
Notifications
You must be signed in to change notification settings - Fork 83
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
JSON/YAML output for CLI commands #629
Comments
Hello @kosmoz I would like to work on this issue. |
All yours! |
Hi @Aryakoste, we created some subtasks now, since these are a lot of separate tasks. Could you please pick one of the linked issues, then we'll re-assign you to that, thanks! |
@christophenne can I work on multiple tasks. |
@Aryakoste which tasks would you like to do? |
I can work on update and configure commands tasks. @christophenne |
I think you can start with #657 and #658 – usually we try to have contributors work at only one issue at a time, but in this case it makes a bit more sense because the tasks are similar. Can you please shortly comment in these two? I think you need to be "active" in the issue, otherwise github does not allow me to assign you it seems. |
I have commented in the above 2 issues. |
All CLI commands should support specifying an output format if applicable. This can be done using the
--output
flag (short:-o
). This flag is optional (default: empty) and can be set to "json" or "yaml". Output should be written to STDOUT.If a command modifies some resources, the output should be of the resource after modification.
Those commands should also provide a
--dry-run
option.Examples
Subtasks
--output
support forglasskube list
#465--output
support forglasskube describe
#655--output
support forglasskube install
#656--dry-run
support forglasskube install
#659--output
support forglasskube update
#657--dry-run
support forglasskube update
#660--output
support forglasskube configure
#658--dry-run
support forglasskube configure
#661--output
support forglasskube bootstrap
#724--dry-run
support forglasskube bootstrap
#725The text was updated successfully, but these errors were encountered: