Re: move to new repos THIS WEEKEND ?

2016-01-28 Thread Richard Downer
+1. In about a week I'll be drafting the last of our three initial monthly reports to the board; I'd love to say that we have shed our "incubating" repository tag! Richard. On 28 January 2016 at 10:09, Alex Heneveld wrote: > > Brooklyners- > > TL;DR: *switch to new repos at the weekend, incubat

Re: move to new repos THIS WEEKEND ?

2016-01-28 Thread Alex Heneveld
One more check/update/cool thing: even with the restructured repos we can fairly easily build older versions just by flattening them, at least for `0.8.0-incubating`. https://github.com/ahgittin/brooklyn-docs/blob/submodules/guide/dev/code/index.md#history-tags-and-workflow Best Alex On 28

[GitHub] incubator-brooklyn pull request: add getting started vagrant env t...

2016-01-28 Thread johnmccabe
Github user johnmccabe commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/1170#issuecomment-176319846 Note that I'm trying to avoid having the user install any plugins - [vagrant-triggers](https://github.com/emyl/vagrant-triggers) for example. --- If yo

Re: move to new repos THIS WEEKEND ?

2016-01-28 Thread Alex Heneveld
I've fixed the problems Aled observed with the `all.sh` script and the problem with previous branches. I've also added PR's which: * set up submodules: https://github.com/ahgittin/brooklyn/pull/5 * add docs for using multi-repos and submodules Key things: * the submodules are there, and work well

[GitHub] incubator-brooklyn pull request: add getting started vagrant env t...

2016-01-28 Thread johnmccabe
Github user johnmccabe commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/1170#issuecomment-176288949 @ahgittin its a bit messy to pull the -dist artifact in from a directory outside of the Vagrantfile directory, hence the copy step. I changed INSTALL_FR

Re: Brooklyn Daemon Solution

2016-01-28 Thread Andrew Kennedy
Or what about running a Brooklyn Docker image as a systemd service! - http://container-solutions.com/running-docker-containers-with-systemd/ - https://github.com/ibuildthecloud/systemd-docker Andrew. On Thu, 28 Jan 2016 at 16:34 Alex Heneveld wrote: > > Hi Aleksandr- > > What's the advantage o

Re: Brooklyn Daemon Solution

2016-01-28 Thread Alex Heneveld
Hi Aleksandr- What's the advantage of a native daemon over just wrapping it as a linux service script ? Best Alex On 28/01/2016 11:32, Aleksandr Vasilev wrote: Hello everyone! I spent last few days looking at the solution to run Brooklyn process as a daemon and found two options: 1. Run d

[GitHub] incubator-brooklyn pull request: Generate config/effector descript...

2016-01-28 Thread iyovcheva
Github user iyovcheva commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/1176#issuecomment-176244738 Jenkins fails due to `DynamicSequentialTaskTest.testCancelled` which I think is not related to the issue. The test is successful locally. --- If your p

Re: move to new repos THIS WEEKEND ?

2016-01-28 Thread Alex Heneveld
Thanks Aled. > branches="0.4.0 0.5.0 0.6.0 0.7.0-incubating 0.8.0-incubating" > I suggest adding 0.8.x. 0.8.x has the old structure so it will be hard to work with here, and unless we release an 0.8.1 it will be noise. I'm expecting instead we can get an 0.9.0 out soon. The other repos also

Brooklyn Daemon Solution

2016-01-28 Thread Aleksandr Vasilev
Hello everyone! I spent last few days looking at the solution to run Brooklyn process as a daemon and found two options: 1. Run daemon via Apache Commons Daemon (jsvc) 2. Write a custom daemon in C Both solutions has its own pros and cons, so let's look at what I think they are: JSVC: Pros: - Re

Re: move to new repos THIS WEEKEND ?

2016-01-28 Thread Aled Sage
Thanks Alex, sounds great. I'm running through the brooklyn-repo-split release script locally now, to reproduce. A few initial comments... --- https://github.com/ahgittin/brooklyn-repo-split/blob/master/README.md needs updated. It says that "Steps 1 and 2 are done and checked in as reorg*

Re: move to new repos THIS WEEKEND ?

2016-01-28 Thread John McCabe
Looks good, checked out, built and deployed a test app without any observed issues. Will there a cut off on getting PRs merged today or tomorrow? On Thu, 28 Jan 2016 at 10:09 Alex Heneveld wrote: > > Brooklyners- > > TL;DR: *switch to new repos at the weekend, incubator commits cut-off > propos

[GitHub] incubator-brooklyn pull request: Improvements to brooklyn-server/r...

2016-01-28 Thread andreaturli
Github user andreaturli commented on a diff in the pull request: https://github.com/apache/incubator-brooklyn/pull/1177#discussion_r51107643 --- Diff: brooklyn-server/rest/rest-server/src/main/java/org/apache/brooklyn/rest/util/json/BrooklynJacksonJsonProvider.java --- @@ -137,25

[GitHub] incubator-brooklyn pull request: Improvements to brooklyn-server/r...

2016-01-28 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/incubator-brooklyn/pull/1177#discussion_r51107270 --- Diff: brooklyn-server/karaf/features/src/main/history/dependencies.xml --- @@ -0,0 +1,103 @@ + --- End diff -- Or even better

[GitHub] incubator-brooklyn pull request: Improvements to brooklyn-server/r...

2016-01-28 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/incubator-brooklyn/pull/1177#discussion_r51106927 --- Diff: brooklyn-server/karaf/features/src/main/history/dependencies.xml --- @@ -0,0 +1,103 @@ + --- End diff -- It's been remo

[GitHub] incubator-brooklyn pull request: Generate config/effector descript...

2016-01-28 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/incubator-brooklyn/pull/1176#discussion_r51106768 --- Diff: brooklyn-server/server-cli/src/main/java/org/apache/brooklyn/cli/ItemLister.java --- @@ -198,17 +277,36 @@ private void mkdir(String ro

[GitHub] incubator-brooklyn pull request: Improvements to brooklyn-server/r...

2016-01-28 Thread ahgittin
Github user ahgittin commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/1177#issuecomment-176119371 @andreaturli - this looks really good. @neykov is working with @CMoH on porting the REST API to CXF. that should help w any karaf issues but migh

[GitHub] incubator-brooklyn pull request: Improvements to brooklyn-server/r...

2016-01-28 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/incubator-brooklyn/pull/1177#discussion_r51106038 --- Diff: brooklyn-server/rest/rest-server/src/main/java/org/apache/brooklyn/rest/util/json/BrooklynJacksonJsonProvider.java --- @@ -137,25 +13

[GitHub] incubator-brooklyn pull request: Improvements to brooklyn-server/r...

2016-01-28 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/incubator-brooklyn/pull/1177#discussion_r51105914 --- Diff: brooklyn-server/rest/rest-server/src/main/java/org/apache/brooklyn/rest/util/json/BrooklynJacksonJsonProvider.java --- @@ -137,25 +13

[GitHub] incubator-brooklyn pull request: Improvements to brooklyn-server/r...

2016-01-28 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/incubator-brooklyn/pull/1177#discussion_r51105587 --- Diff: brooklyn-server/karaf/features/src/main/history/dependencies.xml --- @@ -0,0 +1,103 @@ + --- End diff -- is this file

[GitHub] incubator-brooklyn pull request: Improvements to brooklyn-server/r...

2016-01-28 Thread andreaturli
GitHub user andreaturli opened a pull request: https://github.com/apache/incubator-brooklyn/pull/1177 Improvements to brooklyn-server/rest/rest-api - remove duplication of jackson and jackson2 on brooklyn project - refactor pojos to have a consistent `toString`, `hashCode` and `e

move to new repos THIS WEEKEND ?

2016-01-28 Thread Alex Heneveld
Brooklyners- TL;DR: *switch to new repos at the weekend, incubator commits cut-off proposed for Sat 10am UK* The new project structure seems to be working well and I think it's time to move to the new repos apache/brooklyn and apache/brooklyn-*, and then retire the incubator project. I've

[GitHub] incubator-brooklyn pull request: Generate config/effector descript...

2016-01-28 Thread iyovcheva
Github user iyovcheva commented on a diff in the pull request: https://github.com/apache/incubator-brooklyn/pull/1176#discussion_r51100940 --- Diff: brooklyn-server/server-cli/src/main/java/org/apache/brooklyn/cli/ItemLister.java --- @@ -198,17 +277,36 @@ private void mkdir(String