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

See

[jira] [Created] (BROOKLYN-508) Persistence to aws-ec2:eu-west-1 fails in jclouds, looking up "requested location us-standard"

Aled Sage created BROOKLYN-508: -- Summary: Persistence to aws-ec2:eu-west-1 fails in jclouds, looking up "requested location us-standard" Key: BROOKLYN-508 URL: https://issues.apache.org/jira/browse/BROOKLYN-508

[jira] [Commented] (BROOKLYN-490) Failure of LoopOverGroupMembersTestCase doesn't propagate.

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

[GitHub] brooklyn-server pull request #703: BROOKLYN-490: propagate failure in LoopOv...

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/703 BROOKLYN-490: propagate failure in LoopOverGroupMembersTestCase You can merge this pull request into a Git repository by running: $ git pull

[GitHub] brooklyn-library pull request #114: Bring namespace on feature files to Kara...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-library/pull/114 --- 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-dist pull request #94: Bring namespace on feature files to Karaf fe...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-dist/pull/94 --- 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-server issue #693: jclouds TemplateOptions type coercions

Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/693 I'm currently testing this with the winrm changes and it's failing with the following exception: ``` Caused by: java.lang.IllegalArgumentException at

[GitHub] brooklyn-server pull request #700: Adds getLastConstructorProps to Recording...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/700 --- 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 #701: Bring namespace on feature files to Karaf...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/701 --- 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] [Commented] (BROOKLYN-499) Warning on rebind - UpdatingNotUpFromServiceProcessIsRunning is protected

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

[GitHub] brooklyn-client issue #42: Improve login handling.

Github user robertgmoss commented on the issue: https://github.com/apache/brooklyn-client/pull/42 @m4rkmckenna sorry, I thought I'd implemented that feature, but clearly I didn't. Yes, @geomacy the format was like that so that you could simply change the `target` in the `login`

[GitHub] brooklyn-server pull request #702: BROOKLYN-499: avoid log.warn “no-arg co...

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/702 BROOKLYN-499: avoid log.warn “no-arg constructor accessible” You can merge this pull request into a Git repository by running: $ git pull

[GitHub] brooklyn-server pull request #673: [WIP] [JCLOUDS 2.1.0] Upgrade to jclouds ...

GitHub user neykov reopened a pull request: https://github.com/apache/brooklyn-server/pull/673 [WIP] [JCLOUDS 2.1.0] Upgrade to jclouds 2.1 These are the changes I needed to implement in order to run Brooklyn against current jclouds master. Leaving them around to be reused for

[GitHub] brooklyn-server issue #673: [WIP] [JCLOUDS 2.1.0] Upgrade to jclouds 2.1

Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/673 Github closed this by mistake, confused by the already merged commit (and later reverted). Cherry-picked the same commit and reopening. --- If your project is set up for it, you can reply

[GitHub] brooklyn-client issue #42: Improve login handling.

Github user geomacy commented on the issue: https://github.com/apache/brooklyn-client/pull/42 P.S. I took it out because I thought it was cleaner to keep it simple. --- 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 #701: Bring namespace on feature files to Karaf...

GitHub user geomacy opened a pull request: https://github.com/apache/brooklyn-server/pull/701 Bring namespace on feature files to Karaf features 1.4.0 Updates namespace on the file to http://karaf.apache.org/xmlns/features/v1.4.0 which is required for 'prerequisite="true"' on

[GitHub] brooklyn-library pull request #114: Bring namespace on feature files to Kara...

GitHub user geomacy opened a pull request: https://github.com/apache/brooklyn-library/pull/114 Bring namespace on feature files to Karaf features 1.4.0 Updates namespace on the file to http://karaf.apache.org/xmlns/features/v1.4.0 which is required for 'prerequisite="true"' on

[GitHub] brooklyn-dist pull request #94: Bring namespace on feature files to Karaf fe...

GitHub user geomacy opened a pull request: https://github.com/apache/brooklyn-dist/pull/94 Bring namespace on feature files to Karaf features 1.4.0 Updates namespace on the file to http://karaf.apache.org/xmlns/features/v1.4.0 which is required for 'prerequisite="true"' on

[GitHub] brooklyn-client issue #42: Improve login handling.

Github user geomacy commented on the issue: https://github.com/apache/brooklyn-client/pull/42 @robert I think the original format was done like that to allow for multiple hosts but then we didn't actually provide that ability. What do you all think, should we support the ability to

[GitHub] brooklyn-server pull request #673: [WIP] [JCLOUDS 2.1.0] Upgrade to jclouds ...

Github user neykov closed the pull request at: https://github.com/apache/brooklyn-server/pull/673 --- 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-dist issue #92: Fix namespace on feature.xml

Github user geomacy commented on the issue: https://github.com/apache/brooklyn-dist/pull/92 Will do - I see I missed a change on line 22 here, so will submit a new one for this too! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118498184 --- Diff: cli/io/config.go --- @@ -70,3 +98,111 @@ func (config *Config) Read() { dec := json.NewDecoder(fileToRead) dec.Decode()

Re: Code Contribution: Container Service

Yes, The next step is a vote, and then I am happy to go through the IP clearance process [0][1] (as Richard did for the CLI [2] etc contributions) as it must be done by a PMC member. I'll do this (unless anyone objects) once the vote is proposed and passes... Andrew. 0.

[GitHub] brooklyn-client issue #42: Improve login handling.

Github user m4rkmckenna commented on the issue: https://github.com/apache/brooklyn-client/pull/42 @robertgmoss I was un aware we had that functionality ... How is it supposed to work? I tried ``` br login br apps # Returns host1 apps br login br apps #

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118496514 --- Diff: cli/io/config.go --- @@ -70,3 +98,111 @@ func (config *Config) Read() { dec := json.NewDecoder(fileToRead)

[GitHub] brooklyn-server issue #700: Adds getLastConstructorProps to RecordingWinRmTo...

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/700 LGTM; good to merge once jenkins build has finished. --- 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

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118494527 --- Diff: cli/commands/login.go --- @@ -53,6 +58,30 @@ func (cmd *Login) Metadata() command_metadata.CommandMetadata { } } +

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118494407 --- Diff: cli/io/config.go --- @@ -70,3 +98,111 @@ func (config *Config) Read() { dec := json.NewDecoder(fileToRead) dec.Decode()

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118494358 --- Diff: cli/br/brooklyn.go --- @@ -30,27 +30,14 @@ import ( "path/filepath" ) -func getNetworkCredentialsFromConfig(yamlMap

[GitHub] brooklyn-server pull request #700: Adds getLastConstructorProps to Recording...

GitHub user nakomis opened a pull request: https://github.com/apache/brooklyn-server/pull/700 Adds getLastConstructorProps to RecordingWinRmTool Adds `getLastConstructorProps()` to `RecordingWinRmTool`, in line with `RecordingSshTool` You can merge this pull request into a Git

[GitHub] brooklyn-dist pull request #92: Fix namespace on feature.xml

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

Re: Is nested DSL supported?

From my experience yes, DSL can be nested. If you get( Config) and the raw value is not from T but a "DSL" value then the DeferredSupplier value will be evaluated. Nested DeferredSupplier are also evaluated if necessary. However if your raw value is a list of "DSL" and you `get(Config)`

[GitHub] brooklyn-dist issue #92: Fix namespace on feature.xml

Github user neykov commented on the issue: https://github.com/apache/brooklyn-dist/pull/92 LGTM. @geomacy can you do the same change for `brooklyn-server`. --- 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

Is nested DSL supported?

Can Brooklyn DSL be nested? I want to retrieve a secret from vault using the external provider and the following syntax works: $brooklyn:external("vault", "name-of-secret") However, the secret I want to retrieve is variable and depends on the way the blueprint is deployed. I, therefore, want

Jenkins build is back to normal : brooklyn-server-master #598

See

[GitHub] brooklyn-client issue #42: Improve login handling.

Github user robertgmoss commented on the issue: https://github.com/apache/brooklyn-client/pull/42 @geomacy I think it is a shame to lose the caching of credentials for multiple Brooklyn servers and the ability to switch between them without re-entering credentials. --- If your

[GitHub] brooklyn-server pull request #699: Fix type-coercion to/from arrays

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/699 Fix type-coercion to/from arrays Motivated by investigating the list->iterable coercion problems (see https://github.com/apache/brooklyn-server/pull/698), I also added tests and fixed

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

See -- Started by an SCM change [EnvInject] - Loading node environment variables. ERROR: SEVERE ERROR occurs org.jenkinsci.lib.envinject.EnvInjectException:

[GitHub] brooklyn-server pull request #698: Fix type-coercion to Iterable

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/698 --- 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 #698: Fix type-coercion to Iterable

Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/698 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 so,

[GitHub] brooklyn-server issue #698: Fix type-coercion to Iterable

Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/698 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 so, or

[GitHub] brooklyn-server issue #698: Fix type-coercion to Iterable

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/698 @m4rkmckenna it's an issue when we try to coerce from some complex yaml values to jclouds parameter values - e.g. we try to reflectively call `AzureTemplateOptions.secrets(Iterable)` if

[GitHub] brooklyn-server issue #698: Fix type-coercion to Iterable

Github user ygy commented on the issue: https://github.com/apache/brooklyn-server/pull/698 @geomacy You can find at https://gist.github.com/ygy/59c793028457c1f1f5a1f027df047fa6 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] brooklyn-server issue #698: Fix type-coercion to Iterable

Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/698 LGTM. @ygy what was your blueprint? --- 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 #698: Fix type-coercion to Iterable

Github user ygy commented on the issue: https://github.com/apache/brooklyn-server/pull/698 @aledsage LTGM. Tests are passing for me. Tested successfully with a BP which fails without it. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] brooklyn-server pull request #698: Fix type-coercion to Iterable

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/698 Fix type-coercion to Iterable You can merge this pull request into a Git repository by running: $ git pull https://github.com/aledsage/brooklyn-server fix-TypeCoercion-to-iterable

Re: Code Contribution: Container Service

Hi all, The licences have been updated in [1]. Richard is the next step a vote by the community to see if this is actually wanted within brooklyn? Cheers Mark [1] https://github.com/cloudsoft/brooklyn-container-service/ On 20 May 2017 at 18:35, Andrew Kennedy

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118458597 --- Diff: cli/io/config.go --- @@ -70,3 +98,111 @@ func (config *Config) Read() { dec := json.NewDecoder(fileToRead)

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118458240 --- Diff: cli/br/brooklyn.go --- @@ -30,27 +30,14 @@ import ( "path/filepath" ) -func

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118455814 --- Diff: cli/commands/login.go --- @@ -53,6 +58,30 @@ func (cmd *Login) Metadata() command_metadata.CommandMetadata { } }

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118458766 --- Diff: cli/br/brooklyn.go --- @@ -30,27 +30,14 @@ import ( "path/filepath" ) -func

[jira] [Commented] (BROOKLYN-477) Add docs for target machine requirements/expectations

[ https://issues.apache.org/jira/browse/BROOKLYN-477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16024480#comment-16024480 ] ASF GitHub Bot commented on BROOKLYN-477: - Github user drigodwin commented on a diff in the pull

[jira] [Commented] (BROOKLYN-477) Add docs for target machine requirements/expectations

[ https://issues.apache.org/jira/browse/BROOKLYN-477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16024477#comment-16024477 ] ASF GitHub Bot commented on BROOKLYN-477: - Github user drigodwin commented on a diff in the pull

[jira] [Commented] (BROOKLYN-477) Add docs for target machine requirements/expectations

[ https://issues.apache.org/jira/browse/BROOKLYN-477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16024482#comment-16024482 ] ASF GitHub Bot commented on BROOKLYN-477: - Github user drigodwin commented on a diff in the pull

[jira] [Commented] (BROOKLYN-477) Add docs for target machine requirements/expectations

[ https://issues.apache.org/jira/browse/BROOKLYN-477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16024479#comment-16024479 ] ASF GitHub Bot commented on BROOKLYN-477: - Github user drigodwin commented on a diff in the pull

[jira] [Commented] (BROOKLYN-477) Add docs for target machine requirements/expectations

[ https://issues.apache.org/jira/browse/BROOKLYN-477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16024481#comment-16024481 ] ASF GitHub Bot commented on BROOKLYN-477: - Github user drigodwin commented on a diff in the pull

[GitHub] brooklyn-docs pull request #192: BROOKLYN-477: add “Provisioned Machine Re...

Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/192#discussion_r118453774 --- Diff: guide/locations/provisioned-machine-requirements.md --- @@ -0,0 +1,161 @@ +--- +title: Provisioned Machine Requirements

[GitHub] brooklyn-docs pull request #192: BROOKLYN-477: add “Provisioned Machine Re...

Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/192#discussion_r118453118 --- Diff: guide/locations/provisioned-machine-requirements.md --- @@ -0,0 +1,161 @@ +--- +title: Provisioned Machine Requirements

[GitHub] brooklyn-docs pull request #192: BROOKLYN-477: add “Provisioned Machine Re...

Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/192#discussion_r118455694 --- Diff: guide/locations/provisioned-machine-requirements.md --- @@ -0,0 +1,161 @@ +--- +title: Provisioned Machine Requirements

[GitHub] brooklyn-docs pull request #192: BROOKLYN-477: add “Provisioned Machine Re...

Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/192#discussion_r118456684 --- Diff: guide/locations/provisioned-machine-requirements.md --- @@ -0,0 +1,161 @@ +--- +title: Provisioned Machine Requirements

[GitHub] brooklyn-docs pull request #192: BROOKLYN-477: add “Provisioned Machine Re...

Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/192#discussion_r118454613 --- Diff: guide/locations/provisioned-machine-requirements.md --- @@ -0,0 +1,161 @@ +--- +title: Provisioned Machine Requirements

[GitHub] brooklyn-server pull request #690: External ip config for winrm byon locatio...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/690#discussion_r118450024 --- Diff: core/src/main/java/org/apache/brooklyn/location/byon/ByonLocationResolver.java --- @@ -152,49 +153,54 @@ protected ConfigBag

[GitHub] brooklyn-server pull request #690: External ip config for winrm byon locatio...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/690#discussion_r118449716 --- Diff: core/src/main/java/org/apache/brooklyn/location/byon/ByonLocationResolver.java --- @@ -152,49 +153,55 @@ protected ConfigBag