Mailing-List: contact dev-help@brooklyn.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@brooklyn.incubator.apache.org
MIME-Version: 1.0
References: 
 <CABQFKi08vdgmEdKmVcfOXJW7H4C7aGfVMHYt=na0DoZbNm80Wg@mail.gmail.com>
 <0F9F6077-8597-400B-8710-A2BE2D1F426F@cloudsoftcorp.com>
 <CADBrumZ-Qw9_XZ5L=ZH_2sbzy5e40s9PGjiPG-eLm1wK3PxSZw@mail.gmail.com>
In-Reply-To: 
 <CADBrumZ-Qw9_XZ5L=ZH_2sbzy5e40s9PGjiPG-eLm1wK3PxSZw@mail.gmail.com>
From: Richard Downer <richard@apache.org>
Date: Wed, 24 Jun 2015 10:25:36 +0000
Message-ID: 
 <CABQFKi3zDfF3ZsXA8RtN6=q1Ug+kdp_F5YhkwSiesbffrYq98w@mail.gmail.com>
Subject: Re: 0.7.0 release prep - unmerged PRs
To: dev@brooklyn.incubator.apache.org
Content-Type: multipart/alternative; boundary=001a1133694a0b6335051940eeba

--001a1133694a0b6335051940eeba
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Thanks everyone. We've got through most of the items from yesterday; a
couple of rework tasks now need re-reviewed, and there's been a few new PRs
that fix test failures which would be beneficial to include, so need
reviewed too.

Could I get volunteers to review and merge:
714 Clarify operation of `templateOptions` config key
713 Fixing Integration test - RubyRepIntegrationTest
712 Fixing Integration tests for 0.7.0
710 Remove obsolete Tomcat fail binding test
694 [BROOKLYN-99] add support for NovaSecurityGroupExtension

Sam, please could you submit a PR which updates docs/guide/misc/
release-notes.md, as you discussed on IRC?

Thanks all,

Richard.


On Tue, 23 Jun 2015 at 14:39 Duncan Grant <duncan.grant@cloudsoftcorp.com>
wrote:

> Richard, Svet, Sam,
>
> I've created a PR https://github.com/apache/incubator-brooklyn/pull/711 t=
o
> add some documentation for the AWS VPC issue described in #705.  I'm not
> sure whether this is a good place for it but I couldn't see a better one.
>
> Svet,
> Could you review to make sure that my understanding is something like
> reality.
>
> Sam
> I'll write some release notes that detail this as a known issue and, if
> merged, point to this bit of documentation.
>
> Regards
>
> Duncan
>
> On Tue, 23 Jun 2015 at 12:10 Svetoslav Neykov <
> svetoslav.neykov@cloudsoftcorp.com> wrote:
>
> > I'll go though #705.
> >
> > Svet.
> >
> >
> >
> >
> > > On 23.06.2015 =D0=B3., at 14:07, Richard Downer <richard@apache.org> =
wrote:
> > >
> > > I've been looking through the list of unmerged PRs, and with feedback
> > from
> > > Sam C, have pencilled in this list of PRs that should be merged befor=
e
> > > making the release.
> > >
> > > Reviewed and ready to merge
> > > 708 Ignore entity.addFeed(...) call with same feed object
> > > 707 BROOKLYN-137 stopIptables and openIptables don't work on CentOS7
> > > 706 Fix various integration tests
> > > 703 Jsgui tweaks
> > >
> > > Reviewing not completed
> > > 705 Catalog default niceness
> > > 704 clean up how locations are rendered in the js gui, and expand
> > "delete"
> > >
> > > Reviewed with feedback; awaiting contributor to respond
> > > 694 [BROOKLYN-99] add support for NovaSecurityGroupExtension
> > > 639 Allows for lists in templateOptions
> > >
> > > Actions from here
> > >
> > > Could I get a volunteer to review #705? It's a bit of a monster (54
> > changed
> > > files with 1,533 additions and 409 deletions.)
> > >
> > > And also #704 needs a complete review, but it's an easier size to
> manage.
> > >
> > > Andrea, #694 is waiting on your response to review comments. Could yo=
u
> > take
> > > a look please?
> > >
> > > Nakomis is unable to look at #639 today, but it'd be a really useful
> one
> > to
> > > get into the release. I'm going to take over this one. I'll then need
> > > somebody else to review it.
> > >
> > > Finally I'll get the ready-to-merge PRs merged. Once that's done we'l=
l
> > take
> > > a look at the state of the integration tests.
> > >
> > > Thanks,
> > > Richard.
> >
> >
> > --
> > Cloudsoft Corporation Limited, Registered in Scotland No: SC349230.
> >  Registered Office: 13 Dryden Place, Edinburgh, EH9 1RP
> >
> > This e-mail message is confidential and for use by the addressee only. =
If
> > the message is received by anyone other than the addressee, please retu=
rn
> > the message to the sender by replying to it and then delete the message
> > from your computer. Internet e-mails are not necessarily secure.
> Cloudsoft
> > Corporation Limited does not accept responsibility for changes made to
> this
> > message after it was sent.
> >
> > Whilst all reasonable care has been taken to avoid the transmission of
> > viruses, it is the responsibility of the recipient to ensure that the
> > onward transmission, opening or use of this message and any attachments
> > will not adversely affect its systems or data. No responsibility is
> > accepted by Cloudsoft Corporation Limited in this regard and the
> recipient
> > should carry out such virus and other checks as it considers appropriat=
e.
> >
>
> --
> Cloudsoft Corporation Limited, Registered in Scotland No: SC349230.
>  Registered Office: 13 Dryden Place, Edinburgh, EH9 1RP
>
> This e-mail message is confidential and for use by the addressee only. If
> the message is received by anyone other than the addressee, please return
> the message to the sender by replying to it and then delete the message
> from your computer. Internet e-mails are not necessarily secure. Cloudsof=
t
> Corporation Limited does not accept responsibility for changes made to th=
is
> message after it was sent.
>
> Whilst all reasonable care has been taken to avoid the transmission of
> viruses, it is the responsibility of the recipient to ensure that the
> onward transmission, opening or use of this message and any attachments
> will not adversely affect its systems or data. No responsibility is
> accepted by Cloudsoft Corporation Limited in this regard and the recipien=
t
> should carry out such virus and other checks as it considers appropriate.
>

--001a1133694a0b6335051940eeba--