[jira] [Assigned] (BROOKLYN-280) br cli fails to login to brooklyn instances with self-signed SSL certs

2016-05-23 Thread John McCabe (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-280?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John McCabe reassigned BROOKLYN-280: Assignee: John McCabe > br cli fails to login to brooklyn instances with self-signed SSL

Re: Naming of Brooklyn Client on OSX Homebrew

2016-05-23 Thread Richard Downer
On 23 May 2016 at 10:51, Aled Sage wrote: > +1, thanks John! > > Richard: it will be great to update the brooklyn docs to point people at > `brew install apache-brooklyn-cli`. How do we officially endorse this so we > can update the docs (once John's Homebrew PR is merged)?

[GitHub] brooklyn-server pull request: [WIP] Add a ScriptEffector using JSR...

2016-05-23 Thread grkvlt
GitHub user grkvlt opened a pull request: https://github.com/apache/brooklyn-server/pull/153 [WIP] Add a ScriptEffector using JSR-223 embedded scripting This is an attempt to add scripting language support to Brooklyn blueprints, so that effectors can be added in various languages

[jira] [Updated] (BROOKLYN-280) br cli fails to login to brooklyn instances with self-signed SSL certs

2016-05-23 Thread John McCabe (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-280?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John McCabe updated BROOKLYN-280: - Description: Attempt to log into Brooklyn with a cert generated following the instructions on

[jira] [Commented] (BROOKLYN-276) TomcatServer entity does not install JRE if one is not present

2016-05-23 Thread John McCabe (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15296567#comment-15296567 ] John McCabe commented on BROOKLYN-276: -- assigning to myself so I remember to grab the env yml >

[jira] [Assigned] (BROOKLYN-276) TomcatServer entity does not install JRE if one is not present

2016-05-23 Thread John McCabe (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-276?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John McCabe reassigned BROOKLYN-276: Assignee: John McCabe > TomcatServer entity does not install JRE if one is not present >

[GitHub] brooklyn-server pull request: Resolve AddSensor and AddEffector co...

2016-05-23 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/152#discussion_r64229699 --- Diff: core/src/main/java/org/apache/brooklyn/core/sensor/password/CreatePasswordSensor.java --- @@ -33,23 +34,21 @@ public static

[jira] [Comment Edited] (BROOKLYN-280) br cli fails to login to brooklyn instances with self-signed SSL certs

2016-05-23 Thread John McCabe (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-280?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15296560#comment-15296560 ] John McCabe edited comment on BROOKLYN-280 at 5/23/16 4:04 PM: --- I'll fix

[jira] [Commented] (BROOKLYN-280) br cli fails to login to brooklyn instances with self-signed SSL certs

2016-05-23 Thread John McCabe (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-280?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15296560#comment-15296560 ] John McCabe commented on BROOKLYN-280: -- I'll fix this, have a test client that works, will need to

[jira] [Comment Edited] (BROOKLYN-279) Catalog API lets you submit invalid yaml

2016-05-23 Thread John McCabe (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-279?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15296655#comment-15296655 ] John McCabe edited comment on BROOKLYN-279 at 5/23/16 5:05 PM: --- Changing

[jira] [Commented] (BROOKLYN-279) Catalog API lets you submit invalid yaml

2016-05-23 Thread John McCabe (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-279?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15296655#comment-15296655 ] John McCabe commented on BROOKLYN-279: -- Changing issue title following feedback from [~m4rkmckenna]

[GitHub] brooklyn-client pull request: Support getting arguments from URL o...

2016-05-23 Thread geomacy
Github user geomacy commented on the pull request: https://github.com/apache/brooklyn-client/pull/19#issuecomment-221057396 A fair point. I will try to make some time to extend https://github.com/apache/brooklyn-client/blob/master/test/test.sh and then we can find somewhere to run

[GitHub] brooklyn-client pull request: Support getting arguments from URL o...

2016-05-23 Thread geomacy
Github user geomacy commented on the pull request: https://github.com/apache/brooklyn-client/pull/19#issuecomment-221061653 Oh that sort of thing - yes it would be good to introduce more of that too. Could start gradually building up tests for each command, starting with the most

Scripted Effectors with JSR-223

2016-05-23 Thread Andrew Kennedy
Hi. Currently the only initializer for new effectors that can be used in a YAML blueprint is the SshCommandEffector, which creates an effector that can execute an SSH command on an entity. This is useful, but it would be good to have more flexibility, including access to some of the Brooklyn

[jira] [Created] (BROOKLYN-281) Enricher missing `brooklyn.config`: all config values ignored silently

2016-05-23 Thread Aled Sage (JIRA)
Aled Sage created BROOKLYN-281: -- Summary: Enricher missing `brooklyn.config`: all config values ignored silently Key: BROOKLYN-281 URL: https://issues.apache.org/jira/browse/BROOKLYN-281 Project:

[GitHub] brooklyn-server pull request: Add support for JSR-223 scripting of...

2016-05-23 Thread grkvlt
Github user grkvlt commented on the pull request: https://github.com/apache/brooklyn-server/pull/153#issuecomment-221062489 @ahgittin I'd be interested in your thoughts about this idea... Still needs documentation, more examples and tests --- If your project is set up for it, you

[GitHub] brooklyn-client pull request: Support getting arguments from URL o...

2016-05-23 Thread robertgmoss
Github user robertgmoss commented on the pull request: https://github.com/apache/brooklyn-client/pull/19#issuecomment-221061284 I wonder if we would be better to start introducing tests that are [idomatic to golang](https://golang.org/pkg/testing/)? --- If your project is set up for

[GitHub] brooklyn-server pull request: BROOKLYN-278: fix Propagator validat...

2016-05-23 Thread aledsage
GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/154 BROOKLYN-278: fix Propagator validation/defaults You can merge this pull request into a Git repository by running: $ git pull https://github.com/aledsage/brooklyn-server

Re: YAML blueprint: install package utility

2016-05-23 Thread Andrew Kennedy
Duncan, I like the idea here, of having some extra object that does the package installation, but what about a customizer or initializer instead? You could have something like this: - type: org.apache.brooklyn.entity.basic.VanillaSoftwareProcess brooklyn.initializers: - type:

[GitHub] brooklyn-server pull request: Add support for JSR-223 scripting of...

2016-05-23 Thread johnmccabe
Github user johnmccabe commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/153#discussion_r64288074 --- Diff: core/pom.xml --- @@ -150,6 +150,11 @@ org.apache.commons commons-lang3 +

Re: Naming of Brooklyn Client on OSX Homebrew

2016-05-23 Thread John McCabe
@all, have gone ahead with `apache-brooklyn-cli`, I'll update when its merged and accessible. On Mon, 23 May 2016 at 12:16 Thomas Bouron wrote: > Hi John. > > As stated in Gitter, I +1 for `apache-brooklyn-cli` as the name of the brew > package. > Looking

Installing Apache Brooklyn CLI 0.9.0 on OSX with brew

2016-05-23 Thread John McCabe
All, The Apache Brooklyn CLI 0.9.0 (br) Homebrew [1] formula has just merged, so it's now available for install. # brew update # brew install apache-brooklyn-cli # br -version Heres what it looks like in under 30 seconds: - https://asciinema.org/a/0098jfnikgmsr2fwhargkmayz Enjoy !! /John [1]

[GitHub] brooklyn-server pull request: Add support for JSR-223 scripting of...

2016-05-23 Thread grkvlt
Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/153#discussion_r64304074 --- Diff: core/pom.xml --- @@ -150,6 +150,11 @@ org.apache.commons commons-lang3 + +

Re: Installing Apache Brooklyn CLI 0.9.0 on OSX with brew

2016-05-23 Thread Andrew Kennedy
Awesome, just tested this on my laptop now. Thanks for doing this, John! Andrew. On Mon, 23 May 2016 at 23:35 John McCabe wrote: > All, > The Apache Brooklyn CLI 0.9.0 (br) Homebrew [1] formula has just merged, so > it's now available for install. > > # brew update > #

[jira] [Commented] (BROOKLYN-276) TomcatServer entity does not install JRE if one is not present

2016-05-23 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15295974#comment-15295974 ] Svetoslav Neykov commented on BROOKLYN-276: --- Looks like an environment specific problem. Can

GSoC coding period

2016-05-23 Thread Jose Carrasco
Hi all, Today Google Summer of code period officially starts. we expects to accomplish the proposed goals. Currently, we are defining the first architecture version to add PaaS services to Brooklyn. Thanks to Andrea, Aled and the rest of the community for your help and support during the

Re: Naming of Brooklyn Client on OSX Homebrew

2016-05-23 Thread Richard Downer
John, I've just realised that I may be misinterpreting your message. Is this thread about the naming of the *package* - or the naming of the *command*? I was assuming that your were talking about the package. My comments stand for that. Additionally I'd like you to use "Apache Brooklyn" instead

Re: Naming of Brooklyn Client on OSX Homebrew

2016-05-23 Thread John McCabe
@richard kicked it off myself at the weekend as I was installing br and figured I'd take a look at whats involved in getting it added to brew. re: metadata/attribution etc, what do you propose? On Mon, 23 May 2016 at 10:08 Richard Downer wrote: > Hi all, > > Catching up on

[GitHub] brooklyn-docs pull request: Openstack network documentation

2016-05-23 Thread sjcorbett
GitHub user sjcorbett opened a pull request: https://github.com/apache/brooklyn-docs/pull/71 Openstack network documentation Repeats #67 against 0.9.0. You can merge this pull request into a Git repository by running: $ git pull https://github.com/sjcorbett/brooklyn-docs

Naming of Brooklyn Client on OSX Homebrew

2016-05-23 Thread John McCabe
Hi All, I've currently got a pull request in against Homebrew/homebrew-core#1326 to add the Brooklyn CLI to brew. There's a reluctance however to add a `br` alias due to it being 'too generic', one alternative is to just go with the repo

Re: Naming of Brooklyn Client on OSX Homebrew

2016-05-23 Thread Thomas Bouron
Hey John. So if I'm understanding correctly this PR, this is to install the CLI via brew right? So why not using a simple brooklyn-cli? Best On Mon, 23 May 2016, 08:53 John McCabe, wrote: > Hi All, > > I've currently got a pull request in against

[GitHub] brooklyn-server pull request: Resolve AddSensor and AddEffector co...

2016-05-23 Thread sjcorbett
GitHub user sjcorbett opened a pull request: https://github.com/apache/brooklyn-server/pull/152 Resolve AddSensor and AddEffector config Resolves useful parameters of implementations of the `AddEffector` and `AddSensor` entity initializers, meaning that their values can be sourced

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread neykov
Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/151#issuecomment-220959515 I like the idea of the PR, but feel that it should be more constrained in scope. Either having this behaviour only for `prosivioning.properties` or even more to

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread nakomis
Github user nakomis commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/151#discussion_r64197284 --- Diff: camp/camp-brooklyn/src/test/java/org/apache/brooklyn/camp/brooklyn/EmptySoftwareProcessYamlTest.java --- @@ -60,6 +62,31 @@ public void

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread bostko
Github user bostko commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/151#discussion_r64197998 --- Diff: core/src/main/java/org/apache/brooklyn/location/byon/FixedListMachineProvisioningLocation.java --- @@ -345,7 +346,7 @@ protected void

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/151#discussion_r64206361 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/collections/CollectionHelpers.java --- @@ -0,0 +1,52 @@ +/* + * Licensed to

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/151#discussion_r64206445 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/collections/CollectionHelpers.java --- @@ -0,0 +1,52 @@ +/* + * Licensed to

[GitHub] brooklyn-docs pull request: Improve multi-location docs

2016-05-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/70 --- 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 pull request: Merge Map values in config

2016-05-23 Thread bostko
GitHub user bostko opened a pull request: https://github.com/apache/brooklyn-server/pull/151 Merge Map values in config Merge location configuration and application's provisioning.properties You can merge this pull request into a Git repository by running: $ git pull

[jira] [Commented] (BROOKLYN-276) TomcatServer entity does not install JRE if one is not present

2016-05-23 Thread John McCabe (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15296181#comment-15296181 ] John McCabe commented on BROOKLYN-276: -- [~svet] was actually a docker container, I've been putting

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread bostko
Github user bostko commented on the pull request: https://github.com/apache/brooklyn-server/pull/151#issuecomment-220939386 Is the naming OK. Do you think I named properly the util class? --- 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: Merge Map values in config

2016-05-23 Thread nakomis
Github user nakomis commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/151#discussion_r64197677 --- Diff: core/src/main/java/org/apache/brooklyn/location/byon/FixedListMachineProvisioningLocation.java --- @@ -345,7 +346,7 @@ protected void

[jira] [Created] (BROOKLYN-277) Add support for 'path within value' in org.apache.brooklyn.core.config.external.vault

2016-05-23 Thread david bush (JIRA)
david bush created BROOKLYN-277: --- Summary: Add support for 'path within value' in org.apache.brooklyn.core.config.external.vault Key: BROOKLYN-277 URL: https://issues.apache.org/jira/browse/BROOKLYN-277

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread nakomis
Github user nakomis commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/151#discussion_r64203063 --- Diff: camp/camp-brooklyn/src/test/java/org/apache/brooklyn/camp/brooklyn/EmptySoftwareProcessYamlTest.java --- @@ -60,6 +62,31 @@ public void

Re: Naming of Brooklyn Client on OSX Homebrew

2016-05-23 Thread Thomas Bouron
Hi John. As stated in Gitter, I +1 for `apache-brooklyn-cli` as the name of the brew package. Looking forward to use it! Best. On Mon, 23 May 2016 at 10:55 John McCabe wrote: > Thanks Richard, will aim to proceed with ApacheBrooklynCli, so to flow on > OSX would be: > >

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/151#discussion_r64206860 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/collections/CollectionHelpers.java --- @@ -0,0 +1,52 @@ +/* + * Licensed to

Fixing location leaks

2016-05-23 Thread Svetoslav Neykov
The behaviour Brooklyn follows about locations is that the owner of the location (the one who created it) is responsible for unmanaging it. This falls short when the location is created as part of a CAMP blueprint or an EntitySpec because there's no clear owner in this case. I've created a PR

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread nakomis
Github user nakomis commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/151#discussion_r64203567 --- Diff: core/src/main/java/org/apache/brooklyn/location/byon/FixedListMachineProvisioningLocation.java --- @@ -345,7 +346,7 @@ protected void

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread bostko
Github user bostko commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/151#discussion_r64207286 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/collections/CollectionHelpers.java --- @@ -0,0 +1,52 @@ +/* + * Licensed to

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/151#discussion_r64207206 --- Diff: core/src/main/java/org/apache/brooklyn/util/core/config/ConfigBag.java --- @@ -126,7 +127,11 @@ public void markUsed(String key) {

[GitHub] brooklyn-server pull request: Merge Map values in config

2016-05-23 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/151#discussion_r64207370 --- Diff: core/src/main/java/org/apache/brooklyn/util/core/config/ConfigBag.java --- @@ -126,7 +127,11 @@ public void markUsed(String key) {