[GitHub] brooklyn-dist pull request #59: Set default JVM memory settings for the buil...

GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-dist/pull/59 Set default JVM memory settings for the build * Will be picked up when building a sub-module * Doesn't work with `--file` option ([MNG-5889](https://issues.apache.org/jira/browse/MNG-5889))

[GitHub] brooklyn-client pull request #33: Set default JVM memory settings for the bu...

GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-client/pull/33 Set default JVM memory settings for the build * Will be picked up when building a sub-module * Doesn't work with `--file` option ([MNG-5889](https://issues.apache.org/jira/browse/MNG-5889

[GitHub] brooklyn-server pull request #436: Set default JVM memory settings for the b...

GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-server/pull/436 Set default JVM memory settings for the build * Will be picked up when building a sub-module * Doesn't work with `--file` option ([MNG-5889](https://issues.apache.org/jira/browse/MNG-588

[GitHub] brooklyn-ui pull request #38: Set default JVM memory settings for the build

GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-ui/pull/38 Set default JVM memory settings for the build * Will be picked up when building a sub-module * Doesn't work with `--file` option ([MNG-5889](https://issues.apache.org/jira/browse/MNG-5889))

[GitHub] brooklyn-library pull request #74: Set default JVM memory settings for the b...

GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-library/pull/74 Set default JVM memory settings for the build You can merge this pull request into a Git repository by running: $ git pull https://github.com/neykov/brooklyn-library maven-opts Altern

[GitHub] brooklyn pull request #8: Set default JVM memory settings for the build

GitHub user neykov opened a pull request: https://github.com/apache/brooklyn/pull/8 Set default JVM memory settings for the build You can merge this pull request into a Git repository by running: $ git pull https://github.com/neykov/brooklyn maven-opts Alternatively you can r

[jira] [Created] (BROOKLYN-386) NPE on rebind calling CreateUserPolicy.addUser

Aled Sage created BROOKLYN-386: -- Summary: NPE on rebind calling CreateUserPolicy.addUser Key: BROOKLYN-386 URL: https://issues.apache.org/jira/browse/BROOKLYN-386 Project: Brooklyn Issue Type: B

[jira] [Commented] (BROOKLYN-382) Catalog takes long time to load in web-console

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

[GitHub] brooklyn-server issue #423: BROOKLYN-382: get-catalog performance improvemen...

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/423 The commit `EntityExecutionManagerTest: tidy` is because `org.apache.brooklyn.core.mgmt.internal.EntityExecutionManagerTest.testGcTaskAtNormalTagLimit` failed in the jenkins PR build. But I

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

See Changes: [aled.sage] SoftwareProcess integration tests: use INSTALL_UNIQUE_LABEL -- [...truncated 20809 lines...] 2016-11-14 22:38:13,227 INFO TESTNG INVOKING: "Surefire test" - org.a

[GitHub] brooklyn-server issue #407: App defaults to requiring all children up

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/407 @neykov @sjcorbett can one of you review 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 no

[GitHub] brooklyn-server pull request #431: SoftwareProcess integration tests: use IN...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/431 --- 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 pull request #431: SoftwareProcess integration tests: use IN...

Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/431#discussion_r87901835 --- Diff: software/base/src/test/java/org/apache/brooklyn/entity/software/base/VanillaSoftwareProcessIntegrationTest.java --- @@ -51,7 +51,7 @@ pub

[GitHub] brooklyn-library pull request #71: Don't allow multiple context entities in ...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-library/pull/71 --- 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-library issue #71: Don't allow multiple context entities in a task'...

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-library/pull/71 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

[jira] [Commented] (BROOKLYN-325) Restarting Brooklyn while entity deploying: entity shown as deploying indefinitely

[ https://issues.apache.org/jira/browse/BROOKLYN-325?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15664511#comment-15664511 ] ASF GitHub Bot commented on BROOKLYN-325: - GitHub user aledsage opened a pull re

[GitHub] brooklyn-server pull request #435: BROOKLYN-325: alert if provisioning/termi...

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/435 BROOKLYN-325: alert if provisioning/termination aborted on rebind You can merge this pull request into a Git repository by running: $ git pull https://github.com/aledsage/brooklyn-se

[GitHub] brooklyn-server issue #385: Don't allow multiple context entities in a task'...

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/385 changes look good to me -- nice work @neykov --- 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 t

[GitHub] brooklyn-server issue #426: Improve robustness of MemoryUsageTrackerTest

Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/426 Kind of related discussion on the prefix in a [previous PR](https://github.com/apache/incubator-brooklyn/pull/1068#issuecomment-159588763). According to the link the prefix doesn't exist a

[GitHub] brooklyn-server pull request #385: Don't allow multiple context entities in ...

Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/385#discussion_r87817025 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/proxy/InternalEntityFactory.java --- @@ -320,7 +320,6 @@ private void validateDescendant

[GitHub] brooklyn-server issue #429: BasicJcloudsLocationCustomizer is an EntityIniti...

Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/429 LGTM overall. Would like to see better initializer config integration. --- 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

[GitHub] brooklyn-server pull request #434: More brooklyn-server integration tests fi...

GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-server/pull/434 More brooklyn-server integration tests fixes You can merge this pull request into a Git repository by running: $ git pull https://github.com/neykov/brooklyn-server fix/integration-test

[GitHub] brooklyn-server issue #426: Improve robustness of MemoryUsageTrackerTest

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/426 The `SUREFIRE-859` prefix lines may depend on which JVM is being used? --- 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

[GitHub] brooklyn-server pull request #429: BasicJcloudsLocationCustomizer is an Enti...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/429#discussion_r87814071 --- Diff: locations/jclouds/src/test/java/org/apache/brooklyn/location/jclouds/BasicJcloudsLocationCustomizerTest.java --- @@ -0,0 +1,81 @@ +/*

[GitHub] brooklyn-server pull request #429: BasicJcloudsLocationCustomizer is an Enti...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/429#discussion_r87813756 --- Diff: locations/jclouds/src/test/java/org/apache/brooklyn/location/jclouds/BasicJcloudsLocationCustomizerTest.java --- @@ -0,0 +1,81 @@ +/*

[GitHub] brooklyn-server pull request #429: BasicJcloudsLocationCustomizer is an Enti...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/429#discussion_r87813264 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/BasicJcloudsLocationCustomizer.java --- @@ -18,19 +18,72 @@ */

[GitHub] brooklyn-server pull request #425: Do not merge; for review only; network ex...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/425#discussion_r87810853 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/JcloudsLocationSecurityGroupEditor.java --- @@ -0,0 +1,223

[jira] [Commented] (BROOKLYN-382) Catalog takes long time to load in web-console

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

[GitHub] brooklyn-server issue #423: BROOKLYN-382: get-catalog performance improvemen...

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/423 @neykov great comments - I've addressed them. Can you take another look please? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] brooklyn-server pull request #433: Fix dependency URI path for downstream pr...

Github user tbouron commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/433#discussion_r87794702 --- Diff: karaf/features/src/main/feature/feature.xml --- @@ -256,7 +256,7 @@ https://github.com/square/okhttp/pull/2246

[GitHub] brooklyn-server pull request #433: Fix dependency URI path for downstream pr...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/433 --- 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 #433: Fix dependency URI path for downstream projects ...

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/433 I understand that this has been confirmed manually with a mvn build, checking in karaf that okhttp bundle was added correctly. Therefore merging now. --- If your project is set up for it,

[GitHub] brooklyn-server pull request #433: Fix dependency URI path for downstream pr...

Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/433#discussion_r87794146 --- Diff: karaf/features/src/main/feature/feature.xml --- @@ -256,7 +256,7 @@ https://github.com/square/okhttp/pull/2246

[GitHub] brooklyn-server issue #433: Fix dependency URI path for downstream projects ...

Github user tbouron commented on the issue: https://github.com/apache/brooklyn-server/pull/433 For some reason, the official maven karaf plugin cannot verify this feature if the quotes are not single quotes AND properly URI encoded, hence the `%27` --- If your project is set up for i

[GitHub] brooklyn-server pull request #433: Fix dependency URI path for downstream pr...

GitHub user tbouron opened a pull request: https://github.com/apache/brooklyn-server/pull/433 Fix dependency URI path for downstream projects + Karaf feature verification You can merge this pull request into a Git repository by running: $ git pull https://github.com/tbouron/br

[jira] [Updated] (BROOKLYN-378) Intermittent failures in builds on stop related tests

[ https://issues.apache.org/jira/browse/BROOKLYN-378?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Geoff Macartney updated BROOKLYN-378: - Fix Version/s: 0.10.0 > Intermittent failures in builds on stop related tests >

[jira] [Resolved] (BROOKLYN-378) Intermittent failures in builds on stop related tests

[ https://issues.apache.org/jira/browse/BROOKLYN-378?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Geoff Macartney resolved BROOKLYN-378. -- Resolution: Fixed Haven't observed this failure again, it seems likely this is fixed a

[GitHub] brooklyn-server issue #431: SoftwareProcess integration tests: use INSTALL_U...

Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/431 Good to merge, but don't really understand why randomize the temporary files? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] brooklyn-server pull request #431: SoftwareProcess integration tests: use IN...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/431#discussion_r87782693 --- Diff: software/base/src/test/java/org/apache/brooklyn/entity/software/base/VanillaSoftwareProcessIntegrationTest.java --- @@ -51,7 +51,7 @@ publi

[GitHub] brooklyn-server pull request #432: [TRIVIAL] DslComponent: fix typo in comme...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/432 --- 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 #432: [TRIVIAL] DslComponent: fix typo in comment

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/432 Merging immediately - it only changes a comment! --- 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 hav

[GitHub] brooklyn-server pull request #432: [TRIVIAL] DslComponent: fix typo in comme...

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/432 [TRIVIAL] DslComponent: fix typo in comment Fixes a typo in a comment, spotted during review of https://github.com/apache/brooklyn-server/pull/417 You can merge this pull request into a G

[GitHub] brooklyn-server issue #426: Improve robustness of MemoryUsageTrackerTest

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/426 @Tibor17 not sure what you mean by "The prefix `SUREFIRE-859` does not exist in the console." I see the output that @neykov pasted when I follow the link to https://builds.apache.o

[jira] [Commented] (BROOKLYN-385) Synchronous start of group members

[ https://issues.apache.org/jira/browse/BROOKLYN-385?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15663491#comment-15663491 ] Sam Corbett commented on BROOKLYN-385: -- Two suggestions from [~alex.heneveld]: {qu

[GitHub] brooklyn-server pull request #431: SoftwareProcess integration tests: use IN...

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/431 SoftwareProcess integration tests: use INSTALL_UNIQUE_LABEL Fixes VanillaSoftwareProcessStreamsIntegrationTest.testGetsStreams, for if run multiple times on the same machine. @

[jira] [Created] (BROOKLYN-385) Synchronous start of group members

Sam Corbett created BROOKLYN-385: Summary: Synchronous start of group members Key: BROOKLYN-385 URL: https://issues.apache.org/jira/browse/BROOKLYN-385 Project: Brooklyn Issue Type: Improveme

[jira] [Created] (BROOKLYN-384) ZookeeperEnsemble configuration error

Sam Corbett created BROOKLYN-384: Summary: ZookeeperEnsemble configuration error Key: BROOKLYN-384 URL: https://issues.apache.org/jira/browse/BROOKLYN-384 Project: Brooklyn Issue Type: Bug

[jira] [Commented] (BROOKLYN-375) Brooklyn intermittently uses high CPU levels and becomes unresponsive

[ https://issues.apache.org/jira/browse/BROOKLYN-375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15663378#comment-15663378 ] ASF GitHub Bot commented on BROOKLYN-375: - Github user neykov commented on a dif

[GitHub] brooklyn-docs pull request #122: info on debugging memory usage

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/122#discussion_r87770697 --- Diff: guide/ops/troubleshooting/memory-usage.md --- @@ -0,0 +1,138 @@ +--- +layout: website-normal +title: "Troubleshooting: Monitoring M

[jira] [Commented] (BROOKLYN-375) Brooklyn intermittently uses high CPU levels and becomes unresponsive

[ https://issues.apache.org/jira/browse/BROOKLYN-375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15663322#comment-15663322 ] ASF GitHub Bot commented on BROOKLYN-375: - Github user aledsage commented on a d

[GitHub] brooklyn-docs pull request #122: info on debugging memory usage

Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/122#discussion_r87766901 --- Diff: guide/ops/troubleshooting/memory-usage.md --- @@ -0,0 +1,138 @@ +--- +layout: website-normal +title: "Troubleshooting: Monitoring