Changes

Fixed NPE in ContainerMapper

- seen in server.log when running deployment_all tests

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>

Refactoring of sending HTTP 500 in ContainerMapper

- previous version had too wide try-catch blocks where in one of deployment_all
  test cases one exception caused another which was preventable.
- this version logs just the original exception and then a message if it cannot
  send the http 500 or another exception if it tried and it failed, but no NPE.

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>

For #24158 Asadmin set creates missing configuration nodes
Signed-off-by:Ondro Mihalyi <mihalyi@omnifish.ee>

#24158 Enhance existing test to test an aliased property.
Signed-off-by:Ondro Mihalyi <mihalyi@omnifish.ee>

#24158 Enhance existing test to test setting a custom property
Signed-off-by:Ondro Mihalyi <mihalyi@omnifish.ee>

#24158 Test setting an attribute on an undefined node.
A new node should be defined and attribute set on it.
Signed-off-by:Ondro Mihalyi <mihalyi@omnifish.ee>

#24158 Fix compile errors after an unsuccessful rebase
Clean up of the unit test.
Signed-off-by:Ondro Mihalyi <mihalyi@omnifish.ee>

Deprecated getDomainRoot removed

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>

Using org.glassfish.hk2.api.PopulatorPostProcessor instead of deprecated copy

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>

Fixed server restart - more robust, added validations

- Added host name validation with timeout
  - 5 seconds is an extreme value for the local server
- Honoring command-line arguments so they are not overwritten from domain.xml
  which could be updated since startup and doesn't contain correct values
  usable to stop the server.
- WARN: ServerDirs.serverName doesn't contain the server name, but instance
  name or domain name. Probably terminology yet from SunONE.
- Replaced JPS and JSTACK usages by ProcessHandle usages
- WARN: restart-domain leaves zombies on some systems
  - visible on Jenkins TCK
  - these processes are marked as <defunct> and operating system decides when
    they will be cleaned up. It probably relates to Jenkins which reads IO
    from the container, so stese streams remain always open.
- Fixed NPE in ClassPathUtils
- Added AfterAll to ClusterITest
- Added AS_TRACE support to admin-tests - bound to FINEST in tests
- UnknownHostException is now an error for asadmin, it doesn't mean that
  the server is stopped.
- Command line --host has higher priority than domain.xml and default value
  - However it is not always true, ie restart-domain must use the arg value
    for stop, but domain.xml value for start.
- Generalized waitFor, added time measurement.
- Using addSuppressed to keep all errors (visible with AS_TRACE)
- Fixed situation when there was no asadmin truststore - load null and use empty
- JavaClassRunner revisited
  - Avoid "oversanitization", use the same value as the last time
- ServerDirs revisited (quickly)
  - Added getter getLastPidFile and javadoc explaining the difference
  - pid file uses pid of running server, is deleted when server stops
  - pid.prev file uses pid of last started server, is not deleted, just overwritten
- Using Thread.yield when waiting rather than sleep
  - More assertive, even aggressive
  - Increases probablity of detection of misbehavior if there's any
  - Faster, we don't wait if not necessary

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>

Prepare release 7.0.0

Prepare next development cycle for 7.0.1-SNAPSHOT