Change in ovirt-hosted-engine-setup[master]: packaging: setup: detect display port at runtime

[sbonazzo at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Leonid Natapov <leonidnatapov at gmail.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server