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

FLY-4500 CMResourceService#stop fails to clean up #7334

Merged
merged 1 commit into from Apr 15, 2015

Conversation

marschall
Copy link
Contributor

CMResourceService#stop fails to clean up values that #start creates.

Issue: WFLY-4500
https://issues.jboss.org/browse/WFLY-4500

@wildfly-ci
Copy link

Linux Build 6207 is now running using a merge of 8c320ad

@wildfly-ci
Copy link

Linux with security manager Build 1182 is now running using a merge of 8c320ad

@wildfly-ci
Copy link

Windows Build 1319 is now running using a merge of 8c320ad

@wildfly-ci
Copy link

Linux with security manager Build 1182 outcome was SUCCESS using a merge of 8c320ad
Summary: Tests passed: 822, ignored: 356 Build time: 0:06:07

@wildfly-ci
Copy link

Linux Build 6207 outcome was SUCCESS using a merge of 8c320ad
Summary: Tests passed: 2841, ignored: 416, muted: 1 Build time: 0:44:37

@wildfly-ci
Copy link

Windows Build 1319 outcome was SUCCESS using a merge of 8c320ad
Summary: Tests passed: 2842, ignored: 416 Build time: 0:53:38

@bstansberry
Copy link
Contributor

Hi Philippe,

Can you fix the commit message? Missing "W" in WFLY-4500. Just makes it easier to find this fix in the log later.

Thanks,
Brian

CMResourceService#stop fails to clean up values that #start creates.

Issue: WFLY-4500
https://issues.jboss.org/browse/WFLY-4500
@marschall
Copy link
Contributor Author

Sure, no problem.

@wildfly-ci
Copy link

Linux Build 6210 is now running using a merge of 465d2a6

@wildfly-ci
Copy link

Windows Build 1322 is now running using a merge of 465d2a6

@wildfly-ci
Copy link

Linux with security manager Build 1185 is now running using a merge of 465d2a6

@wildfly-ci
Copy link

Linux with security manager Build 1185 outcome was SUCCESS using a merge of 465d2a6
Summary: Tests passed: 822, ignored: 356 Build time: 0:06:44

@wildfly-ci
Copy link

Linux Build 6210 outcome was SUCCESS using a merge of 465d2a6
Summary: Tests passed: 2842, ignored: 416 Build time: 0:51:23

@wildfly-ci
Copy link

Windows Build 1322 outcome was SUCCESS using a merge of 465d2a6
Summary: Tests passed: 2842, ignored: 416 Build time: 0:51:40

@tomjenkinson
Copy link
Contributor

Change looks good to me from a functional point of view - thanks Philippe!

@bstansberry bstansberry added the ready-for-merge Only for use by those with merge permissions! label Apr 14, 2015
bstansberry added a commit that referenced this pull request Apr 15, 2015
FLY-4500 CMResourceService#stop fails to clean up
@bstansberry bstansberry merged commit d8585d6 into wildfly:master Apr 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge Only for use by those with merge permissions!
Projects
None yet
4 participants