Francesco Romani has uploaded a new change for review. Change subject: vm: remove dup initialization of Vm._guestSockFile ......................................................................
vm: remove dup initialization of Vm._guestSockFile In Vm class' __init__, guestSockFile is initialized, and the re-initialized a few lines below with no use of this field in between. To reduce the confusion, this patch removes the first useless initialization. Change-Id: Ie8d959dfc3ddd96e9d019dd93394bf4390adf5cf Signed-off-by: Francesco Romani <[email protected]> --- M vdsm/virt/vm.py 1 file changed, 0 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/51/40751/1 diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py index 3f4f1d7..45bd980 100644 --- a/vdsm/virt/vm.py +++ b/vdsm/virt/vm.py @@ -305,8 +305,6 @@ self._lastStatus = vmstatus.WAIT_FOR_LAUNCH self._migrationSourceThread = migration.SourceThread(self) self._kvmEnable = self.conf.get('kvmEnable', 'true') - self._guestSocketFile = constants.P_VDSM_RUN + self.conf['vmId'] + \ - '.guest.socket' self._incomingMigrationFinished = threading.Event() self.id = self.conf['vmId'] self._volPrepareLock = threading.Lock() -- To view, visit https://gerrit.ovirt.org/40751 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ie8d959dfc3ddd96e9d019dd93394bf4390adf5cf Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
