Mailing-List: contact reviews-help@aurora.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: reviews@aurora.incubator.apache.org
Content-Type: multipart/alternative;
 boundary="===============1796351385443266442=="
MIME-Version: 1.0
Subject: Re: Review Request 27586: Increase robustness when reading PEX-INFO
From: "Aurora ReviewBot" <wfarner@apache.org>
To: "Bill Farner" <wfarner@apache.org>
Cc: "Aurora ReviewBot" <wfarner@apache.org>,
 "Aurora" <reviews@aurora.incubator.apache.org>,
 "Zameer Manji" <zmanji@twopensource.com>
Date: Tue, 04 Nov 2014 21:51:12 -0000
Message-ID: <20141104215112.10454.22190@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: "Aurora ReviewBot" <noreply@reviews.apache.org>
References: <20141104212236.10454.71143@reviews.apache.org>
In-Reply-To: <20141104212236.10454.71143@reviews.apache.org>
Reply-To: "Aurora ReviewBot" <wfarner@apache.org>

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


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

Ship it!


Master (9ef905a) is green with this patch.
  ./build-support/jenkins/build.sh

- Aurora ReviewBot


On Nov. 4, 2014, 9:22 p.m., Zameer Manji wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27586/
> -----------------------------------------------------------
> 
> (Updated Nov. 4, 2014, 9:22 p.m.)
> 
> 
> Review request for Aurora and Bill Farner.
> 
> 
> Bugs: AURORA-456
>     https://issues.apache.org/jira/browse/AURORA-456
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> This patch prevents aurora version from crashing when it is built with a different version of pants.
> 
> 
> Diffs
> -----
> 
>   src/main/python/apache/aurora/client/commands/core.py 1644f84b5887c2f8172789d82de00e6a735f5d0c 
>   src/test/python/apache/aurora/client/commands/BUILD 17933dedfa08c9d12c369087bf801e7c35cdde9b 
>   src/test/python/apache/aurora/client/commands/test_version.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/27586/diff/
> 
> 
> Testing
> -------
> 
> [tw-mbp-zmanji aurora (fix-version)]$ ./pants ./src/main/python/apache/aurora/client/cli:aurora2
> Build operating on top level addresses: set([BuildFileAddress(/Users/zmanji/workspace/aurora/src/main/python/apache/aurora/client/cli/BUILD, aurora2)])
> Building PythonBinary PythonBinary(BuildFileAddress(/Users/zmanji/workspace/aurora/src/main/python/apache/aurora/client/cli/BUILD, aurora2)):
> Wrote /Users/zmanji/workspace/aurora/dist/aurora2.pex
> [tw-mbp-zmanji aurora (fix-version)]$ ./pants src/main/python/apache/aurora/client/bin:aurora_client
> Build operating on top level addresses: set([BuildFileAddress(/Users/zmanji/workspace/aurora/src/main/python/apache/aurora/client/bin/BUILD, aurora_client)])
> Building PythonBinary PythonBinary(BuildFileAddress(/Users/zmanji/workspace/aurora/src/main/python/apache/aurora/client/bin/BUILD, aurora_client)):
> Wrote /Users/zmanji/workspace/aurora/dist/aurora_client.pex
> [tw-mbp-zmanji aurora (fix-version)]$ ./dist/aurora2.pex version
> Aurora client build info:
> 	sha: d03187404dac583884a979dd302dd3cd36163c98
> 	date: Tuesday Nov 04, 2014 11:39:07
> Aurora API version: APIVersion(major=3)
> [tw-mbp-zmanji aurora (fix-version)]$ ./dist/aurora_client.pex version
> Aurora client build info:
> 	sha: d03187404dac583884a979dd302dd3cd36163c98
> 	date: Tuesday Nov 04, 2014 11:39:18
> Aurora API version: APIVersion(major=3)
> 
> 
> Thanks,
> 
> Zameer Manji
> 
>


--===============1796351385443266442==--