Irit Goihman 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 it runs only over rhv-h. sos report ignores the command if it doesn't exist. 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: Irit Goihman <igoih...@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