Change in vdsm[master]: Fix vdsClient.repoStats() bad path.

danken at redhat.com danken at redhat.com
Wed Oct 2 14:39:00 UTC 2013


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 <ewarszaw at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Gadi Ickowicz <gickowic at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list