Change in vdsm[master]: BZ#741203 Don't return a value in domain selftest
Igor Lvovsky
ilvovsky at redhat.com
Sun Oct 9 13:30:27 UTC 2011
Igor Lvovsky has posted comments on this change.
Change subject: BZ#741203 Don't return a value in domain selftest
......................................................................
Patch Set 1: Looks good to me, but someone else must approve
(1 inline comment)
....................................................
File vdsm/storage/sp.py
Line 1292: 'masterValidate' : { 'mount' : False, 'valid' : False }
Line 1293: }
Line 1294: code = 0
Line 1295: try:
Line 1296: domain.selftest()
Ah, yes you are right, sorry. Indeed it's doesn't change behavior
Line 1297:
Line 1298: res = domain.getStats()
Line 1299: stats.update(res)
Line 1300: # Add here more selftests if needed
--
To view, visit http://gerrit.usersys.redhat.com/1012
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I551970c1f426f2b28ad58e49310338de6b40a29f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
More information about the vdsm-patches
mailing list