Dan Kenigsberg has submitted this change and it was merged.

Change subject: vdsm: pre-defined range for spice/vnc ports
......................................................................


vdsm: pre-defined range for spice/vnc ports

Libvirt's default port range 5900-65535 which is out of VDSM's
documented range 5634 to 6166. But since the medieval time we
already use libvirt's default. We'll define the new range as
5900-6923 (1024 ports) So:

1. Set some limited port range, that can be used in FW conf
2. Use start port 5900 as it is already used by customers
3. Use the endport as 6923
4. Change VDSM documentation according to new port range

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=983088
Signed-off-by: Peter V. Saveliev <p...@redhat.com>
Signed-off-by: Vinzenz Feenstra <vfeen...@redhat.com>
Change-Id: Id42d41064e122f1b2dcc98506b58d0cc74239650
Reviewed-on: http://gerrit.ovirt.org/17058
Reviewed-by: Michal Skrivanek <michal.skriva...@redhat.com>
Tested-by: Michal Skrivanek <michal.skriva...@redhat.com>
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
---
M lib/vdsm/tool/libvirt_configure.sh.in
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved
  Michal Skrivanek: Verified; Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/17058
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id42d41064e122f1b2dcc98506b58d0cc74239650
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Peter V. Saveliev <p...@redhat.com>
Gerrit-Reviewer: Alex Lourie <alou...@redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com>
Gerrit-Reviewer: Peter V. Saveliev <p...@redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to