Vinzenz Feenstra has uploaded a new change for review.

Change subject: virt: Pass missing parameter on desktopLogoff
......................................................................

virt: Pass missing parameter on desktopLogoff

Previously this parameter was not passed and all seemed
fine from the logs, but it actually wasn't the exception
was swallow that the parameter was missing.
The parameter is actually not used, but still mandatory.

Change-Id: I65b030d4a7a7ed7ad3f6279c66120d1f9e45d396
Backport-To: 4.0
Bug-Url: https://bugzilla.redhat.com/1349383
Signed-off-by: Vinzenz Feenstra <vfeen...@redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/66/59866/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 0e756fe..9447f81 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -873,7 +873,7 @@
             if self._consoleDisconnectAction == CDA.LOCK_SCREEN:
                 self.guestAgent.desktopLock()
             elif self._consoleDisconnectAction == CDA.LOGOUT:
-                self.guestAgent.desktopLogoff()
+                self.guestAgent.desktopLogoff(False)
             elif self._consoleDisconnectAction == CDA.REBOOT:
                 self.shutdown(delay=delay, reboot=True, timeout=timeout,
                               message='Scheduled reboot on disconnect',


-- 
To view, visit https://gerrit.ovirt.org/59866
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I65b030d4a7a7ed7ad3f6279c66120d1f9e45d396
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeen...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Reply via email to