Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: setup: detect display port at runtime
......................................................................
packaging: setup: detect display port at runtime
previously the tool wrongly assumed that the SPICE and
the VNC port was always 5900.
It may happen however that secureDisplayPort is set to 5901
instead of 5900.
Now displayPort and secureDisplayPort are detected at runtime
querying the status of the VM.
Firewall ports 5900 and 5901 are now both open.
Change-Id: I8ae46b061b2a3ad0190f798cb141a41431120839
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/vm/runvm.py
M templates/hosted-console.xml.in
2 files changed, 29 insertions(+), 3 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/19869
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8ae46b061b2a3ad0190f798cb141a41431120839
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server