Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Richard Downer
Thanks Aled. I'm kicking off rc3 just now. Richard. On 6 April 2016 at 20:43, Aled Sage wrote: > I've reviewed and tested John's brooklyn-dist PR #31: looks good, so merged > into master and 0.9.0. > > --- > As discussed offline, in the vote email we'll want to include checksums of > the indivi

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Aled Sage
I've reviewed and tested John's brooklyn-dist PR #31: looks good, so merged into master and 0.9.0. --- As discussed offline, in the vote email we'll want to include checksums of the individual brooklyn-client artifacts. This is because we want to include them as individually downloadable artif

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Aled Sage
Thanks Sam, That is now cherry-picked into 0.9.0. Agree it is low-risk and worth including. --- Can someone please review and test John's https://github.com/apache/brooklyn-dist/pull/31? It looks low risk and simple. Aled On 06/04/2016 17:29, Sam Corbett wrote: If I've not missed the cu

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread John McCabe
Likewise can https://github.com/apache/brooklyn-dist/pull/31 be considered for inclusion. /John On Wed, 6 Apr 2016 at 17:29 Sam Corbett wrote: > If I've not missed the cutoff I'd like > https://github.com/apache/brooklyn-server/pull/99 to be included. > > On 6 April 2016 at 17:19, Richard Downer

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Sam Corbett
If I've not missed the cutoff I'd like https://github.com/apache/brooklyn-server/pull/99 to be included. On 6 April 2016 at 17:19, Richard Downer wrote: > On 6 April 2016 at 17:17, Aled Sage wrote: > > I believe that Richard is discussing which Geoff about which > brooklyn-client > > platform a

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Richard Downer
On 6 April 2016 at 17:17, Aled Sage wrote: > I believe that Richard is discussing which Geoff about which brooklyn-client > platform artifacts are required. Once that is agreed and sorted, then I > think we are good to go. I've checked Geoff's PR and it's nice and simple: Mac 64bit, Windows and L

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Aled Sage
Thanks Mark for reviewing brooklyn-ui PR #24. That is now merged into master and cherry-picked into 0.9.0. (I completely agree with Mark's comments in #24, which can be addressed in a future PR just in master). --- I believe that Richard is discussing which Geoff about which brooklyn-client pl

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Aled Sage
Thanks Thomas. I've cherry-picked PR #23. --- The change in #24 looks sensible (without it, no-one would know that drag-and-drop of text-files is possible in the Blueprint Composer). Can someone please review and comment on the PR, and let us know if you agree it's low-risk for including in

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Geoff Macartney
That's fine by me. cheers Geoff Gnu PGP key - http://is.gd/uI > On 6 Apr 2016, at 16:58, Aled Sage wrote: > > Hi Geoff, > > I think we should *not* include it in the 0.9.0 release. It improves the > build process, but does not change the final artifacts. I think fo

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Aled Sage
Hi Geoff, I think we should *not* include it in the 0.9.0 release. It improves the build process, but does not change the final artifacts. I think for rc3 we should stick to just cherry-picking bug fixes. If anyone disagrees then please shout out. Aled On 06/04/2016 15:50, Geoff Macartney

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Thomas Bouron
I also have two candidates to include: https://github.com/apache/brooklyn-ui/pull/23 and https://github.com/apache/brooklyn-ui/pull/24 Thoughts? On Wed, 6 Apr 2016 at 15:51 Geoff Macartney < geoff.macart...@cloudsoftcorp.com> wrote: > Would it be worth including CLI Go vendoring - > https://gith

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Geoff Macartney
Would it be worth including CLI Go vendoring - https://github.com/apache/brooklyn-client/pull/16? Simplifies the build for the CLI (no need for extra tool apart from Go itself). Gnu PGP key - http://is.gd/uI > On

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Aled Sage
Hi all, Svet: agree with brooklyn-client/pull/14 and brooklyn-client/pull/15. I've cherry-picked those. We have reviewed and incorporated the following fixes into master and the 0.9.0 branch, ready for the next release candidate: * BROOKLYN-249: https://github.com/apache/brooklyn-server

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Svetoslav Neykov
+1 to include 102 & 103. Also suggest including [1] - cuts down on the number of cli binaries substantially and fixes the name on Windows. Shaves off close to 20mb from the release. [2] is minor but good to have (documents the go 1.6 requirement). Svet. [1] https://github.com/apache/brooklyn-

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-05 Thread Aled Sage
Thanks all. John has fixed this [1]. I'll review, merge, and cherry-pick to the 0.9.0 branch. Given we are producing another release candidate anyway, can we include the fix for BROOKLYN-249 (see [2,3]). Can someone please review that and give their opinion for whether it should be included

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-05 Thread John McCabe
Note that it also breaks the add location wizard in jsgui. On Tue, 5 Apr 2016 at 10:26 Richard Downer wrote: > After discussing with John, I agree that BROOKLYN-246 is a blocker. > The catalog file that the Brooklyn Vagrant distributable uses is using > the broken format, so it won't work - and

[CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-05 Thread Richard Downer
After discussing with John, I agree that BROOKLYN-246 is a blocker. The catalog file that the Brooklyn Vagrant distributable uses is using the broken format, so it won't work - and we really don't want our simplest-possible-get-started to require manual modifications before it'll work. It also brea

Re: [VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-04 Thread John McCabe
@all, I hit an issue with location handling while following up on @svets comments - https://issues.apache.org/jira/browse/BROOKLYN-246 Looks like a blocker. On Mon, 4 Apr 2016 at 14:35 John McCabe wrote: > @svet, latest isn't required, 1.8+ should suffice. > > On Mon, 4 Apr 2016 at 12:53 Svetos

Re: [VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-04 Thread John McCabe
@svet, latest isn't required, 1.8+ should suffice. On Mon, 4 Apr 2016 at 12:53 Svetoslav Neykov < svetoslav.ney...@cloudsoftcorp.com> wrote: > +1 binding > > * Tested the .zip & .tar.gz dists > * Tested CLI > * Tried a simple blueprint > * Tested the vagrant artifact - "vagrant up" *fails* becaus

Re: [VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-04 Thread Svetoslav Neykov
+1 binding * Tested the .zip & .tar.gz dists * Tested CLI * Tried a simple blueprint * Tested the vagrant artifact - "vagrant up" *fails* because it's looking at the release url https://www.apache.org/dyn/closer.lua?action=download&filename=brooklyn/apache-brooklyn-0.9.0/apache-brooklyn-0.9.0-bi

Re: [VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-04 Thread Sam Corbett
+1 binding I have: * Verified release signatures * Built from the source archive * Generated a project from the archetype * Verified all subsequent archetype instructions were correct * Verified a simple application deployed to AWS. * Given the CLI a work out. On 01/04/2016 16:58, Richard Downe

Re: [VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-02 Thread Andrew Kennedy
OK, I have tested Clocker by building against the source download (built locally, and used in offline mode when building Clocker) and I also validated the archive integrity. Clocker ran successfully, and deployed a Docker environment and test application as expected. Vote +1 (Binding) Andrew. On

Re: [VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-02 Thread Richard Downer
Hi Andrew, You're right - failure to follow my own instructions. I've now pushed the tags. Thanks Richard. On 2 April 2016 at 01:00, Andrew Kennedy wrote: > On Fri, 1 Apr 2016 at 16:58 Richard Downer wrote: > >> brooklyn: acc8ff1930d243d2a5fae1ad2f1a1ef17ca4a19c >> brooklyn-client: 64573c8e1b8

Re: [VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-01 Thread Andrew Kennedy
On Fri, 1 Apr 2016 at 16:58 Richard Downer wrote: > brooklyn: acc8ff1930d243d2a5fae1ad2f1a1ef17ca4a19c > brooklyn-client: 64573c8e1b8630f59b23356520282c91cc46a8a1 > brooklyn-dist: 0b8b81df458fa4323939582d9f0dda10b2f6eaee > brooklyn-docs: 12430d193e1891b87a677d6b45a3b17861c83518 > brooklyn-library

Re: [VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-01 Thread Aled Sage
+1 (binding) I have: * Verified the sha1, sha256, md5 and asc for each of the artifacts. * Verified that each zip and .tar.gz could be unpacked * Installed the RPM * Installed + launched from the .tar.gz * Deployed a tomcat app to OpenStack On 01/04/2016 16:58, Richard Downer wrote: Thi

[VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-01 Thread Richard Downer
This is to call for a vote for the release of Apache Brooklyn 0.9.0 [rc2]. This release comprises of a source code distribution, and a corresponding binary distribution, RPM packages, Vagrant environment package, and Maven artifacts. The source and binary distributions, including signatures, dige