Permalink
Comparing changes
Open a pull request
6
contributors
Commits on Apr 15, 2019
…ityClassificationWriteAttributeHandler rollback handling
…uld have been validated before execution.
Commits on Apr 17, 2019
Commits on Apr 24, 2019
Unified
Split
Showing
with
1,244 additions
and 574 deletions.
- +1 −1 build-legacy/pom.xml
- +1 −1 build/pom.xml
- +1 −1 cli/pom.xml
- +4 −2 cli/src/main/java/org/jboss/as/cli/embedded/ThreadLocalContextSelector.java
- +1 −1 component-matrix-builder/pom.xml
- +1 −1 controller-client/pom.xml
- +1 −1 controller/pom.xml
- +1 −12 controller/src/main/java/org/jboss/as/controller/AbstractControllerService.java
- +9 −2 controller/src/main/java/org/jboss/as/controller/CapabilityServiceBuilder.java
- +3 −10 controller/src/main/java/org/jboss/as/controller/ModelControllerImpl.java
- +6 −0 controller/src/main/java/org/jboss/as/controller/OperationContextImpl.java
- +43 −147 controller/src/main/java/org/jboss/as/controller/ParallelBootOperationStepHandler.java
- +38 −0 controller/src/main/java/org/jboss/as/controller/access/constraint/AbstractSensitivity.java
- +1 −1 controller/src/main/java/org/jboss/as/controller/extension/ExtensionAddHandler.java
- +0 −76 controller/src/main/java/org/jboss/as/controller/extension/ExtensionRegistry.java
- +23 −71 controller/src/main/java/org/jboss/as/controller/extension/ParallelExtensionAddHandler.java
- +8 −9 controller/src/main/java/org/jboss/as/controller/logging/ControllerLogger.java
- +26 −78 controller/src/main/java/org/jboss/as/controller/parsing/DeferredExtensionContext.java
- +2 −2 controller/src/main/java/org/jboss/as/controller/parsing/ExtensionXml.java
- +5 −1 core-feature-pack/pom.xml
- +715 −0 core-feature-pack/src/license/core-feature-pack-licenses.xml
- +1 −0 ...rc/main/resources/modules/system/layers/base/org/wildfly/security/elytron-private/main/module.xml
- +5 −1 core-galleon-pack/pom.xml
- +1 −1 core-management/core-management-client/pom.xml
- +1 −1 core-management/core-management-subsystem/pom.xml
- +1 −1 core-management/pom.xml
- +1 −1 core-model-test/framework/pom.xml
- +1 −1 core-model-test/pom.xml
- +1 −1 core-model-test/test-controller-optional/pom.xml
- +1 −1 core-model-test/tests/pom.xml
- +8 −6 .../src/test/java/org/jboss/as/core/model/test/access/RuntimeSensitivityReconfigurationTestCase.java
- +95 −1 ...test/tests/src/test/java/org/jboss/as/core/model/test/access/StandaloneAccessControlTestCase.java
- +1 −0 core-model-test/tests/src/test/resources/org/jboss/as/core/model/test/access/standalone.xml
- +1 −1 core-security/api/pom.xml
- +1 −1 core-security/implementation/pom.xml
- +1 −1 core-security/pom.xml
- +1 −1 deployment-repository/pom.xml
- +1 −1 deployment-scanner/pom.xml
- +1 −1 discovery/pom.xml
- +1 −1 dist-legacy/pom.xml
- +1 −1 dist/pom.xml
- +1 −1 domain-http/error-context/pom.xml
- +1 −1 domain-http/interface/pom.xml
- +1 −1 domain-http/pom.xml
- +1 −1 domain-management/pom.xml
- +21 −13 ...java/org/jboss/as/domain/management/access/ApplicationClassificationConfigResourceDefinition.java
- +55 −4 ...management/src/main/java/org/jboss/as/domain/management/access/SensitivityResourceDefinition.java
- +5 −0 domain-management/src/main/java/org/jboss/as/domain/management/logging/DomainManagementLogger.java
- +1 −1 elytron/pom.xml
- +1 −1 elytron/src/main/java/org/wildfly/extension/elytron/HttpServerDefinitions.java
- +1 −1 elytron/src/main/java/org/wildfly/extension/elytron/KeyStoreService.java
- +3 −1 elytron/src/main/java/org/wildfly/extension/elytron/TrivialAddHandler.java
- +1 −1 embedded/pom.xml
- +1 −1 host-controller/pom.xml
- +1 −2 host-controller/src/main/java/org/jboss/as/domain/controller/resources/DomainRootDefinition.java
- +0 −8 host-controller/src/main/java/org/jboss/as/host/controller/DomainModelControllerService.java
- +1 −2 host-controller/src/main/java/org/jboss/as/host/controller/model/host/HostResourceDefinition.java
- +2 −1 host-controller/src/main/java/org/jboss/as/host/controller/operations/HttpManagementAddHandler.java
- +1 −1 io/pom.xml
- +1 −1 io/subsystem/pom.xml
- +4 −2 io/subsystem/src/main/java/org/wildfly/extension/io/BufferPoolResourceDefinition.java
- +2 −1 io/subsystem/src/main/java/org/wildfly/extension/io/WorkerAdd.java
- +1 −1 io/tests/pom.xml
- +1 −1 jmx/pom.xml
- +1 −1 launcher/pom.xml
- +1 −1 logging/pom.xml
- +5 −0 logging/src/main/java/org/jboss/as/logging/formatters/PatternFormatterResourceDefinition.java
- +3 −0 logging/src/main/java/org/jboss/as/logging/formatters/StructuredFormatterResourceDefinition.java
- +15 −13 logging/src/main/java/org/jboss/as/logging/handlers/HandlerOperations.java
- +9 −0 logging/src/main/java/org/jboss/as/logging/logging/LoggingLogger.java
- +6 −5 logging/src/test/java/org/jboss/as/logging/HandlerOperationsTestCase.java
- +1 −1 management-client-content/pom.xml
- +1 −1 model-test/pom.xml
- +1 −1 model-test/src/main/java/org/jboss/as/model/test/ChildFirstClassLoaderBuilder.java
- +1 −1 model-test/src/main/java/org/jboss/as/model/test/MavenUtil.java
- +2 −2 model-test/src/test/java/org/jboss/as/model/test/MavenSettingsTest.java
- +2 −2 model-test/src/test/resources/org/jboss/as/model/test/settings-empty-local-repo.xml
- +1 −1 network/pom.xml
- +1 −1 patching/pom.xml
- +1 −1 platform-mbean/pom.xml
- +10 −5 pom.xml
- +1 −1 process-controller/pom.xml
- +1 −1 protocol/pom.xml
- +1 −1 remoting/pom.xml
- +1 −1 remoting/subsystem/pom.xml
- +2 −1 remoting/subsystem/src/main/java/org/jboss/as/remoting/RemotingSubsystemAdd.java
- +1 −1 remoting/tests/pom.xml
- +1 −1 request-controller/pom.xml
- +1 −1 security-manager/pom.xml
- +1 −1 server/pom.xml
- +2 −13 server/src/main/java/org/jboss/as/server/ServerService.java
- +3 −2 server/src/main/java/org/jboss/as/server/operations/HttpManagementAddHandler.java
- +2 −1 server/src/main/java/org/jboss/as/server/services/net/BindingAddHandler.java
- +2 −1 server/src/main/java/org/jboss/as/server/services/net/BindingGroupAddHandler.java
- +2 −1 ...c/main/java/org/jboss/as/server/services/net/LocalDestinationOutboundSocketBindingAddHandler.java
- +2 −1 .../main/java/org/jboss/as/server/services/net/RemoteDestinationOutboundSocketBindingAddHandler.java
- +2 −1 server/src/main/java/org/jboss/as/server/services/net/SpecifiedInterfaceAddHandler.java
- +1 −1 subsystem-test/framework/pom.xml
- +1 −1 subsystem-test/pom.xml
- +1 −1 subsystem-test/test-controller-optional/pom.xml
- +1 −1 subsystem-test/tests/pom.xml
- +1 −1 testsuite/client-old-server/pom.xml
- +1 −1 testsuite/domain/pom.xml
- +1 −1 testsuite/elytron/pom.xml
- +1 −1 testsuite/embedded/pom.xml
- +1 −1 testsuite/layers/pom.xml
- +1 −1 testsuite/manualmode/pom.xml
- +1 −1 testsuite/patching/pom.xml
- +1 −1 testsuite/pom.xml
- +1 −1 testsuite/rbac/pom.xml
- +8 −1 .../src/test/java/org/jboss/as/test/integration/mgmt/access/ConfigurationChangesHistoryTestCase.java
- +8 −1 ...est/java/org/jboss/as/test/integration/mgmt/access/LegacyConfigurationChangesHistoryTestCase.java
- +1 −1 testsuite/scripts/pom.xml
- +1 −1 testsuite/shared/pom.xml
- +1 −1 testsuite/standalone/pom.xml
- +1 −1 testsuite/test-runner/pom.xml
- +1 −1 testsuite/vault-test-feature-pack/pom.xml
- +1 −1 threads/pom.xml
- +1 −1 version/pom.xml
| @@ -30,7 +30,7 @@ | ||
| <parent> | ||
| <groupId>org.wildfly.core</groupId> | ||
| <artifactId>wildfly-core-parent</artifactId> | ||
| <version>9.0.0.Beta3-SNAPSHOT</version> | ||
| <version>9.0.0.Beta4-SNAPSHOT</version> | ||
| </parent> | ||
|
|
||
| <artifactId>wildfly-core-build-legacy</artifactId> | ||
| @@ -30,7 +30,7 @@ | ||
| <parent> | ||
| <groupId>org.wildfly.core</groupId> | ||
| <artifactId>wildfly-core-parent</artifactId> | ||
| <version>9.0.0.Beta3-SNAPSHOT</version> | ||
| <version>9.0.0.Beta4-SNAPSHOT</version> | ||
| </parent> | ||
|
|
||
| <artifactId>wildfly-core-build</artifactId> | ||
| @@ -29,7 +29,7 @@ | ||
| <parent> | ||
| <groupId>org.wildfly.core</groupId> | ||
| <artifactId>wildfly-core-parent</artifactId> | ||
| <version>9.0.0.Beta3-SNAPSHOT</version> | ||
| <version>9.0.0.Beta4-SNAPSHOT</version> | ||
| </parent> | ||
|
|
||
| <artifactId>wildfly-cli</artifactId> | ||
| @@ -69,7 +69,8 @@ void restore(Contexts toRestore) { | ||
| @Override | ||
| public StdioContext getStdioContext() { | ||
| // CLI loggers should only use the default stdio context regardless if the thread-local context is set. | ||
| if (WildFlySecurityManager.getCurrentContextClassLoaderPrivileged().equals(cliClassLoader)) { | ||
| final ClassLoader tccl = WildFlySecurityManager.getCurrentContextClassLoaderPrivileged(); | ||
| if (tccl != null && tccl.equals(cliClassLoader)) { | ||
| return defaultContexts.getStdioContext(); | ||
| } | ||
| Contexts threadContext = threadLocal.get(); | ||
| @@ -81,7 +82,8 @@ public StdioContext getStdioContext() { | ||
| public LogContext getLogContext() { | ||
| // CLI loggers should only use the default stdio context regardless if the thread-local context is set This | ||
| // allows the context configured for CLI, e.g. jboss-cli-logging.properties. | ||
| if (WildFlySecurityManager.getCurrentContextClassLoaderPrivileged().equals(cliClassLoader)) { | ||
| final ClassLoader tccl = WildFlySecurityManager.getCurrentContextClassLoaderPrivileged(); | ||
| if (tccl != null && tccl.equals(cliClassLoader)) { | ||
| return defaultContexts.getLogContext(); | ||
| } | ||
| Contexts threadContext = threadLocal.get(); | ||
| @@ -26,7 +26,7 @@ | ||
| Maintain separation between the artifact id and the version to help prevent | ||
| merge conflicts between commits changing the GA and those changing the V. | ||
| --> | ||
| <version>9.0.0.Beta3-SNAPSHOT</version> | ||
| <version>9.0.0.Beta4-SNAPSHOT</version> | ||
| </parent> | ||
|
|
||
| <groupId>org.wildfly.core</groupId> | ||
| @@ -29,7 +29,7 @@ | ||
| <parent> | ||
| <groupId>org.wildfly.core</groupId> | ||
| <artifactId>wildfly-core-parent</artifactId> | ||
| <version>9.0.0.Beta3-SNAPSHOT</version> | ||
| <version>9.0.0.Beta4-SNAPSHOT</version> | ||
| </parent> | ||
|
|
||
| <artifactId>wildfly-controller-client</artifactId> | ||
| @@ -29,7 +29,7 @@ | ||
| <parent> | ||
| <groupId>org.wildfly.core</groupId> | ||
| <artifactId>wildfly-core-parent</artifactId> | ||
| <version>9.0.0.Beta3-SNAPSHOT</version> | ||
| <version>9.0.0.Beta4-SNAPSHOT</version> | ||
| </parent> | ||
|
|
||
| <artifactId>wildfly-controller</artifactId> | ||
| @@ -334,10 +334,7 @@ public void start(final StartContext context) throws StartException { | ||
| rootResourceRegistration, | ||
| new ContainerStateMonitor(container, getStabilityMonitor()), | ||
| configurationPersister, processType, runningModeControl, prepareStep, | ||
| processState, executorService, | ||
| getMaxParallelBootExtensionTasks(), | ||
| getMaxParallelBootSubsystemTasks(), | ||
| expressionResolver, authorizer, securityIdentitySupplier, auditLogger, notificationSupport, | ||
| processState, executorService, expressionResolver, authorizer, securityIdentitySupplier, auditLogger, notificationSupport, | ||
| bootErrorCollector, createExtraValidationStepHandler(), capabilityRegistry, getPartialModelIndicator(), | ||
| injectedInstabilityListener.getOptionalValue()); | ||
|
|
||
| @@ -398,14 +395,6 @@ public ServiceTarget getServiceTarget() { | ||
| bootThread.start(); | ||
| } | ||
|
|
||
| protected int getMaxParallelBootExtensionTasks() { | ||
| return 1; | ||
| } | ||
|
|
||
| protected int getMaxParallelBootSubsystemTasks() { | ||
| return 1; | ||
| } | ||
|
|
||
| /** | ||
| * Gets whether this controller service should install a {@link ModelControllerClientFactory} | ||
| * and a {@link org.jboss.as.controller.notification.NotificationHandlerRegistry} | ||
| @@ -20,7 +20,7 @@ | ||
|
|
||
| import org.jboss.as.controller.capability.RuntimeCapability; | ||
| import org.jboss.msc.inject.Injector; | ||
| import org.jboss.msc.service.Service; | ||
| import org.jboss.msc.Service; | ||
| import org.jboss.msc.service.ServiceBuilder; | ||
| import org.jboss.msc.service.ServiceController; | ||
| import org.jboss.msc.service.ServiceName; | ||
| @@ -29,7 +29,7 @@ | ||
|
|
||
| /** | ||
| * A builder for an individual service in a {@code CapabilityServiceTarget}. | ||
| * Create an instance via the {@link CapabilityServiceTarget#addCapability(RuntimeCapability, Service)}, | ||
| * Create an instance via the {@link CapabilityServiceTarget#addCapability(RuntimeCapability)}, | ||
| * {@link #addCapabilityRequirement(String, Class, Injector)} or other methods. | ||
| * Builder also add supports to add capability requirement for service injection via {@link #addCapabilityRequirement(String, Class, Injector)} | ||
| * | ||
| @@ -103,6 +103,13 @@ | ||
| @Override | ||
| CapabilityServiceBuilder<T> setInitialMode(ServiceController.Mode mode); | ||
|
|
||
| /** | ||
| * {@inheritDoc} | ||
| * @return this builder | ||
| */ | ||
| @Override | ||
| CapabilityServiceBuilder<T> setInstance(Service service); | ||
|
|
||
| /** | ||
| * Capability requirement. | ||
| * | ||
| @@ -125,8 +125,6 @@ | ||
| private final OperationStepHandler prepareStep; | ||
| private final ControlledProcessState processState; | ||
| private final ExecutorService executorService; | ||
| private final int maxParallelBootExtensionTasks; | ||
| private final int maxParallelBootSubsystemTasks; | ||
| private final ExpressionResolver expressionResolver; | ||
| private final Authorizer authorizer; | ||
| private final Supplier<SecurityIdentity> securityIdentitySupplier; | ||
| @@ -155,8 +153,6 @@ | ||
| final ContainerStateMonitor stateMonitor, final ConfigurationPersister persister, | ||
| final ProcessType processType, final RunningModeControl runningModeControl, | ||
| final OperationStepHandler prepareStep, final ControlledProcessState processState, final ExecutorService executorService, | ||
| final int maxParallelBootExtensionTasks, | ||
| final int maxParallelBootSubsystemTasks, | ||
| final ExpressionResolver expressionResolver, final Authorizer authorizer, final Supplier<SecurityIdentity> securityIdentitySupplier, | ||
| final ManagedAuditLogger auditLogger, NotificationSupport notificationSupport, | ||
| final BootErrorCollector bootErrorCollector, final OperationStepHandler extraValidationStepHandler, | ||
| @@ -190,8 +186,6 @@ | ||
| this.processState = processState; | ||
| this.serviceTarget.addMonitor(stateMonitor.getStabilityMonitor()); | ||
| this.executorService = executorService; | ||
| this.maxParallelBootExtensionTasks = maxParallelBootExtensionTasks; | ||
| this.maxParallelBootSubsystemTasks = maxParallelBootSubsystemTasks; | ||
| assert expressionResolver != null; | ||
| this.expressionResolver = expressionResolver; | ||
| assert securityIdentitySupplier != null; | ||
| @@ -606,10 +600,9 @@ private BootOperations organizeBootOperations(List<ModelNode> bootList, final in | ||
| final ManagementResourceRegistration rootRegistration = managementModel.get().getRootResourceRegistration(); | ||
| final MutableRootResourceRegistrationProvider parallellBRRRProvider = parallelBootRootResourceRegistrationProvider != null ? | ||
| parallelBootRootResourceRegistrationProvider : getMutableRootResourceRegistrationProvider(); | ||
| ParallelExtensionAddHandler parallelExtensionAddHandler = executorService == null || maxParallelBootExtensionTasks < 2 ? null : new ParallelExtensionAddHandler(executorService, maxParallelBootExtensionTasks, parallellBRRRProvider); | ||
| ParallelBootOperationStepHandler parallelSubsystemHandler = (executorService != null && maxParallelBootSubsystemTasks > 1 && processType.isServer() && runningModeControl.getRunningMode() == RunningMode.NORMAL) | ||
| ? new ParallelBootOperationStepHandler(executorService, maxParallelBootSubsystemTasks, rootRegistration, processState, this, lockPermit, extraValidationStepHandler) | ||
| : null; | ||
| ParallelExtensionAddHandler parallelExtensionAddHandler = executorService == null ? null : new ParallelExtensionAddHandler(executorService, parallellBRRRProvider); | ||
| ParallelBootOperationStepHandler parallelSubsystemHandler = (executorService != null && processType.isServer() && runningModeControl.getRunningMode() == RunningMode.NORMAL) | ||
| ? new ParallelBootOperationStepHandler(executorService, rootRegistration, processState, this, lockPermit, extraValidationStepHandler) : null; | ||
| boolean registeredParallelSubsystemHandler = false; | ||
| int subsystemIndex = 0; | ||
| for (ModelNode bootOp : bootList) { | ||
| @@ -2639,6 +2639,12 @@ public ServiceName getCapabilityServiceName(String capabilityBaseName, String .. | ||
| return this; | ||
| } | ||
|
|
||
| @Override | ||
| public CapabilityServiceBuilder<T> setInstance(org.jboss.msc.Service service) { | ||
| super.setInstance(service); | ||
| return this; | ||
| } | ||
|
|
||
| @Override | ||
| public <V> Supplier<V> requiresCapability(String capabilityBaseName, Class<V> dependencyType, String... referenceNames) { | ||
| String capabilityName; | ||
Oops, something went wrong.