Permalink
Comparing changes
Open a pull request
10
contributors
Commits on Mar 21, 2019
Commits on Mar 24, 2019
Commits on Apr 09, 2019
…y.ssl.SSLSocketImpl. This is related to https://bugs.openjdk.java.net/browse/JDK-8219658
Commits on Apr 12, 2019
…ures generated by Galleon The ReadFeatureDescriptionHandler is assuming that requestProperties variable is not empty to process the fixed requirements of the registrations capabilities Jira issue: https://issues.jboss.org/browse/WFCORE-4416
Also fixes a couple minor IDE warns in the file.
This update brings in the following fixes: * [MODULES-387] Add JMX operation to get a class location * [MODULES-386] Fixing Javadoc build issues on JDK11 * [MODULES-375] Check if the artifact was successfully resolved before attempting to create the loader. * [MODULES-372] provide our own "java.se" module on JDK 9+
Commits on Apr 14, 2019
[WFCORE-3185] Run parallel boot tasks in coarser grained chunks
Update service.bat
[WFCORE-4129] WFLYSRV0266: Server home is set to... info msg in domain for RPM installation
Fix for WFCORE-4402, Upgrade CLI to use aesh 2.3
[WFCORE-4409] Avoid hangs from https://bugs.openjdk.java.net/browse/JDK-8219658
[WFCORE-4410] Upgrade Elytron Web to 1.5.0.CR1
[WFCORE-4413] Fixing javax.api and javax.sql.api modules BC issues
[WFCORE-4418] Deprecate unused ParseUtil methods.
[WFCORE-4420] Fix check for XMLConstants.NULL_NS_URI
[WFCORE-4416] Some capabilities requirements are not recorded in features generated by Galleon
[WFCORE-4415] Upgrading JBoss Modules from 1.9.0.Final to 1.9.1.Final
Commits on Apr 17, 2019
…ityClassificationWriteAttributeHandler rollback handling
…uld have been validated before execution.
Unified
Split
Showing
with
1,649 additions
and 260 deletions.
- +20 −0 cli/src/main/java/org/jboss/as/cli/impl/aesh/CLICommandContainer.java
- +10 −0 cli/src/main/java/org/jboss/as/cli/impl/aesh/cmd/operation/LegacyCommandContainer.java
- +10 −0 cli/src/main/java/org/jboss/as/cli/impl/aesh/cmd/operation/OperationCommandContainer.java
- +12 −1 controller/src/main/java/org/jboss/as/controller/AbstractControllerService.java
- +9 −2 controller/src/main/java/org/jboss/as/controller/CapabilityServiceBuilder.java
- +10 −3 controller/src/main/java/org/jboss/as/controller/ModelControllerImpl.java
- +6 −0 controller/src/main/java/org/jboss/as/controller/OperationContextImpl.java
- +147 −43 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
- +76 −0 controller/src/main/java/org/jboss/as/controller/extension/ExtensionRegistry.java
- +71 −23 controller/src/main/java/org/jboss/as/controller/extension/ParallelExtensionAddHandler.java
- +9 −8 controller/src/main/java/org/jboss/as/controller/logging/ControllerLogger.java
- +43 −42 ...roller/src/main/java/org/jboss/as/controller/operations/global/ReadFeatureDescriptionHandler.java
- +78 −26 controller/src/main/java/org/jboss/as/controller/parsing/DeferredExtensionContext.java
- +2 −2 controller/src/main/java/org/jboss/as/controller/parsing/ExtensionXml.java
- +8 −3 controller/src/main/java/org/jboss/as/controller/parsing/ParseUtils.java
- +4 −0 core-feature-pack/pom.xml
- +715 −0 core-feature-pack/src/license/core-feature-pack-licenses.xml
- +0 −1 core-feature-pack/src/main/resources/content/bin/common.bat
- +0 −1 core-feature-pack/src/main/resources/content/bin/common.ps1
- +0 −1 core-feature-pack/src/main/resources/content/bin/common.sh
- +50 −3 core-feature-pack/src/main/resources/content/docs/contrib/scripts/service/service.bat
- +1 −0 core-feature-pack/src/main/resources/modules/system/layers/base/javax/api/main/module.xml
- +1 −0 core-feature-pack/src/main/resources/modules/system/layers/base/javax/sql/api/main/module.xml
- +1 −0 ...rc/main/resources/modules/system/layers/base/org/wildfly/security/elytron-private/main/module.xml
- +4 −0 core-galleon-pack/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
- +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/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
- +2 −1 host-controller/src/main/java/org/jboss/as/domain/controller/resources/DomainRootDefinition.java
- +8 −0 host-controller/src/main/java/org/jboss/as/host/controller/DomainModelControllerService.java
- +0 −1 host-controller/src/main/java/org/jboss/as/host/controller/jvm/JvmType.java
- +2 −1 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 host-controller/src/test/java/org/jboss/as/host/controller/ManagedServerBootCmdFactoryTestCase.java
- +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
- +0 −1 launcher/src/main/java/org/wildfly/core/launcher/AbstractCommandBuilder.java
- +0 −2 launcher/src/test/java/org/wildfly/core/launcher/CommandBuilderTest.java
- +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 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
- +17 −12 pom.xml
- +2 −1 remoting/subsystem/src/main/java/org/jboss/as/remoting/RemotingSubsystemAdd.java
- +0 −9 server/src/main/java/org/jboss/as/server/ServerEnvironment.java
- +13 −2 server/src/main/java/org/jboss/as/server/ServerService.java
- +3 −3 server/src/main/java/org/jboss/as/server/logging/ServerLogger.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
- +2 −1 ...ite/manualmode/src/test/java/org/jboss/as/test/manualmode/auditlog/AuditLogBootingSyslogTest.java
- +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
- +2 −1 .../standalone/src/test/java/org/jboss/as/test/integration/auditlog/AuditLogToTLSSyslogTestCase.java
| @@ -161,6 +161,16 @@ public void complete(AeshCompleteOperation completeOperation, ParsedLine line, I | ||
| public void doPopulate(ProcessedCommand processedCommand, InvocationProviders invocationProviders, AeshContext aeshContext, Mode mode) throws CommandLineParserException, OptionValidatorException { | ||
| parser.doPopulate(processedCommand, invocationProviders, aeshContext, mode); | ||
| } | ||
|
|
||
| @Override | ||
| public void updateAnsiMode(boolean mode) { | ||
| parser.updateAnsiMode(mode); | ||
| } | ||
|
|
||
| @Override | ||
| public String getFormattedCommand(int offset, int descriptionStart) { | ||
| return parser.getFormattedCommand(offset, descriptionStart); | ||
| } | ||
| } | ||
|
|
||
| public class CLICommandParser implements CommandLineParser<CLICommandInvocation> { | ||
| @@ -273,6 +283,16 @@ public void complete(AeshCompleteOperation completeOperation, ParsedLine line, I | ||
| public void doPopulate(ProcessedCommand processedCommand, InvocationProviders invocationProviders, AeshContext aeshContext, Mode mode) throws CommandLineParserException, OptionValidatorException { | ||
| container.getParser().doPopulate(processedCommand, invocationProviders, aeshContext, mode); | ||
| } | ||
|
|
||
| @Override | ||
| public void updateAnsiMode(boolean mode) { | ||
| container.getParser().updateAnsiMode(mode); | ||
| } | ||
|
|
||
| @Override | ||
| public String getFormattedCommand(int offset, int descriptionStart) { | ||
| return container.getParser().getFormattedCommand(offset, descriptionStart); | ||
| } | ||
| } | ||
|
|
||
| private final CommandContainer<CLICommandInvocation> container; | ||
| @@ -192,6 +192,16 @@ public void complete(AeshCompleteOperation completeOperation, ParsedLine line, I | ||
| @Override | ||
| public void doPopulate(ProcessedCommand processedCommand, InvocationProviders invocationProviders, AeshContext aeshContext, Mode mode) throws CommandLineParserException, OptionValidatorException { | ||
| } | ||
|
|
||
| @Override | ||
| public void updateAnsiMode(boolean mode) { | ||
|
|
||
| } | ||
|
|
||
| @Override | ||
| public String getFormattedCommand(int offset, int descriptionStart) { | ||
| return null; | ||
| } | ||
| } | ||
|
|
||
| private final Command<CLICommandInvocation> command = new LegacyCommand(); | ||
| @@ -186,6 +186,16 @@ public void complete(AeshCompleteOperation completeOperation, ParsedLine line, I | ||
| @Override | ||
| public void doPopulate(ProcessedCommand processedCommand, InvocationProviders invocationProviders, AeshContext aeshContext, Mode mode) throws CommandLineParserException, OptionValidatorException { | ||
| } | ||
|
|
||
| @Override | ||
| public void updateAnsiMode(boolean mode) { | ||
|
|
||
| } | ||
|
|
||
| @Override | ||
| public String getFormattedCommand(int offset, int descriptionStart) { | ||
| return null; | ||
| } | ||
| } | ||
|
|
||
| private final Command<CLICommandInvocation> command = new OperationCommand(); | ||
| @@ -334,7 +334,10 @@ public void start(final StartContext context) throws StartException { | ||
| rootResourceRegistration, | ||
| new ContainerStateMonitor(container, getStabilityMonitor()), | ||
| configurationPersister, processType, runningModeControl, prepareStep, | ||
| processState, executorService, expressionResolver, authorizer, securityIdentitySupplier, auditLogger, notificationSupport, | ||
| processState, executorService, | ||
| getMaxParallelBootExtensionTasks(), | ||
| getMaxParallelBootSubsystemTasks(), | ||
| expressionResolver, authorizer, securityIdentitySupplier, auditLogger, notificationSupport, | ||
| bootErrorCollector, createExtraValidationStepHandler(), capabilityRegistry, getPartialModelIndicator(), | ||
| injectedInstabilityListener.getOptionalValue()); | ||
|
|
||
| @@ -395,6 +398,14 @@ 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,6 +125,8 @@ | ||
| 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; | ||
| @@ -153,6 +155,8 @@ | ||
| 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, | ||
| @@ -186,6 +190,8 @@ | ||
| 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; | ||
| @@ -600,9 +606,10 @@ 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 ? null : new ParallelExtensionAddHandler(executorService, parallellBRRRProvider); | ||
| ParallelBootOperationStepHandler parallelSubsystemHandler = (executorService != null && processType.isServer() && runningModeControl.getRunningMode() == RunningMode.NORMAL) | ||
| ? new ParallelBootOperationStepHandler(executorService, rootRegistration, processState, this, lockPermit, extraValidationStepHandler) : null; | ||
| 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; | ||
| 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.