[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/73 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-29 Thread neykov
Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/73#issuecomment-202785125 Moved versions here, commit bd91628. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your proje

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-28 Thread ahgittin
Github user ahgittin commented on the pull request: https://github.com/apache/brooklyn-server/pull/73#issuecomment-202506248 you can push it here, and ping me and i'll cherry-pick it (or open against my branch) --- If your project is set up for it, you can reply to this email and hav

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-28 Thread neykov
Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/73#issuecomment-202473843 Looking at the changes now I see I've forgotten to move okhttp's version to pom.xml. Will do it in another commit not to mess your merge. Or if you are doing any

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-28 Thread neykov
Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/73#issuecomment-202469507 Haven't seen any problems with downgrading to guava 16. Tests are working fine as well. Could look in emails around the time of change for potential rebind pr

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-28 Thread ahgittin
Github user ahgittin commented on the pull request: https://github.com/apache/brooklyn-server/pull/73#issuecomment-202460197 also note the guava version changes here conflicts with the swagger guava addition; trivial to merge, done in https://github.com/ahgittin/brooklyn-server/commit

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-28 Thread ahgittin
Github user ahgittin commented on the pull request: https://github.com/apache/brooklyn-server/pull/73#issuecomment-202459244 @neykov can you confirm nothing wrong dropping to guava 16? --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-22 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/73#discussion_r56971545 --- Diff: karaf/features/src/main/feature/feature.xml --- @@ -16,12 +16,17 @@ See the License for the specific language governing permissions a

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-21 Thread m4rkmckenna
Github user m4rkmckenna commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/73#discussion_r56885604 --- Diff: karaf/features/src/main/feature/feature.xml --- @@ -16,12 +16,17 @@ See the License for the specific language governing permissi

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-21 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/73#discussion_r56856478 --- Diff: karaf/features/src/main/feature/feature.xml --- @@ -16,12 +16,17 @@ See the License for the specific language governing permissions a

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-21 Thread neykov
Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/73#issuecomment-199375076 Blueprint used to test: ``` brooklyn.catalog: items: - id: machine libraries: - name: org.apache.brooklyn.software-base

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-21 Thread neykov
Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/73#issuecomment-199373340 Once tasks started piling up started getting the following error in the logs: ``` 2016-03-21 17:33:11,450 | INFO | tp1904918854-187 | LoggingInInterc

[GitHub] brooklyn-server pull request: [OSGi] Load jclouds bundles in Karaf

2016-03-21 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-server/pull/73 [OSGi] Load jclouds bundles in Karaf Needed to break the `locations-jclouds` dependency on `googlecompute` as it doesn't have an osgi bundle in `1.9.2`. With the change I am able to