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

[WFCORE-787] Discard any management model changes before beginning ro… #853

Merged
merged 3 commits into from Jul 3, 2015

Conversation

bstansberry
Copy link
Contributor

…llback

See http://lists.jboss.org/pipermail/wildfly-dev/2015-July/004170.html

After James Perkins reported WFCORE-788, which looks to be a flaw with the WFCORE-787 fix, I debugged how operations were executing and realized the support in AbstractOperationContext for the no-longer used recursive execution mode was making the logic far too error prone. So I decided to clean that up, and filed WFCORE-792. The second commit in this PR is for that.

Finally, the 3rd commit fixes WFCORE-788 itself by fixing a problem in the CapabilityRegistry copy-on-write behavior where the copy wasn't deep enough, allowing state mutation to leak out of the OperationContext making modifications even if the copy is never published.

@wildfly-ci
Copy link

Windows Build 1622 is now running using a merge of 40168e5

@wildfly-ci
Copy link

Core - Full Integration Build 1218 is now running using a merge of 40168e5

@wildfly-ci
Copy link

Linux Build 2084 is now running using a merge of 40168e5

@wildfly-ci
Copy link

Windows Build 1622 outcome was SUCCESS using a merge of 40168e5
Summary: Tests passed: 3348, ignored: 62 Build time: 0:32:25

@wildfly-ci
Copy link

Linux Build 2084 outcome was SUCCESS using a merge of 40168e5
Summary: Tests passed: 3348, ignored: 62 Build time: 0:34:42

@wildfly-ci
Copy link

Core - Full Integration Build 1218 outcome was SUCCESS using a merge of 40168e5
Summary: Tests passed: 2924, ignored: 371, muted: 1 Build time: 0:47:51

@jamezp jamezp added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Jul 1, 2015
@bstansberry bstansberry added hold Do not merge this PR and removed ready-for-merge This PR is ready to be merged and fulfills all requirements labels Jul 2, 2015
@bstansberry
Copy link
Contributor Author

Hold while I sort WFCORE-788.

@wildfly-ci
Copy link

Windows Build 1627 is now running using a merge of 292e1b2

@wildfly-ci
Copy link

Core - Full Integration Build 1224 is now running using a merge of 292e1b2

@wildfly-ci
Copy link

Linux Build 2089 is now running using a merge of 292e1b2

@wildfly-ci
Copy link

Windows Build 1627 outcome was SUCCESS using a merge of 292e1b2
Summary: Tests passed: 3348, ignored: 62 Build time: 0:32:26

@wildfly-ci
Copy link

Core - Full Integration Build 1224 outcome was SUCCESS using a merge of 292e1b2
Summary: Tests passed: 2927, ignored: 371 Build time: 0:47:55

@wildfly-ci
Copy link

Linux Build 2089 outcome was FAILURE using a merge of 292e1b2
Summary: Execution timeout (new); tests passed: 3002, ignored: 47; org.wildfly.core:wildfly-core-testsuite-domain Build time: 1:00:08

Build problems:

Execution timeout
Process exited with code 137

@wildfly-ci
Copy link

Windows Build 1628 is now running using a merge of 5db8c3e

@wildfly-ci
Copy link

Core - Full Integration Build 1225 is now running using a merge of 5db8c3e

@wildfly-ci
Copy link

Linux Build 2090 is now running using a merge of 5db8c3e

@wildfly-ci
Copy link

Windows Build 1628 outcome was SUCCESS using a merge of 5db8c3e
Summary: Tests passed: 3348, ignored: 62 Build time: 0:32:22

@wildfly-ci
Copy link

Linux Build 2090 outcome was SUCCESS using a merge of 5db8c3e
Summary: Tests passed: 3348, ignored: 62 Build time: 0:34:02

@wildfly-ci
Copy link

Core - Full Integration Build 1225 outcome was SUCCESS using a merge of 5db8c3e
Summary: Tests passed: 2926, ignored: 371, muted: 1 Build time: 0:48:14

@wildfly-ci
Copy link

Windows Build 1629 is now running using a merge of 03628e0

@wildfly-ci
Copy link

Linux Build 2091 is now running using a merge of 03628e0

@wildfly-ci
Copy link

Core - Full Integration Build 1226 is now running using a merge of 03628e0

@bstansberry bstansberry removed the hold Do not merge this PR label Jul 2, 2015
@wildfly-ci
Copy link

Windows Build 1629 outcome was SUCCESS using a merge of 03628e0
Summary: Tests passed: 3348, ignored: 62 Build time: 0:32:45

@wildfly-ci
Copy link

Linux Build 2091 outcome was SUCCESS using a merge of 03628e0
Summary: Tests passed: 3348, ignored: 62 Build time: 0:34:20

@wildfly-ci
Copy link

Core - Full Integration Build 1226 outcome was FAILURE using a merge of 03628e0
Summary: Tests failed: 113 (113 new), passed: 2703, ignored: 367, muted: 1 Build time: 0:49:03

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.clustering.single.registry.RegistryTestCase: org.jboss.as.test.clustering.single.singleton.SingletonServiceTestCase: <no details avaliable>

org.jboss.as.test.clustering.single.registry.RegistryTestCase: org.jboss.as.test.clustering.single.provider.ServiceProviderRegistrationTestCase: <no details avaliable>

org.jboss.as.test.clustering.single.registry.RegistryTestCase: org.jboss.as.test.clustering.single.dispatcher.CommandDispatcherTestCase: <no details avaliable>

org.jboss.as.test.clustering.single.registry.RegistryTestCase: org.jboss.as.test.clustering.single.web.SimpleWebTestCase: <no details avaliable>

org.jboss.as.test.clustering.single.registry.RegistryTestCase: <no details avaliable>

org.jboss.as.test.clustering.messaging.ClusteredMessagingTestCase.testClusteredQueue: <no details avaliable>

org.jboss.as.test.clustering.messaging.ClusteredMessagingTestCase.testClusteredTopic: <no details avaliable>

org.jboss.as.test.clustering.xsite.XSiteSimpleTestCase.testPutRelayedToBackups: <no details avaliable>

org.jboss.as.test.clustering.xsite.XSiteSimpleTestCase.testPutToBackupNotRelayed: <no details avaliable>

org.jboss.as.test.multinode.remotecall.scoped.context.DynamicJNDIContextEJBInvocationTestCase: org.jboss.as.test.multinode.ejb.timer.database.DatabaseTimerServiceMultiNodeExecutionDisabledTestCase: <no details avaliable>

org.jboss.as.test.multinode.remotecall.scoped.context.DynamicJNDIContextEJBInvocationTestCase: org.jboss.as.test.multinode.ejb.timer.database.DatabaseTimerServiceMultiNodeTestCase: <no details avaliable>


##### there are 103 more failed tests, see build details

@wildfly-ci
Copy link

Core - Full Integration Build 1227 is now running using a merge of 03628e0

@wildfly-ci
Copy link

Core - Full Integration Build 1227 outcome was SUCCESS using a merge of 03628e0
Summary: Tests passed: 2928, ignored: 367, muted: 1 Build time: 0:48:24

@jamezp jamezp added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Jul 2, 2015
bstansberry added a commit that referenced this pull request Jul 3, 2015
[WFCORE-787] Discard any management model changes before beginning ro…
@bstansberry bstansberry merged commit 1aad673 into wildfly:master Jul 3, 2015
@bstansberry bstansberry deleted the discard-model branch July 3, 2015 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
3 participants