Change in vdsm[master]: BZ829710 Get VMList with oop if necessary

danken at redhat.com danken at redhat.com
Mon Aug 20 05:01:20 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: BZ829710 Get VMList with oop if necessary
......................................................................


Patch Set 3: Fails

(1 inline comment)

the robot has checked out the wrong branch.

 vdsm/storage/sd.py:25: 'glob' imported but unused

....................................................
File vdsm/storage/sd.py
Line 20: 
Line 21: import os
Line 22: import logging
Line 23: import types
Line 24: import threading
this has to be removed, or pylakes would cry.
Line 25: from glob import glob
Line 26: from collections import namedtuple
Line 27: import codecs
Line 28: 


--
To view, visit http://gerrit.ovirt.org/6867
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I021e9d03b29dfc9cf1c3b7fa5ec8a302e7912cf7
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Dafna Ron <dron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list