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

2017-03-07 Thread Apache Jenkins Server
See -- [...truncated 7.48 MB...] [INFO] --- apache-rat-plugin:0.12:check (default) @ deb-packaging --- [INFO] Enabled default license matchers. [INFO] Will parse SCM ignores for

Jenkins build is back to normal : brooklyn-library-master #205

2017-03-07 Thread Apache Jenkins Server
See

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

2017-03-07 Thread Apache Jenkins Server
See

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

2017-03-07 Thread Apache Jenkins Server
See -- [...truncated 7.11 MB...] [JENKINS] Archiving to

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

2017-03-07 Thread Apache Jenkins Server
See Changes: [alex.heneveld] bump http version -- Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely on H14 (ubuntu) in

[GitHub] brooklyn-library issue #73: bump http version

2017-03-07 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-library/pull/73 https://github.com/apache/brooklyn-server/pull/547 merged - merging this one as well. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] brooklyn-library pull request #73: bump http version

2017-03-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-library/pull/73 --- 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 pull request #570: Terminate winrm client context on Brookly...

2017-03-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/570 --- 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 pull request #547: [WIP] CXF upgrade

2017-03-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/547 --- 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 #570: Terminate winrm client context on Brooklyn exit

2017-03-07 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/570 Tested successfully (together with #547) by deploying a blueprint and running the following tests: * WinRmMachineLocationLiveTest * JcloudsWinrmingLiveTest (AWS only) *

[GitHub] brooklyn-server issue #547: [WIP] CXF upgrade

2017-03-07 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/547 Tested successfully (together with #570) by deploying a blueprint and running the following tests: * WinRmMachineLocationLiveTest * JcloudsWinrmingLiveTest (AWS only) *

[GitHub] brooklyn-server pull request #547: [WIP] CXF upgrade

2017-03-07 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/547#discussion_r104778800 --- Diff: pom.xml --- @@ -117,8 +117,10 @@ 2.3.0_3 2.7.5 -3.1.4 -4.4.1 +

[GitHub] brooklyn-server issue #570: Terminate winrm client context on Brooklyn exit

2017-03-07 Thread geomacy
Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/570 👍 LGTM. I had to merge this with #547 first, and resolve a conflict on the main pom.xml - I just picked the latest version of all dependencies. I'll have to take your word

[GitHub] brooklyn-server issue #570: Terminate winrm client context on Brooklyn exit

2017-03-07 Thread geomacy
Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/570 I've been able to run `JcloudsWinrmingLiveTest` successfully on AWS and GCE (by merging 547 first, needs a wee bit of conflict resolution). Will review the code text now. --- If your

[GitHub] brooklyn-server pull request #582: Fix Entitlements Integration tests

2017-03-07 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-server/pull/582 Fix Entitlements Integration tests Fixes a bunch of `*ApiEntitlementsTest` failures as seen in https://builds.apache.org/view/Brooklyn/job/brooklyn-integration-tests/64/testngreports/

Re: How to express aggregator in yaml

2017-03-07 Thread Graham Ashby
Thanks, Sam Yes, that did work. I was thinking of doing something like that myself, but I didn't know how to translate the Java concepts into yaml. Is there a rule of thumb about how to do that translation? G From: Sam Corbett To:

[jira] [Commented] (BROOKLYN-358) jclouds "swift" missing from karaf build

2017-03-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15899777#comment-15899777 ] ASF GitHub Bot commented on BROOKLYN-358: - Github user neykov commented on the issue:

[GitHub] brooklyn-server issue #374: BROOKLYN-358: karaf includes jclouds-api-swift

2017-03-07 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/374 The fix to get SL working is to configure your object store location with: ``` jclouds.keystone.credential-type: tempAuthCredentials ``` --- If your project is set up for it, you

[GitHub] brooklyn-docs pull request #153: Remove bash prompt so users can directly co...

2017-03-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/153 --- 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-docs pull request #153: Remove bash prompt so users can directly co...

2017-03-07 Thread tbouron
GitHub user tbouron opened a pull request: https://github.com/apache/brooklyn-docs/pull/153 Remove bash prompt so users can directly copy commands You can merge this pull request into a Git repository by running: $ git pull https://github.com/tbouron/brooklyn-docs patch-2

[GitHub] brooklyn-server issue #565: Be truly immediate/non-blocking more often

2017-03-07 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/565 Failure at https://builds.apache.org/job/brooklyn-server-pull-requests/1820/ seems due to build environment -- ``` [INFO] Installing

[jira] [Commented] (BROOKLYN-358) jclouds "swift" missing from karaf build

2017-03-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15899699#comment-15899699 ] ASF GitHub Bot commented on BROOKLYN-358: - Github user sjcorbett commented on the issue:

[GitHub] brooklyn-server issue #374: BROOKLYN-358: karaf includes jclouds-api-swift

2017-03-07 Thread sjcorbett
Github user sjcorbett commented on the issue: https://github.com/apache/brooklyn-server/pull/374 I've run into the issues discussed above. Was either issue filed? Is Brooklyn able to persist to SoftLayer's object store? --- If your project is set up for it, you can reply to this

Re: Uploading ZIPs for a better dev workflow

2017-03-07 Thread Svetoslav Neykov
Having quickly scanned the thread I still don't see a reason why the requirement to have a consistent naming for the bundle itself which is not user visible? Could instead have behaviour similar to the machinNamer here. Slight preference for optional symbolicName when it's a jar file. Though I

Re: [PROPOSAL/DISCUSSION] yaml DSL for invoking effectors

2017-03-07 Thread Alex Heneveld
Hi Geoff - > Would it be too conservative to suggest that we don't do any more on this > until we do have a clear and compelling use-case in mind, when it will be > (I hope) a lot easier to see what way it should be done? On the contrary -- it would be foolish to do something here /without/ a

[GitHub] brooklyn-server issue #485: [WIP] `BundleMaker` REST call allowing to add ZI...

2017-03-07 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/485 (Marked issue as WIP while I implement the revised proposal.) --- 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 issue #565: Be truly immediate/non-blocking more often

2017-03-07 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/565 All tests pass for me locally with this. And re-reading I think @aledsage we're not in disagreement; I simply overlooked the subtlety of `getImmediate` having explicit permission to

Re: Uploading ZIPs for a better dev workflow

2017-03-07 Thread Alex Heneveld
Considering the points made I'm pretty sure it's the case that requiring a symbolic name and version in the BOM presents little if any unnecessary burden, in which case I'm persuaded of the "one true way" philosophy. I really like it. :) Can anyone think of a case where it would be a

[GitHub] brooklyn-server issue #547: [WIP] CXF upgrade

2017-03-07 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/547 Winrm4j 5.0 released. Will merge this shortly. @aledsage could you remove the WIP tag. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] brooklyn-server issue #565: Be truly immediate/non-blocking more often

2017-03-07 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/565 Thanks for the explanation. I realized from your (excellent) tests that the thing we want to allow is to permit an `ImmediateSupplier` to block, which was permitted in my code if it was a

[GitHub] brooklyn-server issue #485: `BundleMaker` REST call allowing to add ZIP/JAR ...

2017-03-07 Thread geomacy
Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/485 Having thought some more about such a default I suggested the following on the mailing list: I'd say let's not even be that flexible - my feeling is that the more flexible we are,

[GitHub] brooklyn-server issue #565: Be truly immediate/non-blocking more often

2017-03-07 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/565 @neykov you beat me to replying - agreed :-) @ahgittin to give some more background, the original use-case for "immediate" stemmed from a failure in clocker:

[NOTICE] switch to Java 8 as "official target" for Apache Brooklyn Jenkins builds

2017-03-07 Thread Geoff Macartney
hello all, In the mail thread "[PROPOSAL] Remove java 7 support" [1] it has been proposed to "1. Update our jenkins etc to always build/run with Java 8." I intend to start changing the build configurations, and I'm going to propose doing so from tomorrow (Wednesday) at 10.00 UTC. I'm hoping

[GitHub] brooklyn-server issue #565: Be truly immediate/non-blocking more often

2017-03-07 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/565 To add some more context (without having looked at the changes) - before adding ImmediateSupplier tasks with a short timeout (the way we implemented "immediate") would get cancelled even

[GitHub] brooklyn-server issue #565: Be truly immediate/non-blocking more often

2017-03-07 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/565 @aledsage I think I better understand now. It isn't the sleeps that are the offenders -- it is the DST/map interplay. Still getting my head round it but wanted to say that. --- If your

[GitHub] brooklyn-server pull request #565: Be truly immediate/non-blocking more ofte...

2017-03-07 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/565#discussion_r104656924 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/AbstractConfigurationSupportInternal.java --- @@ -139,17 +132,19 @@ public T call() {

[GitHub] brooklyn-server issue #565: Be truly immediate/non-blocking more often

2017-03-07 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/565 @aledsage Sorry for the confusion. Thanks for long exposition. I thought almost everyone hated the "wait" as part of "immediate", from discussions ages ago somewhere. Waiting 200ms is

Re: How to express aggregator in yaml

2017-03-07 Thread Sam Corbett
Hi Graham, You need a value for resources.runtime.latch or resources.install.latch that indicates that all values in cm_host_port_list are non-null. My suggestion is to use a parallel set of of sensors rather than trying to do everything with cm_host_port_list: have each cluster member

Re: How to express aggregator in yaml

2017-03-07 Thread Sam Corbett
I managed to miss the references out: 1. https://brooklyn.apache.org/v/latest/yaml/yaml-reference.html#dsl-commands 2. https://brooklyn.apache.org/v/latest/misc/javadoc/org/apache/brooklyn/util/collections/QuorumCheck.QuorumChecks.html On 07/03/2017 12:31, Sam Corbett wrote: Hi Graham,

[GitHub] brooklyn-server issue #338: Support nested catalog item definitions - testCa...

2017-03-07 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/338 @geomacy I think we want `catalogItemId` explicit as the catalog item ID of the item, in addition to the "hierarchy" (which is maybe better called `catalogItemsSearchPath` ?). It is a

[GitHub] brooklyn-server issue #485: `BundleMaker` REST call allowing to add ZIP/JAR ...

2017-03-07 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/485 @geomacy yes, either insist on a version of default to `0.0.0.SNAPSHOT` ... no strong feelings here --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] brooklyn-docs pull request #152: Docs for SharedLocationSecurityGroupCustomi...

2017-03-07 Thread iyovcheva
GitHub user iyovcheva opened a pull request: https://github.com/apache/brooklyn-docs/pull/152 Docs for SharedLocationSecurityGroupCustomizer You can merge this pull request into a Git repository by running: $ git pull https://github.com/iyovcheva/brooklyn-docs

Re: [PROPOSAL] Remove groovy dependency/support

2017-03-07 Thread Martin Harris
Would this include the removal of the Groovy console? I'd kinda miss that :-( Cheers M On 7 March 2017 at 09:10, Geoff Macartney wrote: > +1 certainly sounds like a good plan > > On Tue, 7 Mar 2017 at 01:17 Alex Heneveld >

Re: [PROPOSAL] Remove groovy dependency/support

2017-03-07 Thread Geoff Macartney
+1 certainly sounds like a good plan On Tue, 7 Mar 2017 at 01:17 Alex Heneveld wrote: > > +1 > > Minor comments: > > > 2. For a "MethodEffector", the effector invocation goes through > > `GroovyJavaMethods.invokeMethodOnMetaClass`. This calls into groovy >