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

2016-06-09 Thread Apache Jenkins Server
See -- [...truncated 30769 lines...] Run 6: ConfigInheritanceYamlTest.tearDown:139 » FileSystem C:\Users\CHRIST~1\AppData\... Run 7: ConfigInheritanceYamlTest.tearDown:139 » FileSystem

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

2016-06-09 Thread Apache Jenkins Server
See -- [...truncated 20658 lines...] 2016-06-10 02:07:35,448 INFO TESTNG INVOKING CONFIGURATION: "Surefire test" - @AfterMethod

Jenkins build is back to normal : brooklyn-server-master #128

2016-06-09 Thread Apache Jenkins Server
See

[GitHub] brooklyn-server pull request #180: prevent OOME on big streams and better lo...

2016-06-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/180 --- 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

[GitHub] brooklyn-server issue #180: prevent OOME on big streams and better logging f...

2016-06-09 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/180 LGTM; merging. --- 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

[GitHub] brooklyn-server pull request #180: prevent OOME on big streams and better lo...

2016-06-09 Thread aledsage
Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/180#discussion_r66537291 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/BrooklynTaskTags.java --- @@ -190,9 +190,16 @@ protected WrappedStream(String

Build failed in Jenkins: brooklyn-server-master #127

2016-06-09 Thread Apache Jenkins Server
See Changes: [alex.heneveld] fix for issue where $brooklyn:external refs in provisioning.properties [alex.heneveld] tidy test / logging as per PR comments -- [...truncated 24170

[GitHub] brooklyn-server pull request #193: fix for issue where $brooklyn:external re...

2016-06-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/193 --- 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

[GitHub] brooklyn-server issue #193: fix for issue where $brooklyn:external refs in p...

2016-06-09 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/193 thx @nakomis, all addressed --- 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

[GitHub] brooklyn-server pull request #193: fix for issue where $brooklyn:external re...

2016-06-09 Thread nakomis
Github user nakomis commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/193#discussion_r66481067 --- Diff: camp/camp-brooklyn/src/test/java/org/apache/brooklyn/camp/brooklyn/LocationExternalConfigYamlTest.java --- @@ -0,0 +1,153 @@ +/*

[GitHub] brooklyn-server pull request #193: fix for issue where $brooklyn:external re...

2016-06-09 Thread nakomis
Github user nakomis commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/193#discussion_r66480706 --- Diff: camp/camp-brooklyn/src/test/java/org/apache/brooklyn/camp/brooklyn/LocationExternalConfigYamlTest.java --- @@ -0,0 +1,153 @@ +/*

[GitHub] brooklyn-server pull request #193: fix for issue where $brooklyn:external re...

2016-06-09 Thread nakomis
Github user nakomis commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/193#discussion_r66480033 --- Diff: camp/camp-brooklyn/src/test/java/org/apache/brooklyn/camp/brooklyn/LocationExternalConfigYamlTest.java --- @@ -0,0 +1,153 @@ +/*

[GitHub] brooklyn-server pull request #193: fix for issue where $brooklyn:external re...

2016-06-09 Thread ahgittin
GitHub user ahgittin opened a pull request: https://github.com/apache/brooklyn-server/pull/193 fix for issue where $brooklyn:external refs in provisioning.properties are resolved too early several more tests, including one which failed before the fix -- and of course the fix

[GitHub] brooklyn-server pull request #192: Adds OnSubnetNetworkEnricher

2016-06-09 Thread aledsage
GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/192 Adds OnSubnetNetworkEnricher Some entities publish their endpoint using the public hostname (e.g. `MySqlNode` publishes `datastore.url` using the sensor `host.name`). If trying to

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

2016-06-09 Thread Apache Jenkins Server
See -- [...truncated 28340 lines...] at org.apache.maven.surefire.testng.TestNGExecutor.run(TestNGExecutor.java:115) at

[GitHub] brooklyn-server issue #157: Windows helper script for using Invoke-Command -...

2016-06-09 Thread nakomis
Github user nakomis commented on the issue: https://github.com/apache/brooklyn-server/pull/157 @bostko Ideally, *all* of the scripts and config file should be copied some path relative to `C:\brooklyn-managed-processes\install`. If they're copied to the temp folder, there's always

[GitHub] brooklyn-server pull request #177: Adds PublicNetworkFaceEnricher

2016-06-09 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/177#discussion_r66411519 --- Diff: core/src/main/java/org/apache/brooklyn/core/network/OnPublicNetworkEnricher.java --- @@ -0,0 +1,496 @@ +/* + * Licensed to the

[GitHub] brooklyn-server issue #157: Windows helper script for using Invoke-Command -...

2016-06-09 Thread bostko
Github user bostko commented on the issue: https://github.com/apache/brooklyn-server/pull/157 @nakomis this way the install command can be chained with other commands very easily inside the yaml. The common use case for the `invoke-command-credssp.ps` helper is to be used as a

[GitHub] brooklyn-server pull request #191: Add AbstractTransformer.ALLOW_CYCLIC_PUBL...

2016-06-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/191 --- 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

[GitHub] brooklyn-server issue #191: Add AbstractTransformer.ALLOW_CYCLIC_PUBLISHING

2016-06-09 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/191 Thanks @nakomis - merging. --- 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

[GitHub] brooklyn-server issue #191: Add AbstractTransformer.ALLOW_CYCLIC_PUBLISHING

2016-06-09 Thread nakomis
Github user nakomis commented on the issue: https://github.com/apache/brooklyn-server/pull/191 One comment, but other than that, LGTM --- 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