Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
MIME-Version: 1.0
In-Reply-To: <53cdd062-71ed-46d8-bcbe-3826ef7ddefa@msgid.missiveapp.com>
References: <CAJVWTBLoJn4+NXSGg+Nb61LLP00HOHBs9-gF=V7PcS0Vrs=q2A@mail.gmail.com>
 <53cdd062-71ed-46d8-bcbe-3826ef7ddefa@msgid.missiveapp.com>
From: Kris Sterckx <kris.sterckx@nuagenetworks.net>
Date: Tue, 6 Sep 2016 21:10:10 +0200
Message-ID: <CAJVWTB+cPBuHoAk8LyHBGDxzkvyqWaMpw+WeABT9OPWb9VyFMQ@mail.gmail.com>
Subject: Re: PR reviews for 4.10.0 release
To: dev@cloudstack.apache.org
Content-Type: multipart/alternative; boundary=001a114e27f6a993c1053bdb8bc4
archived-at: Tue, 06 Sep 2016 19:10:23 -0000

--001a114e27f6a993c1053bdb8bc4
Content-Type: text/plain; charset=UTF-8

Thanks Rajani

https://github.com/apache/cloudstack/pull/1578 has votes and tests pass.

What would be the next step ?

Best,

Kris


On 30 August 2016 at 07:24, Rajani Karuturi <rajani@apache.org> wrote:

> If you could get 2 LGTMs(any one from your team can also review
> and give LGTM) and if you can run BVT suite on
> PR(https://github.com/apache/cloudstack/tree/master/test/integration/smoke
> ),
> they can be immediately merged.
>
> ~ Rajani
> http://cloudplatform.accelerite.com/
>
> On August 30, 2016 at 3:47 AM, Kris Sterckx
> (kris.sterckx@nuagenetworks.net) wrote:
> Hi All,
>
> The Nuage Networks team is investing in ACS 4.10.0 and has
> several feature
> PR's outstanding for review :
>
> - https://github.com/apache/cloudstack/pull/1578
> - https://github.com/apache/cloudstack/pull/1580
> - https://github.com/apache/cloudstack/pull/1579
> - https://github.com/apache/cloudstack/pull/1577
>
> How can we better facilitate the review of these PR's ?
>
> Thanks,
>
> - Kris
>
> --
> Kris Sterckx
>
> CloudStack Development Lead
>
> Nuage Networks
>

--001a114e27f6a993c1053bdb8bc4--