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 wrote:
Would it be worth including CLI Go vendoring - 
https://github.com/apache/brooklyn-client/pull/16? 
<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/TTTTuI


On 6 Apr 2016, at 15:47, Aled Sage <aled.s...@gmail.com> wrote:

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/pull/103
* BROOKLYN-248:
   https://github.com/apache/brooklyn-server/pull/104
* BROOKLYN-246:
   https://github.com/apache/brooklyn-server/pull/102
* https://github.com/apache/brooklyn-client/pull/14
* https://github.com/apache/brooklyn-client/pull/15

Anything else that needs done before the next 0.9.0 release candidate?

Aled


On 06/04/2016 09:45, Svetoslav Neykov wrote:
+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-client/pull/14 
<https://github.com/apache/brooklyn-client/pull/14>
[2] https://github.com/apache/brooklyn-client/pull/15 
<https://github.com/apache/brooklyn-client/pull/15>



On 5.04.2016 г., at 20:10, Aled Sage <aled.s...@gmail.com> wrote:

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 in the 0.9.0 release?

Aled

[1] https://github.com/apache/brooklyn-server/pull/102
[2] https://issues.apache.org/jira/browse/BROOKLYN-249
[3] https://github.com/apache/brooklyn-server/pull/103


On 05/04/2016 10:30, John McCabe wrote:
Note that it also breaks the add location wizard in jsgui.

On Tue, 5 Apr 2016 at 10:26 Richard Downer <rich...@apache.org> 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 we really don't want our
simplest-possible-get-started to require manual modifications before
it'll work. It also breaks the documented way of using BYON[1] so we
are at high risk of breaking user's working configurations.

Therefore due to this critical bug, I am cancelling the vote.

Richard.

[1]https://brooklyn.apache.org/v/latest/ops/locations/index.html#byon

On 4 April 2016 at 18:11, John McCabe <j...@johnmccabe.net> wrote:
@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 <j...@johnmccabe.net> wrote:

@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* 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-bin.tar.gz
<

https://www.apache.org/dyn/closer.lua?action=download&filename=brooklyn/apache-brooklyn-0.9.0/apache-brooklyn-0.9.0-bin.tar.gz
.
Once released should work.
    Unrelated, but do we require the absolute latest vagrant version on
purpose? Can we relax this version requirement?

Svet.


On 4.04.2016 г., at 13:14, Sam Corbett <
sam.corb...@cloudsoftcorp.com>
wrote:
+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 Downer wrote:
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, digests,
etc. can be found at:

https://dist.apache.org/repos/dist/dev/brooklyn/apache-brooklyn-0.9.0-rc2
The artifact SHA-256 checksums are as follows:
426ca93aad28ac8281b2015fc0ea419cd6db19c3ed87ffd11910a6baf2e9e3ab
*apache-brooklyn-0.9.0-rc2-1.noarch.rpm
b2a65642fb86198b9fd9992ef3464025d9b41c453c4dd5366699d569a65709c6
*apache-brooklyn-0.9.0-rc2-bin.tar.gz
999fd6f1d21417278ffe153fc1f0346d44235dfbbd39f200815f61cea82edbe5
*apache-brooklyn-0.9.0-rc2-bin.zip
00e0a76775775957e92498d0b5ea3e13f844155eda3251a8ead05cf4b4a4455b
*apache-brooklyn-0.9.0-rc2-src.tar.gz
c7a9a286993b1d520ae6a32e25c0dd3bef7227290d59347256bd96480b70a02b
*apache-brooklyn-0.9.0-rc2-src.zip
907267c5d2e7c1622e11f2bc57b6afe12ddf2136d82bad63661e43570aaa78fa
*apache-brooklyn-0.9.0-rc2-vagrant.tar.gz
0ae8384426d6b7197ca3d194c1e8032b85c5bb80671157c274487220d3ba65e9
*apache-brooklyn-0.9.0-rc2-vagrant.zip

The Nexus staging repositories for the Maven artifacts are located
at:
https://repository.apache.org/content/repositories/orgapachebrooklyn-1016
https://repository.apache.org/content/repositories/orgapachebrooklyn-1017
All release artifacts are signed with the following key:
https://people.apache.org/keys/committer/richard.asc

KEYS file available here:
https://dist.apache.org/repos/dist/release/brooklyn/KEYS

The artifacts were built from these Git commit IDs:
brooklyn: acc8ff1930d243d2a5fae1ad2f1a1ef17ca4a19c
brooklyn-client: 64573c8e1b8630f59b23356520282c91cc46a8a1
brooklyn-dist: 0b8b81df458fa4323939582d9f0dda10b2f6eaee
brooklyn-docs: 12430d193e1891b87a677d6b45a3b17861c83518
brooklyn-library: 2565e6eb2868468ec2528df74fe85efdb887b6d2
brooklyn-server: d031cea080613645219ba982c93264943798d4fc
brooklyn-ui: 307382128951bb237bc351ac22136745e5d50475
All of the above have been tagged as "apache-brooklyn-0.9.0-rc2".


Please download the artifacts, test, and vote on releasing this
package as Apache Brooklyn 0.9.0.

The vote will be open for at least 72 hours.
[ ] +1 Release this package as Apache Brooklyn 0.9.0 (please
describe
the tests you have performed)
[ ] +0 no opinion
[ ] -1 Do not release this package (please describe why not)

Thanks
Richard.


Reply via email to