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
----
---