]
Jason Greene updated WFLY-3545:
-------------------------------
Fix Version/s: 9.0.0.Beta1
(was: 9.0.0.Alpha1)
Boot time only system properties should not modify the runtime only
the model
-----------------------------------------------------------------------------
Key: WFLY-3545
URL:
https://issues.jboss.org/browse/WFLY-3545
Project: WildFly
Issue Type: Bug
Components: Domain Management
Reporter: James Perkins
Assignee: James Perkins
Fix For: 8.2.0.CR1, 9.0.0.Beta1
The three properties {{jboss.server.base.dir}}, {{jboss.server.config.dir}} and
{{jboss.server.log.dir}} are allowed to be overridden for servers. They cannot be set as a
system-property resource, but can be overridden via {{JAVA_OPTS}} or the command line. For
a domain server these values need to be allowed to be set as a {{boot-time=true}} system
property.
The {{add}}, {{write-attribute}} and {{remove}} operations should not modify the runtime,
but should modify the model. It's debatable whether the server should be put in a
{{restart-required}} state. Currently similar resources do _not_ set the server state to
{{restart-required}}. For now we should stick with the same state.
{code}
[domain@localhost:9999 /]
/host=master/system-property=jboss.server.log.dir:add(boot-time=true,value="/var/log")
{
"outcome" => "failed",
"result" => undefined,
"failure-description" => "JBAS010839: Operation failed or was
rolled back on all servers.",
"rolled-back" => true,
"server-groups" => {"main-server-group" =>
{"host" => {"master" => {
"server-one" => {"response" => {
"outcome" => "failed",
"failure-description" => "JBAS015845: System property
jboss.server.log.dir cannot be set via the xml configuration file or from a management
client; it's value must be known at initial process start so it can only set from the
commmand line",
"rolled-back" => true
}},
"server-two" => {"response" => {
"outcome" => "failed",
"failure-description" => "JBAS015845: System property
jboss.server.log.dir cannot be set via the xml configuration file or from a management
client; it's value must be known at initial process start so it can only set from the
commmand line",
"rolled-back" => true
}}
}}}}
}
{code}