[jboss-jira] [JBoss JIRA] (WFCORE-837) Running tearDown() method of custom server setup task throws AssertionError in ActiveOperationSupport

[Vitalii Chepeliuk (JIRA)]

    [ https://issues.jboss.org/browse/WFCORE-837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13092648#comment-13092648 ] 

Vitalii Chepeliuk edited comment on WFCORE-837 at 7/24/15 7:58 AM:
-------------------------------------------------------------------

Yes [~brian.stansberry] it fails always. We have wildfly-arquillian [~ctomc] 1.0.0.Final as dependency in parent/pom.xml. I tried this and still not working. Two wildfly servers are started manually. I try dynamically reconfigure them and add security domains, trace logging, generating kerberos passwords etc. Problem could be when managementClient tries to connect to second server but first is already shutdowned and throws above issue because shutdown property is already set. Maybe there should be as many shutdown properties as servers available at moment   


was (Author: vchepeli):
Yes [~brian.stansberry] it fails always. We have wildfly-arquillian [~ctomc] 1.0.0.Final as dependency in parent/pom.xml. I tried this and still not working. 2 wildfly servers are started manually. I try dynamically reconfigure them and add security domains, trace logging, generating kerberos passwords etc. Problem could be when managementClient tries to connect to second server but first is already shutdowned and throws above issue because shutdown property is already set. Maybe there should be as many shutdown properties as servers available at moment   

> Running tearDown() method of custom server setup task throws AssertionError in ActiveOperationSupport 
> ------------------------------------------------------------------------------------------------------
>
>                 Key: WFCORE-837
>                 URL: https://issues.jboss.org/browse/WFCORE-837
>             Project: WildFly Core
>          Issue Type: Bug
>          Components: Domain Management
>    Affects Versions: 1.0.0.Final, 1.0.1.Final, 2.0.0.Alpha10
>         Environment: Arquillian, Wildfly-9.0.0.CR2
>            Reporter: Vitalii Chepeliuk
>            Assignee: ehsavoie Hugonnet
>
> Running tearDown() method of custom server setup tasks in AbstractNodeAuthentication abstract test:
>       KerberosSystemPropertiesSetupTask.class,
>       SecurityDomainsSetupTask.class,
>       SecurityTraceLoggingServerSetupTask.class,
>       KrbLdapServerSetupTask.class,
>       Krb5ConfServerSetupTask.class
> throws AssertionError. 
> Example of SecurityTraceLoggingServerSetupTask
> {code}
>     @Override
>     public void tearDown(ManagementClient managementClient, String containerId) throws Exception {
>         if (categories == null || categories.isEmpty()) {
>             return;
>         }
>         final List<ModelNode> updates = new ArrayList<ModelNode>();
>         for (String category : categories) {
>             if (category == null || category.length() == 0) {
>                 continue;
>             }
>             updates.add(Util.createRemoveOperation(PATH_LOGGING.append("logger", category)));
>         }
>         ModelNode op = Util.createEmptyOperation("write-attribute", PATH_LOGGING.append("console-handler", "CONSOLE"));
>         op.get("name").set("level");
>         op.get("value").set("INFO");
>         CoreUtils.applyUpdates(updates, managementClient.getControllerClient()); // Applying changes fails here
>     }
> {code}
> and in ActiveOperationSupport:151 shutdown property is already true but should be still false
> {code}
>     protected <T, A> ActiveOperation<T, A> registerActiveOperation(final Integer id, A attachment, ActiveOperation.CompletedCallback<T> callback) {
>         lock.lock(); try {
>             // Check that we still allow registration
>             assert ! shutdown; // Here it throws AssertionError
>             final Integer operationId;
>             if(id == null) {
>                 // If we did not get an operationId, create a new one
>                 operationId = operationIdManager.createBatchId();
>             } else {
>                 // Check that the operationId is not already taken
>                 if(! operationIdManager.lockBatchId(id)) {
>                     throw ProtocolLogger.ROOT_LOGGER.operationIdAlreadyExists(id);
>                 }
>                 operationId = id;
>             }
>             final ActiveOperationImpl<T, A> request = new ActiveOperationImpl<T, A>(operationId, attachment, getCheckedCallback(callback));
>             final ActiveOperation<?, ?> existing =  activeRequests.putIfAbsent(operationId, request);
>             if(existing != null) {
>                 throw ProtocolLogger.ROOT_LOGGER.operationIdAlreadyExists(operationId);
>             }
>             activeCount++; // condition.signalAll();
>             return request;
>         } finally {
>             lock.unlock();
>         }
>     }
> {code}
> Stack trace is following
> java.lang.AssertionError: null
>         at org.jboss.as.protocol.mgmt.ActiveOperationSupport.registerActiveOperation(ActiveOperationSupport.java:151)
>         at org.jboss.as.protocol.mgmt.ActiveOperationSupport.registerActiveOperation(ActiveOperationSupport.java:121)
>         at org.jboss.as.protocol.mgmt.ManagementChannelHandler.executeRequest(ManagementChannelHandler.java:121)
>         at org.jboss.as.controller.client.impl.AbstractModelControllerClient.executeRequest(AbstractModelControllerClient.java:263)
>         at org.jboss.as.controller.client.impl.AbstractModelControllerClient.execute(AbstractModelControllerClient.java:168)
>         at org.jboss.as.controller.client.impl.AbstractModelControllerClient.executeForResult(AbstractModelControllerClient.java:147)
>         at org.jboss.as.controller.client.impl.AbstractModelControllerClient.execute(AbstractModelControllerClient.java:80)
>         at org.jboss.as.test.integration.security.common.CoreUtils.applyUpdate(CoreUtils.java:177)
>         at org.jboss.as.test.integration.security.common.CoreUtils.applyUpdates(CoreUtils.java:172)
>         at org.infinispan.test.integration.security.tasks.AbstractTraceLoggingServer



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)