Alon Bar-Lev has posted comments on this change.

Change subject: setup: Adds service configuration based on provided JBoss
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.ovirt.org/#/c/40152/1/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/jboss.py
File packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/jboss.py:

Line 97:                         oengcommcons.ConfigEnv.JBOSS_HOME
Line 98:                     ],
Line 99:                 ),
Line 100:                 '-v',
Line 101:             ),
we must not run standalone as it will create indexes, logs and other stuff and 
probably fail.

please run the standalone with -v, it is the same as the current jboss 
execution only with -v instead of -c.

++ java '-D[Standalone]' -server -XX:+UseCompressedOops -Xms1303m -Xmx1303m 
-XX:MaxPermSize=256m -Djava.net.preferIPv4Stack=true 
-Djboss.modules.system.pkgs=org.jboss.byteman -Djava.awt.headless=true 
-Dorg.jboss.boot.log.file=/usr/share/jbossas/standalone/log/server.log 
-Dlogging.configuration=file:/usr/share/jbossas/standalone/configuration/logging.properties
 -jar /usr/share/jbossas/jboss-modules.jar -mp /usr/share/jbossas/modules 
-jaxpmodule javax.xml.jaxp-provider org.jboss.as.standalone 
-Djboss.home.dir=/usr/share/jbossas 
-Djboss.server.base.dir=/usr/share/jbossas/standalone -v
Line 102:             # We need to bypass return code check, because by default
Line 103:             # standalone.sh -v returns 1
Line 104:             raiseOnError=False,
Line 105:             envAppend={


-- 
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: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Martin Peřina <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to