10:11 a.m.
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(a)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(a)redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server