GitHub user ahgittin opened a pull request:

    https://github.com/apache/brooklyn-server/pull/842

    Improve Brooklyn-management of OSGi-already-managed bundles

    Previously could claim it was already installed but not brooklyn-manage it.
    
    Now it is much smarter about checking whether what is being supplied now 
(by reference or by contents) is an exact match and if so, don't uninstall and 
reinstall it.
    
    Some annoying issues fixing tests due to how karaf/java deals with 
`target/classes/` bundles in dev, but tests are all working too now.
    
    NB this requires #810 due to conflicts in osgi bundle removal semantics

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ahgittin/brooklyn-server 
bundles-brooklyn-import-osgi-already-managed

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/brooklyn-server/pull/842.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #842
    
----
commit 9b337035992f77c7e156bba4d3b3ce35438ca180
Author: Alex Heneveld <[email protected]>
Date:   2017-09-07T15:43:45Z

    REST API for bundles, types, and subtypes
    
    also tweaks to other classes to make them more usable, eg VersionedName 
comparables and bundle removal giving an OsgiBundleInstallationResult
    
    test in BundleAndTypeAndSubtypeResourcesTest based on CatalogResourceTest, 
basically giving parity in terms of functionality and test coverage

commit 1cd6bf455d4cd0004a98881cc2d9a045bf2ffd1e
Author: Alex Heneveld <[email protected]>
Date:   2017-09-13T11:53:30Z

    Merge branch 'master' into bundle-rest-api
    
    BrooklynCatalog api tidy minor conflict

commit 87a28524f5891431aabec1cc04a318f21758e3e5
Author: Alex Heneveld <[email protected]>
Date:   2017-09-21T12:16:42Z

    Merge branch 'bundle-rest-api' into 
bundles-brooklyn-import-osgi-already-managed

commit 277cb22aa6b8553bb5369dfd4d1d5eba2185a858
Author: Alex Heneveld <[email protected]>
Date:   2017-09-21T10:35:25Z

    fix bundle install issue if already osgi-installed, and tidy
    
    take the URL from the existing osgi-installed bundle.
    much better logging.

commit 3816fb7d7b5e167fd74bb156f26f1c8450136c1a
Author: Alex Heneveld <[email protected]>
Date:   2017-09-21T12:06:21Z

    add Streams.compare function

commit 69ea984467e79dc3554d0079a662648c5e9bd922
Author: Alex Heneveld <[email protected]>
Date:   2017-09-21T12:07:07Z

    handle edge cases where importing OSGi-already-managed bundles to Brooklyn 
mgmt

commit dcde6806264904ea31a71bf8565994d01b0e8d69
Author: Alex Heneveld <[email protected]>
Date:   2017-09-21T13:08:40Z

    improve making bundle from folders
    
    needed for tests now that we do more with bundles supplied in 
brooklyn.libraries

commit a017a1c58d5dc1df12a8166a4ab8b1dc253f3be8
Author: Alex Heneveld <[email protected]>
Date:   2017-09-21T13:29:02Z

    add support for dev mode bundle urls so tests pass
    
    makes behaviour consistent with dist behaviour regarding karaf urls

----


---

Reply via email to