Francesco Romani has uploaded a new change for review. Change subject: vm: fix 'port' reporting with SSL enabled ......................................................................
vm: fix 'port' reporting with SSL enabled If spice is configured with ssl enabled, the 'port' attribute may be missing. This patch fixes that. Change-Id: If0cfab79f8f353957a43dd7f4bf908a51a5dca57 Bug-Url: https://bugzilla.redhat.com/1099425 Signed-off-by: Francesco Romani <from...@redhat.com> --- M vdsm/virt/vm.py 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/24/27924/1 diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py index ca4029a..c12f188 100644 --- a/vdsm/virt/vm.py +++ b/vdsm/virt/vm.py @@ -5060,7 +5060,8 @@ def _updateLegacyConf(self, dev): self.conf['display'] = 'qxl' if dev['device'] == 'spice' else 'vnc' - self.conf['displayPort'] = dev['port'] + if 'port' in dev: + self.conf['displayPort'] = dev['port'] if 'tlsPort' in dev: self.conf['displaySecurePort'] = dev['tlsPort'] -- To view, visit http://gerrit.ovirt.org/27924 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: If0cfab79f8f353957a43dd7f4bf908a51a5dca57 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <from...@redhat.com> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches