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

mds: pretty json from `tell` commands #14105

Merged
merged 1 commit into from Apr 18, 2017

Conversation

Projects
None yet
2 participants
@jcsp
Contributor

jcsp commented Mar 23, 2017

The overhead of the whitespace is trivial and
makes the output somewhat human readable. Previously
I was always taking damage ls into a file and
parsing it out with python.

Signed-off-by: John Spray john.spray@redhat.com

@jcsp

This comment has been minimized.

Contributor

jcsp commented Apr 3, 2017

retest this please

mds: pretty json from `tell` commands
The overhead of the whitespace is trivial and
makes the output somewhat human readable.  Previously
I was always taking `damage ls` into a file and
parsing it out with python.

Signed-off-by: John Spray <john.spray@redhat.com>

@jcsp jcsp requested a review from batrick Apr 14, 2017

@jcsp

This comment has been minimized.

@batrick

This comment has been minimized.

Member

batrick commented Apr 17, 2017

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>

John, if you haven't already, you might try the jq command which pretty prints/colorizes json for you and allows you to do json manipulation from the CLI.

@jcsp jcsp merged commit 612fe84 into ceph:master Apr 18, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@jcsp jcsp deleted the jcsp:wip-pretty-tell branch Apr 18, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment