[GitHub] brooklyn-dist pull request #47: Update tests following changes in files.prei...

2016-09-13 Thread geomacy
GitHub user geomacy opened a pull request: https://github.com/apache/brooklyn-dist/pull/47 Update tests following changes in files.preinstall inheritance Need to update the tests following changes in inheritance of files.preinstall, see

Re: [PROPOSAL] Remove unauthenticated localhost login

2016-09-13 Thread Martin Harris
@Geoff, you echo my sentiments exactly. The super-easy first touch is a big win and enabling security during 'productionization' is a common pattern Cheers M On 13 September 2016 at 16:25, Geoff Macartney < geoff.macart...@cloudsoftcorp.com> wrote: > hi Aled, > > Have been mulling this over

[GitHub] brooklyn-ui pull request #33: Update pom SCM & Jenkins URLs

2016-09-13 Thread bostko
Github user bostko commented on a diff in the pull request: https://github.com/apache/brooklyn-ui/pull/33#discussion_r78585881 --- Diff: pom.xml --- @@ -39,6 +39,33 @@ JavaScript+HTML GUI for interacting with Brooklyn, using the REST API +

[GitHub] brooklyn-server issue #328: Update pom SCM & Jenkins URLs

2016-09-13 Thread bostko
Github user bostko commented on the issue: https://github.com/apache/brooklyn-server/pull/328 LGTM! --- 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

[GitHub] brooklyn-library issue #63: Update pom SCM & Jenkins URLs

2016-09-13 Thread bostko
Github user bostko commented on the issue: https://github.com/apache/brooklyn-library/pull/63 LGTM! --- 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

[GitHub] brooklyn-dist issue #46: Update pom SCM & Jenkins URLs

2016-09-13 Thread bostko
Github user bostko commented on the issue: https://github.com/apache/brooklyn-dist/pull/46 LGTM! --- 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 #329: fix long-running unit test

2016-09-13 Thread ahgittin
GitHub user ahgittin opened a pull request: https://github.com/apache/brooklyn-server/pull/329 fix long-running unit test /cc @aledsage is this the best fix? (it stalls for 2m most the time when doing `mvn clean install` without this. haven't thought too much about it though!)

Re: [PROPOSAL] Remove unauthenticated localhost login

2016-09-13 Thread Geoff Macartney
hi Aled, Have been mulling this over since this debate started, and I actually like your ‘Alternative Proposal 2’ - out-of-the-box there is no authentication. That would be a super-easy experience for first-time users. For admins installing a production machine, they’ll always expect to have

Re: [PROPOSAL] Remove unauthenticated localhost login

2016-09-13 Thread Svetoslav Neykov
+1 for "New Proposal" Svet. > On 13.09.2016 г., at 17:56, John McCabe wrote: > > +1 on the prompt for user creation if none-exists > > On Tue, 13 Sep 2016, 15:32 Aled Sage, wrote: > >> Hi all, >> >> I still think we really need to change the

[GitHub] brooklyn-ui pull request #33: Update pom SCM & Jenkins URLs

2016-09-13 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-ui/pull/33 Update pom SCM & Jenkins URLs You can merge this pull request into a Git repository by running: $ git pull https://github.com/neykov/brooklyn-ui fix/pom-info Alternatively you can review

Re: [PROPOSAL] Remove unauthenticated localhost login

2016-09-13 Thread John McCabe
+1 on the prompt for user creation if none-exists On Tue, 13 Sep 2016, 15:32 Aled Sage, wrote: > Hi all, > > I still think we really need to change the status quo. For me, there are > two very compelling arguments: > > 1. In karaf, it will always prompt for a username and

[GitHub] brooklyn-client pull request #27: Re-use pom config from brooklyn-server

2016-09-13 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-client/pull/27 Re-use pom config from brooklyn-server Analogously to brooklyn-library reuse pom configuration from brooklyn-server. While not ideal this allows us to have version and plugin configuration

[GitHub] brooklyn-dist pull request #46: Update pom SCM & Jenkins URLs

2016-09-13 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-dist/pull/46 Update pom SCM & Jenkins URLs You can merge this pull request into a Git repository by running: $ git pull https://github.com/neykov/brooklyn-dist fix/pom-info Alternatively you can

[GitHub] brooklyn-client pull request #26: Update pom SCM & Jenkins URLs

2016-09-13 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-client/pull/26 Update pom SCM & Jenkins URLs You can merge this pull request into a Git repository by running: $ git pull https://github.com/neykov/brooklyn-client fix/pom-info Alternatively you can

[GitHub] brooklyn-library pull request #63: Update pom SCM & Jenkins URLs

2016-09-13 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-library/pull/63 Update pom SCM & Jenkins URLs You can merge this pull request into a Git repository by running: $ git pull https://github.com/neykov/brooklyn-library fix/pom-info Alternatively you

[GitHub] brooklyn-server pull request #328: Update pom SCM & Jenkins URLs

2016-09-13 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-server/pull/328 Update pom SCM & Jenkins URLs You can merge this pull request into a Git repository by running: $ git pull https://github.com/neykov/brooklyn-server fix/pom-info Alternatively you

Re: [PROPOSAL] Remove unauthenticated localhost login

2016-09-13 Thread Aled Sage
Hi all, I still think we really need to change the status quo. For me, there are two very compelling arguments: 1. In karaf, it will always prompt for a username and password. If running in localhost, you can enter any text and it will accept it. But you do have to "log in". 2. If

Re: Karaf and the BrooklynNode entity

2016-09-13 Thread Aled Sage
Hi Sam, Long term, we want to switch to just Karaf (installed via RPM/DEB). However, before we switch to Karaf being the "official Brooklyn way", we'll need to spend some more time working on Karaf and on RPM/DEB. (The previous RPM/DEB work was against the tgz dist, and will need revisiting).

[GitHub] brooklyn-server issue #326: OSGi Features cleanup

2016-09-13 Thread geomacy
Github user geomacy commented on the issue: https://github.com/apache/brooklyn-server/pull/326 👍 LGTM, changes make sense --- 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