Francesco Romani has submitted this change and it was merged.

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
Bug-Url: https://bugzilla.redhat.com/1371264
Signed-off-by: Vinzenz Feenstra <vfeen...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/59866
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <from...@redhat.com>
Reviewed-by: Milan Zamazal <mzama...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/63972
Continuous-Integration: Francesco Romani <from...@redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Vinzenz Feenstra: Verified
  Francesco Romani: Looks good to me, approved; Passed CI tests
  Milan Zamazal: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I65b030d4a7a7ed7ad3f6279c66120d1f9e45d396
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Vinzenz Feenstra <vfeen...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to