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

2017-03-06 Thread Apache Jenkins Server
See

Re: [PROPOSAL] Remove groovy dependency/support

2017-03-06 Thread Alex Heneveld
+1 Minor comments: > 2. For a "MethodEffector", the effector invocation goes through > `GroovyJavaMethods.invokeMethodOnMetaClass`. This calls into groovy > code to find the method that matches the given arguments. Reflections.findMethodMaybe and invokeMethodWithArgs might be drop-in

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

2017-03-06 Thread Apache Jenkins Server
See -- [...truncated 26.74 KB...] [INFO] CAMP Base .. SKIPPED [INFO] Brooklyn Test Support .. SKIPPED [INFO]

Re: [PROPOSAL] Remove java 7 support

2017-03-06 Thread Geoff Macartney
hi Aled, sure, I'll have a look at that. cheers Geoff On Mon, 6 Mar 2017 at 18:26 Aled Sage wrote: > Hi all, > > There have been six +1s (including myself) and no negatives, so let's do > this. > > --- > > For Robert's question, I did some investigation for use of

[PROPOSAL] Remove groovy dependency/support

2017-03-06 Thread Aled Sage
Hi all, I propose we deprecate and then remove the remaining groovy code from Brooklyn: deprecate for 0.11.0, and delete a couple of releases later. I think we should treat Groovy like any other JVM-based language that users might want to use: it's the user's responsibility; we present a

Re: [PROPOSAL] Remove java 7 support

2017-03-06 Thread Aled Sage
Hi all, There have been six +1s (including myself) and no negatives, so let's do this. --- For Robert's question, I did some investigation for use of lambdas as config key values. They don't persist correctly unfortunately [1], so we shouldn't use them anywhere that could end up in

[jira] [Created] (BROOKLYN-448) Support java 8 lambdas as config key values

2017-03-06 Thread Aled Sage (JIRA)
Aled Sage created BROOKLYN-448: -- Summary: Support java 8 lambdas as config key values Key: BROOKLYN-448 URL: https://issues.apache.org/jira/browse/BROOKLYN-448 Project: Brooklyn Issue Type:

[GitHub] brooklyn-docs pull request #150: Add deprecated functionality which can't be...

2017-03-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/150 --- 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

Jenkins build is back to stable : brooklyn-master-build #896

2017-03-06 Thread Apache Jenkins Server
See

Re: How to express aggregator in yaml

2017-03-06 Thread Graham Ashby
Hi Sam I've got an iterator in my template: <#list config['cm_host_port_list'] as cm_item> http://$ {cm_item}/p2pd/servlet So, that's why I have an array. So, I think I could do something with a Predicate, but I have no idea how I would wire this into the yaml. Thanks Graham

Jenkins build became unstable: brooklyn-master-build #895

2017-03-06 Thread Apache Jenkins Server
See

Re: How to express aggregator in yaml

2017-03-06 Thread Sam Corbett
Hi Graham, May I ask how you intend to use the values in your template? If it would be sufficient to combine them into a single string you should use a `Joiner` enricher whose minimum size is the initial size of the cluster: brooklyn.config: cluster.initial.size: 4

[GitHub] brooklyn-server pull request #581: EntityConfigTest: add tests for Immediate...

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

[jira] [Created] (BROOKLYN-447) waitForSshable deprecated without replacement

2017-03-06 Thread Graeme Miller (JIRA)
Graeme Miller created BROOKLYN-447: -- Summary: waitForSshable deprecated without replacement Key: BROOKLYN-447 URL: https://issues.apache.org/jira/browse/BROOKLYN-447 Project: Brooklyn Issue

Re: [PROPOSAL] Remove java 7 support

2017-03-06 Thread Geoff Macartney
hi all, any more thoughts on this? What concrete steps do we want to take, and when? What can I do to help out? Plus, has anyone any views on Robert's question? cheers Geoff On Mon, 27 Feb 2017 at 14:21 Robert Moss wrote: > Aled, > > WRT Java 8 features. Is

[GitHub] brooklyn-ui issue #36: Updated logout step to work in karaf mode

2017-03-06 Thread sjcorbett
Github user sjcorbett commented on the issue: https://github.com/apache/brooklyn-ui/pull/36 @bostko thanks for the update. I've confirmed that the browser requires re-authentication in both cases. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] brooklyn-ui pull request #36: Updated logout step to work in karaf mode

2017-03-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-ui/pull/36 --- 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 is

[jira] [Commented] (BROOKLYN-323) Inconsistent logout behavior for Basic Authentication

2017-03-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-323?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15897395#comment-15897395 ] ASF GitHub Bot commented on BROOKLYN-323: - Github user asfgit closed the pull request at:

[GitHub] brooklyn-server issue #581: EntityConfigTest: add tests for ImmediateSupplie...

2017-03-06 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/581 Jenkins failed with an unrelated test: `HttpCommandEffectorYamlRebindTest.testRebindWhenHealthy:64->runRebindWhenIsUp:81`. I'll do a `git commit --amend; git push -f` to kick-off jenkins

[jira] [Commented] (BROOKLYN-323) Inconsistent logout behavior for Basic Authentication

2017-03-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-323?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15897249#comment-15897249 ] ASF GitHub Bot commented on BROOKLYN-323: - Github user asfgit closed the pull request at:

[GitHub] brooklyn-server pull request #578: BROOKLYN-323: Use proper WWW-Authorizatio...

2017-03-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/578 --- 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 #565: Be truly immediate/non-blocking more ofte...

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

[GitHub] brooklyn-server pull request #581: EntityConfigTest: add tests for Immediate...

2017-03-06 Thread aledsage
GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/581 EntityConfigTest: add tests for ImmediateSupplier You can merge this pull request into a Git repository by running: $ git pull https://github.com/aledsage/brooklyn-server

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

2017-03-06 Thread geomacy
Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/338 Thanks @ahgittin I will have a look at #573 hopefully today or very soon. Nice to have extra tests! Regarding the persistence, basically in the persisted XML my change replaces

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

2017-03-06 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/338 As for `catalogItemId`, I think we should continue to populate and persist this. It is useful to know from whence something came, and for audit purposes this will become even more

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

2017-03-06 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/338 Hi @geomacy - have added another commit to #573 ; hadn't realized other inheritance semantics re catalog item id. It now has tests and several explicit TODO comments which this PR should

[GitHub] brooklyn-server issue #573: [WIP] Do not runtime-inherit catalog item

2017-03-06 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/573 @neykov Prescient suggestions. Completely forget there were _two_ other paths where `catalogItemId` was being set to an inheritance/context value. It doesn't cause problems with

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

2017-03-06 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/565 @aledsage rebased and ready for review --- 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