Federico Simoncelli has submitted this change and it was merged. Change subject: storage: fix getDomUuidFromVolumePath() ......................................................................
storage: fix getDomUuidFromVolumePath() getDomUuidFromVolumePath returned the imgUUID instead of the sdUUID. Since this is used as the parameter for getProcessPool() a process pool was created per image id instead of per domain id. Since imgUUIDs are much more abundant this can cause the creation of an obscene number of process pools and in turn a lot of processes. Change-Id: I17267993c8e3a582b1277f1e8aacd6bc69c8e4ec Bug-Url:http://bugzilla.redhat.com/017735 Signed-off-by: Saggi Mizrahi <[email protected]> Reviewed-on: http://gerrit.ovirt.org/20747 Reviewed-by: Federico Simoncelli <[email protected]> Tested-by: Federico Simoncelli <[email protected]> --- M tests/Makefile.am A tests/fileVolumeTests.py M vdsm/storage/fileVolume.py 3 files changed, 38 insertions(+), 4 deletions(-) Approvals: Federico Simoncelli: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20747 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I17267993c8e3a582b1277f1e8aacd6bc69c8e4ec Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
