Dan Kenigsberg has posted comments on this change. Change subject: Fix vdsClient.repoStats() bad path. ......................................................................
Patch Set 1: Code-Review+2 (1 comment) .................................................... File client/vdsClient.py Line 1684: def repoStats(self, args): Line 1685: stats = self.s.repoStats() Line 1686: if stats['status']['code']: Line 1687: print "count not get repo stats" Line 1688: return int(stats['status']['code']) apparently, repoStats never failed, so we never got to this code block. Line 1689: for d in stats: Line 1690: if d == "status": Line 1691: continue Line 1692: print 'Domain %s %s' % (d, str(stats[d])) -- To view, visit http://gerrit.ovirt.org/19764 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Idf7f6c367c83c2d00967e31dda6c9d0ef2fd3e57 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Eduardo <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Gadi Ickowicz <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
