Dan Kenigsberg 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/outOfProcess.py
Line 81: def OopWrapper(procPool):
Line 82: return _ModuleWrapper("oop", procPool, DEFAULT_TIMEOUT,
Line 83: (("os",
Line 84: ("path",)),
Line 85: ("vdsm", ("utils",)),
doesn't this cause an unfortunate bump to the size of oop handlers?
Line 86: "glob",
--
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: Daniel Paikov <[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
