This is an automatically generated e-mail. To reply, visit:

(Updated Oct. 22, 2014, 11:05 a.m.)

Review request for Aurora, Bill Farner and Zameer Manji.


Removed debugging statements.

Bugs: AURORA-876

Repository: aurora


Use virtualenv to build pants instead of pex.

The goal of this change is to reduce CI flakiness by using `pip install` rather 
than `pex` to bootstrap pants. This still reaches out to external servers (and 
thus has the potential to be flaky) but is at least configurable (via 
`--default-timeout`). In a future review we can consider mirroring to 
svn.apache.org and disabling PyPI lookups.

This requires a change to production executor code to use `sys.executable` 
rather than chmod+x-ing the runner PEX. Failure to do this results in the pants 
virtualenv's `site-packages` shadowing the dependencies in the `pex` (fairly 
obviously as there is a test failure related to psutil1/2 conflicts). I'm sure 
I'm papering over a PEX bug here but IMO this is fine as the only reason to 
execute the thermos runner PEX under a different python interpreter than the 
executor is using (based on the hash-bang in the pex) is not a configuration we 
need to support IMO.

Diffs (updated)

  pants 168419d7d8f5d89a0d87bfc6ac9f06c9e8e3dbc2 

Diff: https://reviews.apache.org/r/27009/diff/


git clean -fdx
./pants src/test/python:all


Kevin Sweeney

Reply via email to