Jenkins build is back to normal : brooklyn-master-build #1148

2017-07-10 Thread Apache Jenkins Server
See

[GitHub] brooklyn-server issue #753: Update jclouds to 2.0.2

2017-07-10 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/753 retest this please --- 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 wishe

[GitHub] brooklyn-server pull request #759: Revert PolicyConfigResource changes from ...

2017-07-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/759 --- 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 featur

[GitHub] brooklyn-server issue #759: Revert PolicyConfigResource changes from the RES...

2017-07-10 Thread geomacy
Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/759 LGTM, merging. Note this reverts bits of #754. --- 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

[GitHub] brooklyn-server pull request #759: Revert PolicyConfigResource changes from ...

2017-07-10 Thread iyovcheva
GitHub user iyovcheva opened a pull request: https://github.com/apache/brooklyn-server/pull/759 Revert PolicyConfigResource changes from the REST API response fix. Those changes require more careful testing and updates. We need to revert them from master and reopen in a separate

[GitHub] brooklyn-server pull request #754: Fix REST API response of methods with Str...

2017-07-10 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/754#discussion_r126417847 --- Diff: rest/rest-resources/src/main/java/org/apache/brooklyn/rest/resources/PolicyConfigResource.java --- @@ -66,7 +66,7 @@ ((B

[GitHub] brooklyn-server pull request #754: Fix REST API response of methods with Str...

2017-07-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/754 --- 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 featur

[GitHub] brooklyn-server issue #753: Update jclouds to 2.0.2

2017-07-10 Thread m4rkmckenna
Github user m4rkmckenna commented on the issue: https://github.com/apache/brooklyn-server/pull/753 retest this please --- 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

[GitHub] brooklyn-server issue #754: Fix REST API response of methods with String ret...

2017-07-10 Thread geomacy
Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/754 per @neykov's suggestion, merging this with the policy changes included. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] brooklyn-server pull request #746: Auto wrap YAML BOMs as catalog bundles

2017-07-10 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/746#discussion_r126401057 --- Diff: core/src/main/java/org/apache/brooklyn/core/typereg/RegisteredTypes.java --- @@ -125,21 +178,52 @@ public static RegisteredType bean(@Nonn

[GitHub] brooklyn-server pull request #746: Auto wrap YAML BOMs as catalog bundles

2017-07-10 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/746#discussion_r126365587 --- Diff: api/src/main/java/org/apache/brooklyn/api/typereg/ManagedBundle.java --- @@ -27,9 +27,13 @@ /** A URL-like thing that we ca

[GitHub] brooklyn-server pull request #746: Auto wrap YAML BOMs as catalog bundles

2017-07-10 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/746#discussion_r126385667 --- Diff: core/src/main/java/org/apache/brooklyn/core/BrooklynFeatureEnablement.java --- @@ -121,6 +120,8 @@ * The functionality loads cat

[GitHub] brooklyn-server pull request #746: Auto wrap YAML BOMs as catalog bundles

2017-07-10 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/746#discussion_r126389365 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/BasicBrooklynCatalog.java --- @@ -1060,28 +1337,87 @@ private String makeAsI

[GitHub] brooklyn-server pull request #746: Auto wrap YAML BOMs as catalog bundles

2017-07-10 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/746#discussion_r126387732 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/BasicBrooklynCatalog.java --- @@ -472,12 +524,46 @@ private void collectCata

[GitHub] brooklyn-server pull request #746: Auto wrap YAML BOMs as catalog bundles

2017-07-10 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/746#discussion_r126396417 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/BasicBrooklynCatalog.java --- @@ -792,28 +1027,58 @@ private String setFromI

[GitHub] brooklyn-server pull request #746: Auto wrap YAML BOMs as catalog bundles

2017-07-10 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/746#discussion_r126373385 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/BasicBrooklynCatalog.java --- @@ -449,21 +494,28 @@ public static VersionedN

[GitHub] brooklyn-server pull request #746: Auto wrap YAML BOMs as catalog bundles

2017-07-10 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/746#discussion_r126391110 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/BasicBrooklynCatalog.java --- @@ -511,30 +597,76 @@ private void collectCata

Jenkins build is unstable: brooklyn-dist-master #107

2017-07-10 Thread Apache Jenkins Server
See

[GitHub] brooklyn-dist pull request #98: Don't avoid importing jclouds google compute...

2017-07-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-dist/pull/98 --- 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 i

[GitHub] brooklyn-dist issue #98: Don't avoid importing jclouds google compute engine...

2017-07-10 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-dist/pull/98 Thanks @geomacy, merging. Lot's more entries need removing from here, can be addressed in another PR. --- If your project is set up for it, you can reply to this email and have your rep