Change in vdsm[ovirt-3.4]: virt: Remove all guest sockets on cleanup

danken at redhat.com danken at redhat.com
Tue Apr 1 13:26:24 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: virt: Remove all guest sockets on cleanup
......................................................................


virt: Remove all guest sockets on cleanup

Previously vdsm was cleaning up the guest socket files, but when
introducing the symlinks we started leaving behind the real sockets
and just removed the symlinks.
This patch fixes this by first removing the file the symlink points
to and then removes the symlink.

Change-Id: Iaf3bfbd0eedcb6486cbde685d7b6ee64e8dc8d8c
Bug-Url: https://bugzilla.redhat.com/1082986
Signed-off-by: Vinzenz Feenstra <vfeenstr at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/26278
Reviewed-by: Michal Skrivanek <michal.skrivanek at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/26288
---
M vdsm/vm.py
1 file changed, 8 insertions(+), 2 deletions(-)

Approvals:
  Vinzenz Feenstra: Verified
  Dan Kenigsberg: Looks good to me, approved
  Michal Skrivanek: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaf3bfbd0eedcb6486cbde685d7b6ee64e8dc8d8c
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.4
Gerrit-Owner: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list