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-4295] Fix -ds.xml deprecation logging #7138

Merged
merged 1 commit into from Feb 4, 2015

Conversation

Projects
None yet
3 participants
@kabir
Contributor

kabir commented Feb 3, 2015

I found that the deprecation logging message was getting logged whether or not a deployment contains a -ds.xml. I have moved it to where I believe it would get triggered if it actually does.

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Linux Build 5833 is now running using a merge of 6e9e3b5

wildfly-ci commented Feb 3, 2015

Linux Build 5833 is now running using a merge of 6e9e3b5

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Linux with security manager Build 818 is now running using a merge of 6e9e3b5

wildfly-ci commented Feb 3, 2015

Linux with security manager Build 818 is now running using a merge of 6e9e3b5

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Windows Build 942 is now running using a merge of 6e9e3b5

wildfly-ci commented Feb 3, 2015

Windows Build 942 is now running using a merge of 6e9e3b5

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Linux with security manager Build 818 outcome was SUCCESS using a merge of 6e9e3b5
Summary: Tests passed: 839, ignored: 316 Build time: 0:05:10

wildfly-ci commented Feb 3, 2015

Linux with security manager Build 818 outcome was SUCCESS using a merge of 6e9e3b5
Summary: Tests passed: 839, ignored: 316 Build time: 0:05:10

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Windows Build 942 outcome was FAILURE using a merge of 6e9e3b5
Summary: Tests failed: 1, passed: 2920, ignored: 312 Build time: 0:47:34

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.integration.jpa.cfgfile.CfgFileTestCase: org.jboss.arquillian.container.spi.client.container.DeploymentException: Cannot deploy: jpa_cfgfile.jar
    at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:83)
    at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:64)

wildfly-ci commented Feb 3, 2015

Windows Build 942 outcome was FAILURE using a merge of 6e9e3b5
Summary: Tests failed: 1, passed: 2920, ignored: 312 Build time: 0:47:34

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.integration.jpa.cfgfile.CfgFileTestCase: org.jboss.arquillian.container.spi.client.container.DeploymentException: Cannot deploy: jpa_cfgfile.jar
    at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:83)
    at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:64)

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Linux Build 5833 outcome was FAILURE using a merge of 6e9e3b5
Summary: Tests failed: 1, passed: 2920, ignored: 312 Build time: 0:48:34

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.integration.jpa.cfgfile.CfgFileTestCase: org.jboss.arquillian.container.spi.client.container.DeploymentException: Cannot deploy: jpa_cfgfile.jar
    at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:83)
    at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:64)

wildfly-ci commented Feb 3, 2015

Linux Build 5833 outcome was FAILURE using a merge of 6e9e3b5
Summary: Tests failed: 1, passed: 2920, ignored: 312 Build time: 0:48:34

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.integration.jpa.cfgfile.CfgFileTestCase: org.jboss.arquillian.container.spi.client.container.DeploymentException: Cannot deploy: jpa_cfgfile.jar
    at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:83)
    at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:64)

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Linux with security manager Build 821 is now running using a merge of 74b555d

wildfly-ci commented Feb 3, 2015

Linux with security manager Build 821 is now running using a merge of 74b555d

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Windows Build 945 is now running using a merge of 74b555d

wildfly-ci commented Feb 3, 2015

Windows Build 945 is now running using a merge of 74b555d

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Linux Build 5836 is now running using a merge of 74b555d

wildfly-ci commented Feb 3, 2015

Linux Build 5836 is now running using a merge of 74b555d

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Linux with security manager Build 821 outcome was SUCCESS using a merge of 74b555d
Summary: Tests passed: 839, ignored: 316 Build time: 0:05:05

wildfly-ci commented Feb 3, 2015

Linux with security manager Build 821 outcome was SUCCESS using a merge of 74b555d
Summary: Tests passed: 839, ignored: 316 Build time: 0:05:05

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Windows Build 945 outcome was FAILURE using a merge of 74b555d
Summary: Tests failed: 1 (1 new), passed: 2921, ignored: 312 Build time: 0:45:44

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.integration.domain.suites.DeploymentManagementTestCase.testFullReplaceViaHash: java.lang.AssertionError: C:\BuildAgent\work\340a2b56aab3b1db\testsuite\domain\target\domains\DomainTestSuite\master\data\content\0f
    at org.junit.Assert.fail(Assert.java:88)
    at org.junit.Assert.assertTrue(Assert.java:41)

wildfly-ci commented Feb 3, 2015

Windows Build 945 outcome was FAILURE using a merge of 74b555d
Summary: Tests failed: 1 (1 new), passed: 2921, ignored: 312 Build time: 0:45:44

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.integration.domain.suites.DeploymentManagementTestCase.testFullReplaceViaHash: java.lang.AssertionError: C:\BuildAgent\work\340a2b56aab3b1db\testsuite\domain\target\domains\DomainTestSuite\master\data\content\0f
    at org.junit.Assert.fail(Assert.java:88)
    at org.junit.Assert.assertTrue(Assert.java:41)

@wildfly-ci

This comment has been minimized.

Show comment
Hide comment
@wildfly-ci

wildfly-ci Feb 3, 2015

Linux Build 5836 outcome was SUCCESS using a merge of 74b555d
Summary: Tests passed: 2922, ignored: 312 Build time: 0:48:43

wildfly-ci commented Feb 3, 2015

Linux Build 5836 outcome was SUCCESS using a merge of 74b555d
Summary: Tests passed: 2922, ignored: 312 Build time: 0:48:43

kabir added a commit that referenced this pull request Feb 4, 2015

Merge pull request #7138 from kabir/fix-ds-deprecation
[WFLY-4295] Fix -ds.xml deprecation logging

@kabir kabir merged commit ef6d9d0 into wildfly:master Feb 4, 2015

1 check passed

default TeamCity Build WildFly :: Pull Request :: Linux finished: Tests passed: 2922, ignored: 312
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment