passing brooklyn config variable value in files.install or template.install map Value

2015-01-28 Thread Azhar I Hashmi
Hello, I have been trying to pass the config variable (libPathConfig) in the templates.install or files.install value, but i am keep getting error. I also tried with `$brooklyn:formatString("%s/%s", config("libPathConfig"), "myfileName")` or `$brooklyn:formatString("%s/%s", $brooklyn:config ("

Re: new jclouds/login features

2015-01-28 Thread Richard Downer
Hi Alex, A bit late (since it has already been merged!) but I can confirm that your changes have *not* broken my use case :-) Richard. On 23 January 2015 at 13:50, Alex Heneveld wrote: > > Richard- > > Sure thing. The aim is to make those awkward login configs easier of > course, not harder, s

[jira] [Updated] (BROOKLYN-129) brooklyn.util.net.Urls.mergePaths(String... items) mishandles nulls in input path parts

2015-01-28 Thread michael dye (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-129?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] michael dye updated BROOKLYN-129: - Summary: brooklyn.util.net.Urls.mergePaths(String... items) mishandles nulls in input path parts

[jira] [Commented] (BROOKLYN-129) brooklyn.util.net.Urls.mergePaths(String... items) doesn't filter null values

2015-01-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-129?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14295385#comment-14295385 ] ASF GitHub Bot commented on BROOKLYN-129: - Github user michaeldye commented on t

[GitHub] incubator-brooklyn pull request: fixed BROOKLYN-129: brooklyn.util...

2015-01-28 Thread michaeldye
Github user michaeldye commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/471#issuecomment-71870374 Sounds good, and thanks for the note on the NPE convention. I've updated the PR. --- If your project is set up for it, you can reply to this email and hav

Build failed in Jenkins: incubator-brooklyn-master-windows #162

2015-01-28 Thread Apache Jenkins Server
See -- Started by upstream project "incubator-brooklyn-master-build" build number 361 originally caused by: Started by an SCM change Building remotely on windows1 (Windows) in workspace

[GitHub] incubator-brooklyn pull request: obfuscate passwords and credentia...

2015-01-28 Thread ahgittin
Github user ahgittin commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/484#issuecomment-71861055 @alasdairhodge @bernd-clemenz one of you care to review? --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] incubator-brooklyn pull request: obfuscate passwords and credentia...

2015-01-28 Thread ahgittin
GitHub user ahgittin opened a pull request: https://github.com/apache/incubator-brooklyn/pull/484 obfuscate passwords and credentials in the gui (literal "shadow passwords") applies text shadowing to blur keys that say obvious things like "password" and "credential", to config t

disinheritance of config keys

2015-01-28 Thread Alex Heneveld
By popular demand, we now have a way to make config *not* be inherited! https://github.com/apache/incubator-brooklyn/pull/483 This fixes an "on-fire" which can occur with one of the website tutorials; now we write: public static final ConfigKey UP_QUORUM_CHECK = ConfigKeys.build

[GitHub] incubator-brooklyn pull request: Add deploy repo

2015-01-28 Thread richardcloudsoft
Github user richardcloudsoft commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/464#issuecomment-71859723 I'd be in favour of removing the non-Apache repos because that's the only place the Apache Brooklyn will deploy to. If anyone (*cough*Cloudsoft*cough*

[GitHub] incubator-brooklyn pull request: add support for defining explicit...

2015-01-28 Thread ahgittin
GitHub user ahgittin opened a pull request: https://github.com/apache/incubator-brooklyn/pull/483 add support for defining explicit inheritance for config keys obeyed for entities getConfig AND getAllConfig; for locations only obeyed for getConfig. notes in code explain why.

[GitHub] incubator-brooklyn pull request: JcloudsLocation set arbitrary tem...

2015-01-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-brooklyn/pull/460 --- 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 fea

[GitHub] incubator-brooklyn pull request: Improve recursion checks for cata...

2015-01-28 Thread neykov
Github user neykov commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/383#issuecomment-71853205 @sjcorbett Rebased on latest master, conflicts are resolved. --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] incubator-brooklyn pull request: JcloudsLocation set arbitrary tem...

2015-01-28 Thread ahgittin
Github user ahgittin commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/460#issuecomment-71847030 great addition --- 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

[GitHub] incubator-brooklyn pull request: JcloudsLocation set arbitrary tem...

2015-01-28 Thread sjcorbett
Github user sjcorbett commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/460#issuecomment-71845693 Fire away. --- 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 thi

[GitHub] incubator-brooklyn pull request: Skip dry run on node upgrade.

2015-01-28 Thread neykov
Github user neykov commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/370#issuecomment-71845753 Test failure should be fixed in https://github.com/apache/incubator-brooklyn/pull/482 (not included in this PR). Rebasing again to kick another build. --- If

[GitHub] incubator-brooklyn pull request: Wait for STOP effector to complet...

2015-01-28 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/incubator-brooklyn/pull/482 Wait for STOP effector to complete before unamanging BrooklynNode Otherwise if there still are running tasks for the entity they will be canceled. You can merge this pull request into a

[GitHub] incubator-brooklyn pull request: JcloudsLocation set arbitrary tem...

2015-01-28 Thread andreaturli
Github user andreaturli commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/460#issuecomment-71835966 lgtm, if @sjcorbett is happy as well, please merge it --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] incubator-brooklyn pull request: fix JcloudsMachineNamer

2015-01-28 Thread andreaturli
GitHub user andreaturli opened a pull request: https://github.com/apache/incubator-brooklyn/pull/481 fix JcloudsMachineNamer - remove maxLenght for SoftLayer You can merge this pull request into a Git repository by running: $ git pull https://github.com/andreaturli/incubator-br

[GitHub] incubator-brooklyn pull request: JcloudsLocation set arbitrary tem...

2015-01-28 Thread richardcloudsoft
Github user richardcloudsoft commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/460#issuecomment-71833560 Pushed a fix for the failing test. If the next test run passes, am I good to merge this @sjcorbett @andreaturli ? --- If your project is set up for

[GitHub] incubator-brooklyn pull request: JcloudsLocation set arbitrary tem...

2015-01-28 Thread rdowner
Github user rdowner commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/460#issuecomment-71821985 This time the test failure is genuine. The test is supposed to be a live test so should not be run here. I'll investigate. @andreaturli, the reflectio

[GitHub] incubator-brooklyn pull request: JcloudsLocation set arbitrary tem...

2015-01-28 Thread andreaturli
Github user andreaturli commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/460#issuecomment-71819914 @rdowner thanks! 1. makes sense to me 2. I think my question is more "how brooklyn infers that `instanceProfileName` is part of `AWSEC2TemplateOpti

nohup vs disown

2015-01-28 Thread Alasdair Hodge
Brooklyn-ers, For those still confused about 'nohup' vs 'disown' and all things SIGHUP, here's an oldie-but-goodie from StackExchange on the very subject: http://unix.stackexchange.com/questions/3886/difference-between-nohup-disown-and Hopefully makes it clearer why Brooklyn launches long-l

[GitHub] incubator-brooklyn pull request: Release ports acquired when check...

2015-01-28 Thread mattunderscorechampion
Github user mattunderscorechampion commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/480#issuecomment-71818654 Closing and reopening to see if build failure happens again --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] incubator-brooklyn pull request: Release ports acquired when check...

2015-01-28 Thread mattunderscorechampion
Github user mattunderscorechampion closed the pull request at: https://github.com/apache/incubator-brooklyn/pull/480 --- 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 s

[jira] [Commented] (BROOKLYN-130) Available port detection does not release ports

2015-01-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14295025#comment-14295025 ] ASF GitHub Bot commented on BROOKLYN-130: - GitHub user mattunderscorechampion re

[jira] [Commented] (BROOKLYN-130) Available port detection does not release ports

2015-01-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14295024#comment-14295024 ] ASF GitHub Bot commented on BROOKLYN-130: - Github user mattunderscorechampion cl

[GitHub] incubator-brooklyn pull request: Release ports acquired when check...

2015-01-28 Thread mattunderscorechampion
GitHub user mattunderscorechampion reopened a pull request: https://github.com/apache/incubator-brooklyn/pull/480 Release ports acquired when checking available Fixes issue [BROOKLYN-130](https://issues.apache.org/jira/browse/BROOKLYN-130). Ensures that the setReuseAddress is calle

[GitHub] incubator-brooklyn pull request: JcloudsLocation set arbitrary tem...

2015-01-28 Thread rdowner
Github user rdowner commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/460#issuecomment-71817326 Hi @andreaturli, 1 - I don't think there's a need to do that since it's still tidier in blueprints to refer to the Brooklyn-supported properties as be

[GitHub] incubator-brooklyn pull request: JcloudsLocation set arbitrary tem...

2015-01-28 Thread rdowner
Github user rdowner closed the pull request at: https://github.com/apache/incubator-brooklyn/pull/460 --- 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 fe

[GitHub] incubator-brooklyn pull request: JcloudsLocation set arbitrary tem...

2015-01-28 Thread rdowner
GitHub user rdowner reopened a pull request: https://github.com/apache/incubator-brooklyn/pull/460 JcloudsLocation set arbitrary template options Adds new config key for JcloudsLocation, templateOptions, that takes a Map for setting arbitrary template options. For each entry, searc

[GitHub] incubator-brooklyn pull request: Release ports acquired when check...

2015-01-28 Thread mattunderscorechampion
Github user mattunderscorechampion commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/480#issuecomment-71802104 Having reviewed the build failure. A NullPointerException is thrown when parsing a string to URI. [URI.create(..)](http://docs