[GitHub] brooklyn-server pull request #300: Location DSL Updates

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/300#discussion_r79810923 --- Diff: camp/camp-brooklyn/src/main/java/org/apache/brooklyn/camp/brooklyn/spi/creation/LocationSpecConfiguration.java --- @@ -0,0 +1,58 @@

[jira] [Created] (BROOKLYN-349) NPE using yaml DSL in entity's "provisioning.properties"

Aled Sage created BROOKLYN-349: -- Summary: NPE using yaml DSL in entity's "provisioning.properties" Key: BROOKLYN-349 URL: https://issues.apache.org/jira/browse/BROOKLYN-349 Project: Brooklyn

[GitHub] brooklyn-client pull request #28: Updated resteasy version

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-client/pull/28 Updated resteasy version This is successor to [this PR](https://github.com/apache/brooklyn-server/pull/317). After the rest client was moved to brooklyn-client in [this

[GitHub] brooklyn-server issue #317: Changed jackson version to 2.7.5

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/317 replacing with: https://github.com/apache/brooklyn-client/pull/28 https://github.com/apache/brooklyn-server/pull/339 --- If your project is set up for it, you can reply to

[GitHub] brooklyn-server pull request #317: Changed jackson version to 2.7.5

Github user Graeme-Miller closed the pull request at: https://github.com/apache/brooklyn-server/pull/317 --- 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

[GitHub] brooklyn-client issue #28: Updated resteasy version

Github user neykov commented on the issue: https://github.com/apache/brooklyn-client/pull/28 @Graeme-Miller this change won't be needed if https://github.com/apache/brooklyn-client/pull/27 is merged. --- If your project is set up for it, you can reply to this email and have your

[GitHub] brooklyn-client issue #28: Updated resteasy version

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-client/pull/28 Test failures are because brooklyn-client relies on brooklyn-rest-api which pulls in an older verision of jackson. All tests pass locally when

[GitHub] brooklyn-server pull request #300: Location DSL Updates

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/300#discussion_r79832991 --- Diff: camp/camp-brooklyn/src/main/java/org/apache/brooklyn/camp/brooklyn/spi/dsl/methods/DslComponent.java --- @@ -84,51 +89,72 @@ public

[GitHub] brooklyn-server issue #300: Location DSL Updates

Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/300 Changes look good (apart from Aled's comments). Very useful! I prefer if we don't re-use DslComponent for implementing both entity and location behaviour. The parser supports object

[GitHub] brooklyn-server pull request #300: Location DSL Updates

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/300#discussion_r79842886 --- Diff: core/src/main/java/org/apache/brooklyn/entity/group/DynamicFabric.java --- @@ -18,39 +18,41 @@ */ package

[GitHub] brooklyn-server pull request #300: Location DSL Updates

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/300#discussion_r79842987 --- Diff: core/src/main/java/org/apache/brooklyn/entity/group/DynamicFabric.java --- @@ -67,9 +69,7 @@ public static final AttributeSensor

[GitHub] brooklyn-server pull request #300: Location DSL Updates

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/300#discussion_r79833570 --- Diff: core/src/main/java/org/apache/brooklyn/entity/group/DynamicFabricImpl.java --- @@ -134,29 +125,39 @@ public void start(Collection locsO) {

[jira] [Commented] (BROOKLYN-349) NPE using yaml DSL in entity's "provisioning.properties"

[ https://issues.apache.org/jira/browse/BROOKLYN-349?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15510072#comment-15510072 ] Andrew Kennedy commented on BROOKLYN-349: - See

[GitHub] brooklyn-server pull request #338: Not for merge, work in progress - testCat...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/338#discussion_r79871837 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogItemDtoAbstract.java --- @@ -45,6 +45,8 @@ import

[GitHub] brooklyn-server pull request #338: Not for merge, work in progress - testCat...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/338#discussion_r79872120 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogUtils.java --- @@ -170,6 +173,13 @@ public static void

Re: Google Summer of Code - PaaS services addition proposal

GSoC 2016 has come to a close - thank you Jose for your great effort and thanks to the Brooklyn community for the opportunity! Jose, I still think it would be great to arrange your demo asap so that others can see your work, let us know when you can and we'll help promoting the event. Best,

[GitHub] brooklyn-server issue #338: Not for merge, work in progress - testCatalogIte...

Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/338 Latest update - adds resource lookup from nested catalog definitions. Nested resources already have a test,

[GitHub] brooklyn-docs pull request #109: Various fixes. Mainly changing relative lin...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/109 --- 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 issue #340: ConfigMap refactor, respecting inheritance

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/340 I think there are just two things discussed last week which I've not addressed: * For classpath URL resolution from runtime ancestors, the caller still needs to look up the

[GitHub] brooklyn-server pull request #340: ConfigMap refactor, respecting inheritanc...

GitHub user ahgittin opened a pull request: https://github.com/apache/brooklyn-server/pull/340 ConfigMap refactor, respecting inheritance As discussed on #281, there were some severe limitations in the way we handled config and inheritance. This addresses many of those, and

[jira] [Commented] (BROOKLYN-349) NPE using yaml DSL in entity's "provisioning.properties"

[ https://issues.apache.org/jira/browse/BROOKLYN-349?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15510550#comment-15510550 ] ASF GitHub Bot commented on BROOKLYN-349: - Github user aledsage commented on the issue:

[GitHub] brooklyn-server issue #279: Allow provisioning.properties to reference resol...

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/279 See https://issues.apache.org/jira/browse/BROOKLYN-349 for a description of the underlying problem. Also see https://github.com/apache/brooklyn-server/pull/341 for an alternative

[GitHub] brooklyn-server issue #341: BROOKLYN-349: fix DSL resolution in location

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/341 Can we merge #340 and then look at this? It gives clearer methods for dealing with semantics of copying resolved / unresolved semantics, and it moves `Location`'s config to be treated

[GitHub] brooklyn-server pull request #341: BROOKLYN—349: fix DSL resolution in loc...

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/341 BROOKLYN—349: fix DSL resolution in location You can merge this pull request into a Git repository by running: $ git pull https://github.com/aledsage/brooklyn-server BROOKLYN-349

[jira] [Commented] (BROOKLYN-349) NPE using yaml DSL in entity's "provisioning.properties"

[ https://issues.apache.org/jira/browse/BROOKLYN-349?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15510542#comment-15510542 ] Aled Sage commented on BROOKLYN-349: As per [~andrew.kennedy]'s comment, his PR is an alternative

[GitHub] brooklyn-server issue #341: BROOKLYN—349: fix DSL resolution in location

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/341 See #279 for an alternative fix (though no test - we could copy the test from here into that PR). We'll only want one of this PR or #279 to be merged, I think. @grkvlt @ahgittin

[jira] [Commented] (BROOKLYN-349) NPE using yaml DSL in entity's "provisioning.properties"

[ https://issues.apache.org/jira/browse/BROOKLYN-349?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15510532#comment-15510532 ] Aled Sage commented on BROOKLYN-349: It turns out it's not as simple as just not dumbly copying

[jira] [Commented] (BROOKLYN-349) NPE using yaml DSL in entity's "provisioning.properties"

[ https://issues.apache.org/jira/browse/BROOKLYN-349?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15510712#comment-15510712 ] ASF GitHub Bot commented on BROOKLYN-349: - Github user ahgittin commented on the issue:

Build failed in Jenkins: brooklyn-master-windows #294

See -- [...truncated 37833 lines...] [JENKINS] Recording test results [INFO] [INFO] Reactor Summary: [INFO] [INFO]

[jira] [Created] (BROOKLYN-348) "java.lang.OutOfMemoryError: PermGen space" with Karaf dist

Svetoslav Neykov created BROOKLYN-348: - Summary: "java.lang.OutOfMemoryError: PermGen space" with Karaf dist Key: BROOKLYN-348 URL: https://issues.apache.org/jira/browse/BROOKLYN-348 Project: