Mailing-List: contact reviews-help@aurora.apache.org; run by ezmlm
Precedence: bulk
Reply-To: reviews@aurora.apache.org
Content-Type: multipart/alternative;
 boundary="===============6650119254700489107=="
MIME-Version: 1.0
Subject: Re: Review Request 33854: Fix and enhance RC verification script.
From: "Kevin Sweeney" <kevints@apache.org>
To: "Jake Farrell" <jfarrell@apache.org>
Cc: "Bill Farner" <wfarner@apache.org>, "Kevin Sweeney" <kevints@apache.org>,
 "Aurora" <reviews@aurora.apache.org>
Date: Thu, 07 May 2015 18:34:25 -0000
Message-ID: <20150507183425.1563.78697@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: "Kevin Sweeney" <noreply@reviews.apache.org>
References: <20150507170638.1562.74537@reviews.apache.org>
In-Reply-To: <20150507170638.1562.74537@reviews.apache.org>
Reply-To: "Kevin Sweeney" <kevints@apache.org>

--===============6650119254700489107==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


> On May 7, 2015, 10:06 a.m., Jake Farrell wrote:
> > build-support/release/verify-release-candidate, line 70
> > <https://reviews.apache.org/r/33854/diff/3/?file=950636#file950636line70>
> >
> >     Not crazy about having multiple locations for gradle versions to be set, do we have to include gradle download here or could we just assume that gradle is all ready on the system as a build dependency
> 
> Jake Farrell wrote:
>     +1 to the patch other than the gradle download, since we all ready depend on vagrant in the e2e test done see why we cant make gradle a similar required dependency
> 
> Bill Farner wrote:
>     The issue becomes the current strict gradle version check.  I will change the gradle version check to be less strict, which will make me comfortable with using the system-installed gradle here.

This is a complete duplication of the gradle wrapper. Is there a way we can include this standard tool rather than reinvent it?


- Kevin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33854/#review82837
-----------------------------------------------------------


On May 5, 2015, 3:14 p.m., Bill Farner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/33854/
> -----------------------------------------------------------
> 
> (Updated May 5, 2015, 3:14 p.m.)
> 
> 
> Review request for Aurora and Jake Farrell.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> This makes the verification script more complete, automating the basic set of tests that release voters are expected to run.
> 
> 
> Diffs
> -----
> 
>   build-support/release/verify-release-candidate 0d4d6e0c01ebe006056198d25b165b9658156653 
> 
> Diff: https://reviews.apache.org/r/33854/diff/
> 
> 
> Testing
> -------
> 
> TODO(wfarner): Report back results of run against 0.8.0-rc0
> 
> 
> Thanks,
> 
> Bill Farner
> 
>


--===============6650119254700489107==--