Eduardo has posted comments on this change.

Change subject: Add forceLink() which replaces a link if it already exists.
......................................................................


Patch Set 3: (1 inline comment)

....................................................
File vdsm/storage/fileVolume.py
Line 22: import os
Line 23: import sanlock
Line 24: 
Line 25: import storage_exception as se
Line 26: from vdsm.config import config
I think no because utils is not referenced directly.

All the references are using oop.
Line 27: from sdc import sdCache
Line 28: import outOfProcess as oop
Line 29: import volume
Line 30: import image


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idea4ab66cc2954fdb11606daa4a73ad3edb0c7ef
Gerrit-PatchSet: 3
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: Eduardo <[email protected]>
Gerrit-Reviewer: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to