Alon Bar-Lev has posted comments on this change. Change subject: core: Adds service configuration based on provided JBoss ......................................................................
Patch Set 5: (3 comments) https://gerrit.ovirt.org/#/c/40152/5/packaging/services/ovirt-engine/ovirt-engine.xml.in File packaging/services/ovirt-engine/ovirt-engine.xml.in: Line 491: <expose-resolved-model/> Line 492: <expose-expression-model/> Line 493: <remoting-connector/> Line 494: </subsystem> Line 495: #end if > The above is the default as defined in standalone.xml of WildFly 8. I will ok, this is the interesting part, how to make this service the lowest common feature set. Line 496: Line 497: #if $JBOSS_MAJOR < 7 Line 498: <subsystem xmlns="urn:jboss:domain:jpa:1.0"> Line 499: <jpa default-datasource=""/> Line 533: </login-module> Line 534: #if $JBOSS_MAJOR >= 7 Line 535: <login-module code="RealmDirect" flag="required"> Line 536: <module-option name="password-stacking" value="useFirstPass"/> Line 537: </login-module> > It's default for WildFly a must? can the eap handle this definition without being referenced to remove conditionals? Line 538: #end if Line 539: </authentication> Line 540: </security-domain> Line 541: <security-domain name="jboss-web-policy" cache-type="default"> Line 741: port="$getinteger('ENGINE_HTTPS_PORT')" Line 742: #else Line 743: port="0" Line 744: #end if Line 745: /> > Sorry, but I don't understand. Could you please elaborate? we have template replace so you can: <socket-binding name="redirect-True" port="$getinteger('ENGINE_PROXY_HTTPS_PORT')"/> <socket-binding name="redirect-False" port="$getinteger('ENGINE_HTTPS_PORT')"/> and reference this by: <ajp-listener name="ajp" socket-binding="ajp" redirect-socket="redirect-$getboolean('ENGINE_PROXY_ENABLED')" /> Line 746: #end if Line 747: Line 748: <socket-binding name="txn-recovery-environment" port="8704"/> Line 749: <socket-binding name="txn-status-manager" port="8705"/> -- To view, visit https://gerrit.ovirt.org/40152 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic35f8a0c276735b9685affea1e068f6ef7298f8c Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mper...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mper...@redhat.com> Gerrit-Reviewer: Moti Asayag <masa...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stira...@redhat.com> Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
