Ryan Harper has posted comments on this change.
Change subject: vdsm/storage/misc.py: ensure uuid input is str or unicode for regex
......................................................................
Patch Set 3: (1 inline comment)
....................................................
File vdsm/storage/misc.py
Line 490: if isinstance(uuid, str) or isinstance(uuid, unicode):
New to python "standards". There's a whopping one line in the entire code
base that has this form for string type checking, so it's not like it was beating me
over the head with the exact form that this check should take.
In any case, it's easy enough to fix.
--
To view, visit
http://gerrit.ovirt.org/3687
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If16466e88a50d3af03ae7830d4d8148ebf69d0d9
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ryan Harper <ryanh(a)us.ibm.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Lei Li <lilei(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Ryan Harper <ryanh(a)us.ibm.com>