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-1231 Remove OSGI bundle related stuff from patching tests #1295

Merged

Conversation

jmartisk
Copy link
Contributor

@jmartisk jmartisk commented Dec 9, 2015

OSGi subsystem is removed, so this PR removes OSGi-related stuff from the patching test suite. Interesting is that the tests still pass, apparently the Patching subsystem still can handle this stuff.

https://issues.jboss.org/browse/WFCORE-1231

@jmartisk jmartisk force-pushed the remove-osgi-bundles-patching-tests branch 2 times, most recently from 0c42667 to 4aa53f2 Compare December 16, 2015 08:22
@jmartisk jmartisk changed the title Remove OSGI bundle related stuff from patching tests WFCORE-1231 Remove OSGI bundle related stuff from patching tests Dec 16, 2015
@bstansberry bstansberry added the rebase-this This PR must be rebased before it is merged label Mar 22, 2016
@bstansberry
Copy link
Contributor

@aloubyansky please approve

@aloubyansky
Copy link
Contributor

Looks good, thanks!

@bstansberry
Copy link
Contributor

@jmartisk can you rebase this please? Thanks.

@jmartisk jmartisk force-pushed the remove-osgi-bundles-patching-tests branch from 4aa53f2 to 22c6055 Compare March 29, 2016 05:01
@jmartisk
Copy link
Contributor Author

@bstansberry Done.

@wildfly-ci
Copy link

Windows Build 3265 outcome was FAILURE using a merge of 22c6055
Summary: Tests passed: 3491, ignored: 57; compilation error: Compiler (new) Build time: 00:25:32

@wildfly-ci
Copy link

Linux Build 3844 outcome was FAILURE using a merge of 22c6055
Summary: Tests passed: 3491, ignored: 57; compilation error: Compiler (new) Build time: 00:25:58

@jmartisk jmartisk force-pushed the remove-osgi-bundles-patching-tests branch from 9d7a901 to bb6b5bd Compare March 29, 2016 05:54
@jmartisk
Copy link
Contributor Author

retest this please

@bstansberry bstansberry added ready-for-merge This PR is ready to be merged and fulfills all requirements 3.x and removed rebase-this This PR must be rebased before it is merged labels Mar 29, 2016
@bstansberry bstansberry merged commit a3b3b44 into wildfly:master Mar 30, 2016
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
4 participants