[jboss-jira] [JBoss JIRA] (WFLY-3545) Boot time only system properties should not modify the runtime only the model

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}