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

luminous: crush/CrushWrapper: fix crush tree json dumper #24482

Merged
merged 1 commit into from Oct 25, 2018

Conversation

Projects
None yet
4 participants
@pdvian

pdvian commented Oct 8, 2018

songshuangyang Prashant D
crush/CrushWrapper: fix crush tree json dumper
    The output json string is invalid for 'osd crush tree --format=json'
    command. It contains a array of 'nodes' and a array of 'stray', but
    not in a json object, and the stray array was not implemented.
    Applications which depends on the output of the above MonCommand will
    occur json parse error.

    Signed-off-by: Oshyn Song <dualyangsong@gmail.com>

(cherry picked from commit 35c0d1f)

@smithfarm smithfarm added this to the luminous milestone Oct 9, 2018

@smithfarm smithfarm requested review from liewegas and tchaikov Oct 9, 2018

@yuriw

This comment has been minimized.

Contributor

yuriw commented Oct 19, 2018

@yuriw yuriw merged commit f6fd799 into ceph:luminous Oct 25, 2018

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment