From Dan Kenigsberg <dan...@redhat.com>:

Dan Kenigsberg has posted comments on this change.

Change subject: lvs-stats: Analyze sosreport lvm commands output
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/79698/2//COMMIT_MSG
Commit Message:

Line 6: 
Line 7: lvs-stats: Analyze sosreport lvm commands output
Line 8: 
Line 9: This tool reads the lvs sos command output, and print a report about the
Line 10: found logical volumes.
> I'm not sure we want this tool installed on a host, it makes sense only whe
we ship sos/plugins/vdsm.py in vdsm.rpm. The scripts it calls can be included 
there as well.

This can be done in a follow-up patch.
Line 11: 
Line 12: Here is an examle run:
Line 13: 
Line 14: $ cd sosreport-hostname.*/sos_commands/lvm2


-- 
To view, visit https://gerrit.ovirt.org/79698
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8a172092210d02eb49d1d54de6958d3f80bcba1e
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Ala Hino <ah...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Germano Veit Michel <germ...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to