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
[ceph-volume] Use a delimited CLI output parser instead of JSON #17097
Conversation
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Signed-off-by: Alfredo Deza <adeza@redhat.com>
jenkins test ceph-volume tox |
jenkins test ceph-volume centos7-create |
jenkins test ceph-volume xenial-create |
1 similar comment
jenkins test ceph-volume xenial-create |
Signed-off-by: Alfredo Deza <adeza@redhat.com>
jenkins test ceph-volume xenial-create |
@ktdreyer there is a small change to Debian packaging I had to add that was preventing ceph-volume-systemd (the executable) from being added to the deb package. Xenial tests caught these, which are now passing |
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.
thanks
Older versions of LVM do not support JSON output. This PR uses LVM's flags to customize the output so that we can parse it providing the same functionality. Specifically the
--reportformat
flag does not work on XenialFixes: https://bugzilla.redhat.com/show_bug.cgi?id=1478599