[GitHub] brooklyn-server issue #572: Increase timeout in ReachableSocketFinderTest

Github user sjcorbett commented on the issue: https://github.com/apache/brooklyn-server/pull/572 @geomacy @neykov could you review this? --- 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

[GitHub] brooklyn-server issue #462: Inherit config default values

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/462 Seems there may be a pathway by which REST does not populate an entity's `EntityType.configKeys` in this PR, reported by @sjcorbett . Looking in to this now. --- If your project is set

[GitHub] brooklyn-server pull request #572: wip: debug ReachableSocketFinderTest.getR...

GitHub user sjcorbett opened a pull request: https://github.com/apache/brooklyn-server/pull/572 wip: debug ReachableSocketFinderTest.getReachableSockets Don't merge. You can merge this pull request into a Git repository by running: $ git pull

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

See

Re: [PROPOSAL] Remove java 7 support

Aled, WRT Java 8 features. Is it still recommended to use named inner classes over anonymous inner classes? Can we now simplify those areas to use lambdas? There are a number of classes now built into Java that were previously provided by Guava etc. e.g. Optional, Predicate - should we start

Re: [PROPOSAL] Remove java 7 support

+1. However because of intense usage of Lambdas in Java 8, we should bring up guide lines for writing persistable code [1] and elaborate more on the paragraph from [1] "Never use anonymous inner classes - even in static contexts. The auto-generated class names are brittle, making backwards

Re: [PROPOSAL] Remove java 7 support

Like the others: +1 On Mon, 27 Feb 2017 at 13:47 Mark Mc Kenna wrote: > +1 > > *Mark McKenna* > > *Web :: markmckenna.ie * > > *Work :: mark.mcke...@cloudsoft.io * > > *PGP :: A7A9 24DE 638C 681A 8DEA FAD4 2B5D C759 B1EB

[GitHub] brooklyn-ui pull request #42: Use the "itemType" to redirect the user to the...

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

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

See -- [...truncated 252.88 KB...] 2017-02-27 13:40:48,565 INFO TESTNG INVOKING: "Surefire test" - org.apache.brooklyn.util.text.QuotedStringTokenizerTest.testTokenizing()

Re: [PROPOSAL] Remove java 7 support

+1 *Mark McKenna* *Web :: markmckenna.ie * *Work :: mark.mcke...@cloudsoft.io * *PGP :: A7A9 24DE 638C 681A 8DEA FAD4 2B5D C759 B1EB 76A7 * On 27 February 2017 at 13:42, Andrea Turli

[jira] [Commented] (BROOKLYN-444) Effector call works from GUI but fails from CLI

[ https://issues.apache.org/jira/browse/BROOKLYN-444?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15885799#comment-15885799 ] Geoff Macartney commented on BROOKLYN-444: -- Looks like the CLI is sending a parameter for the

[jira] [Assigned] (BROOKLYN-444) Effector call works from GUI but fails from CLI

[ https://issues.apache.org/jira/browse/BROOKLYN-444?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Geoff Macartney reassigned BROOKLYN-444: Assignee: Geoff Macartney > Effector call works from GUI but fails from CLI >

Re: [PROPOSAL] Remove java 7 support

+1 Il 27/feb/2017 14:35, "Geoff Macartney" ha scritto: > +1 > > I have been hoping we could get round to this soon, think it's a good idea. > > > > On Mon, 27 Feb 2017 at 13:09 Aled Sage wrote: > > > Hi all, > > > > I propose we switch to

[PROPOSAL] Remove java 7 support

Hi all, I propose we switch to Java 8 for Apache Brooklyn (i.e. require Java 8; remove support for using Java 7 when running Brooklyn). This will make our testing simpler, improving coverage (e.g. we don't currently test everything with Brooklyn running both on Java 7 and Java 8). It will

[GitHub] brooklyn-server issue #571: Better error reporting

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/571 Test failure due to Apache build system configuration: `insufficient memory for the Java Runtime Environment to continue` --- If your project is set up for it, you can reply to this

[GitHub] brooklyn-server pull request #567: Add catalog "itemType" to the CatalogItem...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/567 --- 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 #571: Better error reporting

GitHub user ahgittin opened a pull request: https://github.com/apache/brooklyn-server/pull/571 Better error reporting populate error field of `ApiError`, and better logging on REST errors You can merge this pull request into a Git repository by running: $ git pull

[GitHub] brooklyn-server issue #567: Add catalog "itemType" to the CatalogItemSummary...

Github user tbouron commented on the issue: https://github.com/apache/brooklyn-server/pull/567 @drigodwin Tests pass on Jenkins now --- 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

Re: Beginner's issue in launching brooklyn from vagrant

Please ignore. Obviously wrong reply :). Svet. > On 26.02.2017 г., at 20:34, Svetoslav Neykov > wrote: > > +1 (non-binding) > > I successfully ran: > * the verification script on Java 7 > * the live tests for the vagrant provider (with pre-loaded

[GitHub] brooklyn-server issue #567: Add catalog "itemType" to the CatalogItemSummary...

Github user tbouron commented on the issue: https://github.com/apache/brooklyn-server/pull/567 @drigodwin Tried 2 times locally, the tests pass. I'll trigger another jenkins build to make sure it was not a false-positive --- If your project is set up for it, you can reply to this

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

See -- [...truncated 252.65 KB...] 2017-02-27 09:56:15,103 INFO TESTNG INVOKING: "Surefire test" - org.apache.brooklyn.util.text.QuotedStringTokenizerTest.testQuoting() 2017-02-27

[GitHub] brooklyn-docs pull request #148: Adds doc for urlEncode DSL

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/148 --- 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 issue #148: Adds doc for urlEncode DSL

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-docs/pull/148 @neykov's comment addressed; @drigodwin deferred for a future PR. Merging now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] brooklyn-docs pull request #148: Adds doc for urlEncode DSL

Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/148#discussion_r103166453 --- Diff: guide/yaml/yaml-reference.md --- @@ -213,6 +213,9 @@ concise DSL defined here: `$brooklyn:root()`. *

[GitHub] brooklyn-dist pull request #86: switch vagrant boxes to bento images

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-dist/pull/86 --- 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-dist issue #86: switch vagrant boxes to bento images

Github user geomacy commented on the issue: https://github.com/apache/brooklyn-dist/pull/86 looks good thanks @johnmccabe - 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

[GitHub] brooklyn-docs pull request #148: Adds doc for urlEncode DSL

Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/148#discussion_r103165332 --- Diff: guide/yaml/yaml-reference.md --- @@ -213,6 +213,9 @@ concise DSL defined here: `$brooklyn:root()`. *