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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Alex Lourie <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Ofer Schreiber <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
