Jenkins build is back to normal : brooklyn-master-windows #250

See

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

See

[GitHub] brooklyn-server pull request #276: Effector for opening inbound ports in sec...

Github user bostko commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/276#discussion_r75734999 --- Diff: software/base/pom.xml --- @@ -116,7 +116,6 @@ org.apache.brooklyn brooklyn-locations-jclouds

Build failed in Jenkins: brooklyn-master-build #359

See -- [...truncated 29674 lines...] Tests run: 184, Failures: 1, Errors: 0, Skipped: 0 [JENKINS] Recording test results [INFO] --

[jira] [Updated] (BROOKLYN-336) Policy added even though the API returns an error

[ https://issues.apache.org/jira/browse/BROOKLYN-336?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Thomas Bouron updated BROOKLYN-336: --- Attachment: brooklyn-policy-bug.gif > Policy added even though the API returns an error > --

[jira] [Updated] (BROOKLYN-336) Policy added even though the API returns an error

[ https://issues.apache.org/jira/browse/BROOKLYN-336?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Thomas Bouron updated BROOKLYN-336: --- Description: Deploying this simple blueprint: {code:title=Bar.java|borderStyle=solid} locati

[jira] [Updated] (BROOKLYN-336) Policy added even though the API returns an error

[ https://issues.apache.org/jira/browse/BROOKLYN-336?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Thomas Bouron updated BROOKLYN-336: --- Description: Deploying this simple blueprint: {code:title=blueprint.yaml|borderStyle=solid}

[jira] [Created] (BROOKLYN-336) Policy added even though the API returns an error

Thomas Bouron created BROOKLYN-336: -- Summary: Policy added even though the API returns an error Key: BROOKLYN-336 URL: https://issues.apache.org/jira/browse/BROOKLYN-336 Project: Brooklyn Is

[GitHub] brooklyn-server pull request #304: Adds a target configuration to the SSH co...

Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/304#discussion_r75693969 --- Diff: core/src/main/java/org/apache/brooklyn/core/entity/AbstractEntity.java --- @@ -196,7 +196,11 @@ "entity.group.added", "Grou

Build failed in Jenkins: brooklyn-dist-master #29

See Changes: [svetoslav.neykov] OSGi features cleanup [svetoslav.neykov] jstack helper script to dump a single named thread -- [...truncated 392 lines...] [INFO] Copying 3 resources [INFO] [

[GitHub] brooklyn-dist pull request #42: OSGi features cleanup

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-dist/pull/42 --- 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-server pull request #303: OSGi features cleanup

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/303 --- 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 #285: Adding HttpExecutor to be used for interchangeab...

Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/285 Looks like a great start. --- 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 an

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75689880 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/http/executor/apacheclient/HttpExecutorImpl.java --- @@ -0,0 +1,106 @@ +/* + *

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75689508 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/http/HttpExecutorFactory.java --- @@ -0,0 +1,65 @@ +/* + * Licensed to the Apac

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75688758 --- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java --- @@ -275,7 +293,23 @@ public HttpFeed() { protected HttpFeed(Build

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75688643 --- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java --- @@ -275,7 +293,23 @@ public HttpFeed() { protected HttpFeed(Build

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75687758 --- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java --- @@ -275,7 +293,23 @@ public HttpFeed() { protected HttpFeed(Build

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75687462 --- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java --- @@ -275,7 +293,23 @@ public HttpFeed() { protected HttpFeed(Build

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75687134 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/http/executor/HttpRequest.java --- @@ -0,0 +1,128 @@ +/* + * Licensed to the Ap

[GitHub] brooklyn-server pull request #304: Adds a target configuration to the SSH co...

Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/304#discussion_r75686872 --- Diff: core/src/main/java/org/apache/brooklyn/enricher/stock/Joiner.java --- @@ -92,36 +96,48 @@ protected Object compute(SensorEvent event) {

[GitHub] brooklyn-server pull request #304: Adds a target configuration to the SSH co...

Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/304#discussion_r75686795 --- Diff: core/src/main/java/org/apache/brooklyn/enricher/stock/Enrichers.java --- @@ -293,42 +307,33 @@ protected String getDefaultUniqueTag() {

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75685586 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/http/executor/HttpRequest.java --- @@ -0,0 +1,128 @@ +/* + * Licensed to the Ap

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75685265 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/http/executor/HttpRequest.java --- @@ -0,0 +1,128 @@ +/* + * Licensed to the Ap

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75684986 --- Diff: core/src/main/java/org/apache/brooklyn/core/location/LocationConfigKeys.java --- @@ -35,7 +35,9 @@ public static final ConfigKey DI

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75684207 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/http/HttpExecutorFactory.java --- @@ -0,0 +1,65 @@ +/* + * Licensed to the Apac

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75683850 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/http/HttpExecutorFactory.java --- @@ -0,0 +1,65 @@ +/* + * Licensed to the Apac

[GitHub] brooklyn-server pull request #285: Adding HttpExecutor to be used for interc...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/285#discussion_r75683600 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/http/HttpExecutorFactory.java --- @@ -0,0 +1,65 @@ +/* + * Licensed to the Apac

[GitHub] brooklyn-server pull request #304: Adds a target configuration to the SSH co...

Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/304#discussion_r75683154 --- Diff: core/src/main/java/org/apache/brooklyn/core/entity/AbstractEntity.java --- @@ -196,7 +196,11 @@ "entity.group.added", "Grou

[GitHub] brooklyn-server pull request #304: Adds a target configuration to the SSH co...

Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/304#discussion_r75682542 --- Diff: core/src/main/java/org/apache/brooklyn/entity/group/SshCommandMembershipTrackingPolicy.java --- @@ -62,6 +69,38 @@ public static f

[GitHub] brooklyn-server issue #276: Effector for opening inbound ports in security g...

Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/276 LGTM, minor comments only. --- 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

[GitHub] brooklyn-server pull request #276: Effector for opening inbound ports in sec...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/276#discussion_r75681798 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/NetworkingEffectors.java --- @@ -0,0 +1,90 @@ +/* +

[GitHub] brooklyn-server pull request #276: Effector for opening inbound ports in sec...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/276#discussion_r75679060 --- Diff: software/base/src/test/java/org/apache/brooklyn/entity/software/base/location/NetworkingEffectorsLiveTests.java --- @@ -0,0 +1,107 @@ +

[GitHub] brooklyn-server pull request #276: Effector for opening inbound ports in sec...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/276#discussion_r75678644 --- Diff: software/base/src/main/java/org/apache/brooklyn/entity/software/base/SoftwareProcess.java --- @@ -147,6 +147,9 @@ @SetFromFlag("ru

[GitHub] brooklyn-server pull request #276: Effector for opening inbound ports in sec...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/276#discussion_r75678389 --- Diff: software/base/src/main/java/org/apache/brooklyn/entity/software/base/SoftwareProcess.java --- @@ -147,6 +147,9 @@ @SetFromFlag("ru

[GitHub] brooklyn-server pull request #276: Effector for opening inbound ports in sec...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/276#discussion_r75677727 --- Diff: software/base/pom.xml --- @@ -116,7 +116,6 @@ org.apache.brooklyn brooklyn-locations-jclouds

[GitHub] brooklyn-server pull request #276: Effector for opening inbound ports in sec...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/276#discussion_r75677407 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/NetworkingEffectors.java --- @@ -0,0 +1,90 @@ +/* +

[GitHub] brooklyn-server pull request #276: Effector for opening inbound ports in sec...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/276#discussion_r75676700 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/NetworkingEffectors.java --- @@ -0,0 +1,90 @@ +/* +

[GitHub] brooklyn-dist issue #42: OSGi features cleanup

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-dist/pull/42 successfully tested --- 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 #303: OSGi features cleanup

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/303 successfully tested --- 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 a

[GitHub] brooklyn-dist issue #42: OSGi features cleanup

Github user geomacy commented on the issue: https://github.com/apache/brooklyn-dist/pull/42 Looks good to me from review point of view and AMP starts successfully. Haven't tested a large number of restarts using the script though. --- If your project is set up for it, you can reply t

[GitHub] brooklyn-server issue #303: OSGi features cleanup

Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/303 Looks good to me from review point of view and AMP starts successfully. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[jira] [Created] (BROOKLYN-335) brooklyn-client runtime panic on an app with no location

Geoff Macartney created BROOKLYN-335: Summary: brooklyn-client runtime panic on an app with no location Key: BROOKLYN-335 URL: https://issues.apache.org/jira/browse/BROOKLYN-335 Project: Brooklyn

[GitHub] brooklyn-server pull request #304: Adds a target configuration to the SSH co...

Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/304#discussion_r75631918 --- Diff: core/src/main/java/org/apache/brooklyn/core/entity/AbstractEntity.java --- @@ -196,7 +196,11 @@ "entity.group.added", "Gr

[GitHub] brooklyn-server pull request #304: Adds a target configuration to the SSH co...

Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/304#discussion_r75630862 --- Diff: core/src/main/java/org/apache/brooklyn/enricher/stock/Joiner.java --- @@ -92,36 +96,48 @@ protected Object compute(SensorEvent event) {

[GitHub] brooklyn-server pull request #304: Adds a target configuration to the SSH co...

Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/304#discussion_r75630441 --- Diff: core/src/main/java/org/apache/brooklyn/enricher/stock/Enrichers.java --- @@ -293,42 +307,33 @@ protected String getDefaultUniqueTag() {

[GitHub] brooklyn-server issue #285: Adding HttpExecutor to be used for interchangeab...

Github user ygy commented on the issue: https://github.com/apache/brooklyn-server/pull/285 @nakomis @neykov Can you review this PR? --- 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 featur

[GitHub] brooklyn-server pull request #304: Adds a target configuration to the SSH co...

Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/304#discussion_r75629783 --- Diff: core/src/main/java/org/apache/brooklyn/core/entity/AbstractEntity.java --- @@ -196,7 +196,11 @@ "entity.group.added", "Gr

[GitHub] brooklyn-server pull request #304: Adds a target configuration to the SSH co...

Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/304#discussion_r75628728 --- Diff: core/src/main/java/org/apache/brooklyn/entity/group/SshCommandMembershipTrackingPolicy.java --- @@ -62,6 +69,38 @@ public static