Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WFLY-1896, WFLY1913 - CLI 'module remove' command now deletes files recursively #4934

Closed
wants to merge 2 commits into from

Conversation

istudens
Copy link
Contributor

@wildfly-ci
Copy link

Build 470 is now running using a merge of 8e9d2d1

@wildfly-ci
Copy link

Build 470 outcome was FAILURE using a merge of 8e9d2d1
Summary: Tests failed: 1 (1 new), passed: 4399, ignored: 84 Build time: 1:29:26

Failed tests

org.jboss.as.test.clustering.cluster.ejb3.security.FailoverWithSecurityTestCase(SYNC-tcp).testDomainSecurityAnnotation: java.lang.IllegalStateException: EJBCLIENT000025: No EJB receiver available for handling [appName:, moduleName:cluster-security-domain-test, distinctName:] combination for invocation context org.jboss.ejb.client.EJBClientInvocationContext@1f9ef0c
    at org.jboss.ejb.client.EJBClientContext.requireEJBReceiver(EJBClientContext.java:743)
    at org.jboss.ejb.client.ReceiverInterceptor.handleInvocation(ReceiverInterceptor.java:116)

@wildfly-ci
Copy link

Build 204 is now running using a merge of 8e9d2d1

@wildfly-ci
Copy link

Build 204 outcome was FAILURE using a merge of 8e9d2d1
Summary: Tests failed: 1 (1 new), passed: 4207, ignored: 81 Build time: 1:17:49

Failed tests

org.jboss.as.test.integration.security.loginmodules.LdapExtLoginModuleTestCase.test2throw: java.lang.AssertionError: Unexpected status code returned after the authentication. expected:<200> but was:<401>
    at org.junit.Assert.fail(Assert.java:88)
    at org.junit.Assert.failNotEquals(Assert.java:743)

@bstansberry
Copy link
Contributor

When I attempted to merge this along with #4927 and #4936, the tests failed:

http://teamcity.cafe-babe.org/viewLog.html?buildId=6331&buildTypeId=bt13

@istudens
Copy link
Contributor Author

This looks like a bug in 'module add' command on MS Windows. I will look at it.
Thanks.

@istudens
Copy link
Contributor Author

retest this please

@wildfly-ci
Copy link

Build 241 is now running using a merge of 8998ee5

@wildfly-ci
Copy link

Build 241 outcome was SUCCESS using a merge of 8998ee5
Summary: Tests passed: 4403, ignored: 84 Build time: 1:9:18

@bstansberry
Copy link
Contributor

Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants