Build failed in Jenkins: brooklyn-library-master #427

2017-09-19 Thread Apache Jenkins Server
See -- Started by timer [EnvInject] - Loading node environment variables. Building remotely on H24 (couchdbtest ubuntu xenial) in workspace

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

2017-09-19 Thread Apache Jenkins Server
See -- [...truncated 8.20 MB...] [INFO] Brooklyn Karaf Shell Commands .. SKIPPED [INFO] Brooklyn Library Root .. SKIPPED [INFO] Brookl

[GitHub] brooklyn-server pull request #833: Fix PeriodicEffectorPolicy

2017-09-19 Thread aledsage
GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/833 Fix PeriodicEffectorPolicy The `PeriodicEffectorPolicy` rebind test(s) were failing non-deterministically in the jenkins master build, and same for me locally. This PR makes a bun

Re: UsageListener: notified of promotion to master?

2017-09-19 Thread Aled Sage
Hi all, I finally got round to implementing this! See PR at https://github.com/apache/brooklyn-server/pull/832 Aled On 22/03/2017 19:52, Svetoslav Neykov wrote: It feels like (2) is a more generally useful feature to add (for programmatic support, at least). Does it sound useful for things

[GitHub] brooklyn-server pull request #832: Add ManagementNodeStateListener support

2017-09-19 Thread aledsage
Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/832#discussion_r139803014 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/internal/ManagementNodeStateListenerManager.java --- @@ -0,0 +1,175 @@ +/* + *

[GitHub] brooklyn-server issue #832: Add ManagementNodeStateListener support

2017-09-19 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/832 retest this please ---

[GitHub] brooklyn-library issue #129: Move chef from brooklyn-software-base to own mo...

2017-09-19 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-library/pull/129 retest this please ---

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

2017-09-19 Thread Apache Jenkins Server
See Changes: [aled.sage] BROOKLYN-533: adds maxConcurrentMachineDeletions -- [...truncated 3.73 MB...] at org.testng.TestNG.run(TestNG.java:1104) at

[jira] [Commented] (BROOKLYN-533) AWS VM deletion failed with "Request limit exceeded"

2017-09-19 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16171879#comment-16171879 ] ASF GitHub Bot commented on BROOKLYN-533: - Github user asfgit closed the pull re

[GitHub] brooklyn-server pull request #828: BROOKLYN-533: adds maxConcurrentMachineDe...

2017-09-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/828 ---

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

2017-09-19 Thread Apache Jenkins Server
See Changes: [aled.sage] Adds FunctionSensor (for yaml FunctionFeed) [aled.sage] EmptySoftwareProcessYamlTest: make unit test [aled.sage] EmptySoftwareProcessImpl: improve setting of service.isRunning [al

[GitHub] brooklyn-server pull request #832: Add ManagementNodeStateListener support

2017-09-19 Thread Graeme-Miller
Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/832#discussion_r139721438 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/internal/ManagementNodeStateListenerManager.java --- @@ -0,0 +1,175 @@ +/*

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

2017-09-19 Thread Apache Jenkins Server
See Changes: [alex.heneveld] fix non-det failure in scaling down test -- [...truncated 3.74 MB...] at org.testng.TestNG.run(TestNG.java:1104) at org.

[GitHub] brooklyn-server pull request #824: Config for VanillaSoftwareProcess disabli...

2017-09-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/824 ---

[GitHub] brooklyn-server issue #824: Config for VanillaSoftwareProcess disabling ssh ...

2017-09-19 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/824 Thanks @drigodwin - I'll merge this, and will update the brooklyn docs. ---

[GitHub] brooklyn-server pull request #825: fix non-det failure in scaling down test

2017-09-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/825 ---

[GitHub] brooklyn-server pull request #825: fix non-det failure in scaling down test

2017-09-19 Thread aledsage
Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/825#discussion_r139718436 --- Diff: policy/src/test/java/org/apache/brooklyn/policy/autoscaling/AutoScalerPolicyPoolSizeTest.java --- @@ -101,10 +103,18 @@ public void test

[GitHub] brooklyn-library issue #129: Move chef from brooklyn-software-base to own mo...

2017-09-19 Thread tbouron
Github user tbouron commented on the issue: https://github.com/apache/brooklyn-library/pull/129 > jenkins failed with the error: See my previous comment ---

[GitHub] brooklyn-library issue #129: Move chef from brooklyn-software-base to own mo...

2017-09-19 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-library/pull/129 jenkins failed with the error: ``` java.io.IOException: remote file operation failed: /home/jenkins/jenkins-slave/workspace/brooklyn-library-pull-requests at hudson.remoting.Channe

Build failed in Jenkins: brooklyn-library-master #426

2017-09-19 Thread Apache Jenkins Server
See Changes: [aled.sage] Add icons to catalog.bom files -- Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely on qnode1 (ub

[GitHub] brooklyn-library pull request #128: Add icons to catalog.bom files

2017-09-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-library/pull/128 ---

[GitHub] brooklyn-server pull request #832: Add ManagementNodeStateListener support

2017-09-19 Thread aledsage
GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/832 Add ManagementNodeStateListener support You can merge this pull request into a Git repository by running: $ git pull https://github.com/aledsage/brooklyn-server add-managementNodeSt

[GitHub] brooklyn-server pull request #831: Add check for test resources

2017-09-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/831 ---

[GitHub] brooklyn-server pull request #831: Add check for test resources

2017-09-19 Thread drigodwin
GitHub user drigodwin opened a pull request: https://github.com/apache/brooklyn-server/pull/831 Add check for test resources Apache releases require there to be no binary files included in the release. Some of the tests depend on binary files and these tests must be skipped at rele

[GitHub] brooklyn-server pull request #830: Add support for kubernetes helm

2017-09-19 Thread andreaturli
GitHub user andreaturli opened a pull request: https://github.com/apache/brooklyn-server/pull/830 Add support for kubernetes helm You can merge this pull request into a Git repository by running: $ git pull https://github.com/andreaturli/brooklyn-server feature/container-k8s-

[GitHub] brooklyn-library issue #129: Move chef from brooklyn-software-base to own mo...

2017-09-19 Thread tbouron
Github user tbouron commented on the issue: https://github.com/apache/brooklyn-library/pull/129 @aledsage Just tested this with https://github.com/apache/brooklyn-server/pull/826, works great but requires to be rebased on the latest master + change the version in pom.xml to `0.13.0-S

[GitHub] brooklyn-server issue #826: Move chef from brooklyn-software-base to own mod...

2017-09-19 Thread tbouron
Github user tbouron commented on the issue: https://github.com/apache/brooklyn-server/pull/826 @aledsage Just tested this with https://github.com/apache/brooklyn-library/pull/129, works great but requires to be rebased on the latest master. After that, good to merge 👍 --

[GitHub] brooklyn-library issue #128: Add icons to catalog.bom files

2017-09-19 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-library/pull/128 retest this please ---

[GitHub] brooklyn-library issue #128: Add icons to catalog.bom files

2017-09-19 Thread tbouron
Github user tbouron commented on the issue: https://github.com/apache/brooklyn-library/pull/128 Great news! Then it's +1 from me :) ---

[GitHub] brooklyn-library issue #129: Move chef from brooklyn-software-base to own mo...

2017-09-19 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-library/pull/129 @tbouron changes made; ready for review + (hopefully!) merge. @tbouron For why we have ansible, chef and salt in different modules (versus databases in one module), we don't have a

[GitHub] brooklyn-library issue #128: Add icons to catalog.bom files

2017-09-19 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-library/pull/128 @tbouron yes, the rest endpoint `/children` also includes the `iconUrl` (for both the `pr-128-top-level` and `pr-128-inside-item`). I deployed the app shown below: ``` services:

[GitHub] brooklyn-library pull request #129: Move chef from brooklyn-software-base to...

2017-09-19 Thread aledsage
Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-library/pull/129#discussion_r139619615 --- Diff: software/cm/chef/src/main/resources/catalog.bom --- @@ -0,0 +1,26 @@ +# Licensed to the Apache Software Foundation (ASF) under one