Yaniv Bronhaim has posted comments on this change.

Change subject: sos: collect 'nodectl info' output
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.ovirt.org/#/c/63765/1/vdsm/sos/vdsm.py.in
File vdsm/sos/vdsm.py.in:

Line 131:         self.collectExtOutput("/sbin/fdisk -l")
Line 132:         self.collectExtOutput("/usr/bin/iostat")
Line 133:         self.collectExtOutput("/sbin/iscsiadm -m node")
Line 134:         self.collectExtOutput("/sbin/iscsiadm -m session")
Line 135:         self.collectExtOutput("/usr/sbin/nodectl info")
I don't have /usr/sbin/nodectl over my f24 and centos7 .. should we run it only 
over rhv-h?
Line 136: 
Line 137:         with closing(_connectToVdsm()) as cli:
Line 138:             self.addObjectAsFile(cli.getVdsCapabilities(), 
"getVdsCapabilities")
Line 139:             self.addObjectAsFile(cli.getVdsStats(), "getVdsStats")


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8360e3812d9a84a592099ba93c583aa6c68a3ed0
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <igoih...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Reply via email to