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

2016-12-20 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/462#discussion_r93292616 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/BasicSpecParameter.java --- @@ -404,7 +431,126 @@ public static void

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

2016-12-20 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/462#discussion_r93286450 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/BasicSpecParameter.java --- @@ -404,7 +431,126 @@ public static void

[GitHub] brooklyn-server issue #492: Allow a user to add or override deserialize conf...

2016-12-20 Thread geomacy
Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/492 hi @svet, @tbouron, the `ConfigurationAdmin` is an OSGI compendium service, is that a dependency that we'd be happy to add to brooklyn-core? I think I would prefer not to, but if so , then

[GitHub] brooklyn-server issue #489: BROOKLYN-409: add test

2016-12-20 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/489 Thanks @neykov - changes incorporated, and squashed into single commit. Merging now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] brooklyn-server pull request #489: BROOKLYN-409: add test

2016-12-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/489 --- 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-410) Rebind fails in karaf, with app creating location from class in bundle

2016-12-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15763994#comment-15763994 ] ASF GitHub Bot commented on BROOKLYN-410: - Github user aledsage commented on the issue:

[GitHub] brooklyn-server issue #490: Fix BROOKLYN-410: rebind locations from bundles

2016-12-20 Thread aledsage
Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/490 I've rebased (having merged #489). Once jenkins has run the tests, I'll merge - unless there are any other comments from anyone. --- If your project is set up for it, you can reply to

[GitHub] brooklyn-server issue #484: Small JcloudsLocation refactor

2016-12-20 Thread geomacy
Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/484 Looks good @sjcorbett, a useful tidy-up to this big class; have had a look through PR but nothing to add to above comments. --- If your project is set up for it, you can reply to this

[jira] [Created] (BROOKLYN-417) In default install, web-console/REST from localhost needs username:password

2016-12-20 Thread Aled Sage (JIRA)
Aled Sage created BROOKLYN-417: -- Summary: In default install, web-console/REST from localhost needs username:password Key: BROOKLYN-417 URL: https://issues.apache.org/jira/browse/BROOKLYN-417 Project:

Re: [VOTE] Release Apache Brooklyn 0.10.0 [rc3]

2016-12-20 Thread Aled Sage
Alex, all, I've created https://issues.apache.org/jira/browse/BROOKLYN-417 to track this - and have included details from my initial investigation. I'm in two minds about whether it blocks the release. We expect people to quickly move to supplying credentials (rather than Brooklyn

[jira] [Commented] (BROOKLYN-410) Rebind fails in karaf, with app creating location from class in bundle

2016-12-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15764150#comment-15764150 ] ASF GitHub Bot commented on BROOKLYN-410: - Github user neykov commented on a diff in the pull

Re: [VOTE] Release Apache Brooklyn 0.10.0 [rc3]

2016-12-20 Thread Andrea Turli
+1 (binding) Tested by running Svet's verify script. Do a clean extract of source repo for next steps. --- Checks successfully completed: [✓] Download links work. [✓] Checksums and PGP signatures are valid. [✓] Expanded source archive matches contents of

Re: [VOTE] Release Apache Brooklyn 0.10.0 [rc3]

2016-12-20 Thread Alex Heneveld
Hi Devs, -0 ? -1 ? The "bin" build when run locally on a fresh system (no brooklyn.properties) requires that a username/password is supplied. It doesn't do any authentication of the local credentials but gives a 401 if not supplied. Specifically: $ curl -v http://localhost:8081/ 2>&1 | grep

[GitHub] brooklyn-server pull request #484: Small JcloudsLocation refactor

2016-12-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/484 --- 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

Re: [DISCUSS][VOTE] Release Apache Brooklyn 0.10.0 [rc3]

2016-12-20 Thread Richard Downer
Ok, I can reproduce this behaviour when trying to connect to a Brooklyn instance on localhost. *But...* I've started up a new instance on AWS and I'm connecting to it on port 8081. A username and password is printed to the console on startup, and I can use those credentials to access the web UI.

[GitHub] brooklyn-server issue #484: Small JcloudsLocation refactor

2016-12-20 Thread sjcorbett
Github user sjcorbett commented on the issue: https://github.com/apache/brooklyn-server/pull/484 Thanks @aledsage and @geomacy. I will merge this 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

Re: Uploading ZIPs for a better dev workflow

2016-12-20 Thread Aled Sage
Hi all, +1 (D) sounds good. What version are you imagining the bundle would be, if one runs `br catalog add ~/my/project/ --name com.example.myproject`? --- I like the idea of uploading a plain zip (rather than only supporting OSGi bundles) - that makes it simpler for non-java folk. The use

[GitHub] brooklyn-server issue #482: DSL deferred execution

2016-12-20 Thread geomacy
Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/482 Actually I think my comments above are not worth holding this up for, and I answered my question above myself with a bit of further inspection and running tests, so happy to merge. LGTM.

[jira] [Commented] (BROOKLYN-418) MariaDB does not install correctly

2016-12-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-418?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15764666#comment-15764666 ] ASF GitHub Bot commented on BROOKLYN-418: - GitHub user bostko opened a pull request:

Re: Uploading ZIPs for a better dev workflow

2016-12-20 Thread Svetoslav Neykov
> Svet, if instead we tried to infer it from the catalog.bom, would we require > some additional metadata within the .bom file? Or would we use the catalog > item's id + version? I'm not convinced by the latter - it would mean some > .bom files would work and others wouldn't (e.g. if the .bom

[jira] [Created] (BROOKLYN-418) MariaDB does not install correctly

2016-12-20 Thread Valentin Aitken (JIRA)
Valentin Aitken created BROOKLYN-418: Summary: MariaDB does not install correctly Key: BROOKLYN-418 URL: https://issues.apache.org/jira/browse/BROOKLYN-418 Project: Brooklyn Issue Type:

[GitHub] brooklyn-server pull request #497: Add findOpenSocketsOnNode to ReachableSoc...

2016-12-20 Thread sjcorbett
GitHub user sjcorbett opened a pull request: https://github.com/apache/brooklyn-server/pull/497 Add findOpenSocketsOnNode to ReachableSocketFinder This changes the semantics of ReachableSocketFinder. Previously it would return as soon as any host+port was reached. Now it determines

[GitHub] brooklyn-server pull request #498: Use SIGTERM in stop script inside Abstrac...

2016-12-20 Thread bostko
GitHub user bostko opened a pull request: https://github.com/apache/brooklyn-server/pull/498 Use SIGTERM in stop script inside AbstractSoftwareProcessSshDriver - fixes BROOKLYN-418 You can merge this pull request into a Git repository by running: $ git pull