Change in ovirt-engine[master]: core, packaging: Don't use JBoss default ports (#831193)

[juan.hernandez at redhat.com]

Juan Hernandez has submitted this change and it was merged.

Change subject: core, packaging: Don't use JBoss default ports (#831193)
......................................................................


core, packaging: Don't use JBoss default ports (#831193)

https://bugzilla.redhat.com/831193

This patch changes the default ports used by the engine so that
they don't conflict with the default ports used by JBoss. The port
numbers proposed are the following:

  http: 8700
  https: 8701
  ajp: 8702
  remoting: 8703
  txn-recovery: 8704
  txn-status: 8705

Why these port numbers and not others?

1. They are not allocated according to /etc/services.

2. They are reasonably low.

3. The 8700-8731 range is not allocated by IANA, so we could
request them for ovirt-engine, to reduce the chances of conflicts
in the future.

4. 8700 is reasonabley easy to remember.

5. They are reasonably far away from the ranges used by VNC and spice
displays of virtual machines.

Change-Id: I3c0ffcab66fe9ae6f3dcb932040feb4421e18bf6
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M backend/manager/conf/engine.conf.defaults
M backend/manager/conf/standalone.xml
M backend/manager/dbscripts/config_devel.sql
M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M backend/manager/modules/restapi/interface/common/jaxrs/src/test/java/org/ovirt/engine/api/common/util/LinkHelperTest.java
M backend/manager/tools/engine-notifier/engine-notifier-service/src/test/resources/conf/error-notifier.conf
M backend/manager/tools/engine-notifier/engine-notifier-service/src/test/resources/conf/notifier.conf
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Configurator.java
M frontend/webadmin/modules/userportal-gwtp/pom.xml
M frontend/webadmin/modules/webadmin/pom.xml
M packaging/fedora/engine-service.xml.in
M packaging/fedora/setup/basedefs.py
M packaging/fedora/setup/common_utils.py
M packaging/fedora/setup/engine_validators.py
M packaging/fedora/setup/ovirt_port80.py
16 files changed, 36 insertions(+), 36 deletions(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3c0ffcab66fe9ae6f3dcb932040feb4421e18bf6
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server