Re: Review Request 36972: One python_library per exported setup.py project

> On July 31, 2015, 3:21 p.m., Brian Wickman wrote: > > I think it makes sense to split into two binary-exporting packages: one > > client-side and one server-side. The client side (aurora.client) should > > contain aurora client and aurora admin client. The server-side (reuse > > aurora.exec

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93932 --- Ship it! - Brian Wickman On Aug. 1, 2015, 10:38 p.m., Kevin Sween

Re: Review Request 36972: One python_library per exported setup.py project

> On July 31, 2015, 10:21 p.m., Brian Wickman wrote: > > I think it makes sense to split into two binary-exporting packages: one > > client-side and one server-side. The client side (aurora.client) should > > contain aurora client and aurora admin client. The server-side (reuse > > aurora.exe

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93930 --- Ship it! Ship It! - Bill Farner On Aug. 1, 2015, 10:38 p.m., Kev

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93851 --- Master (a45a8ee) is green with this patch. ./build-support/jenkins

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/ --- (Updated Aug. 1, 2015, 3:38 p.m.) Review request for Aurora, Bill Farner and Br

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93829 --- Ship it! Ship It! - Joshua Cohen On July 31, 2015, 11:22 p.m., K

Re: Review Request 36972: One python_library per exported setup.py project

> On July 31, 2015, 10:57 p.m., Joshua Cohen wrote: > > I'm -1 to shipping without something under docs/ explaining this > > organizational structure. > > Bill Farner wrote: > maybe a README.md under `src/main/python/apache` instead? > > Kevin Sweeney wrote: > I've added a new file doc

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93818 --- Master (a45a8ee) is green with this patch. ./build-support/jenkins

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/ --- (Updated July 31, 2015, 4:22 p.m.) Review request for Aurora, Bill Farner and B

Re: Review Request 36972: One python_library per exported setup.py project

> On July 31, 2015, 3:57 p.m., Joshua Cohen wrote: > > I'm -1 to shipping without something under docs/ explaining this > > organizational structure. > > Bill Farner wrote: > maybe a README.md under `src/main/python/apache` instead? I've added a new file docs/build-system.md - Kevin --

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/ --- (Updated July 31, 2015, 4:18 p.m.) Review request for Aurora, Bill Farner and B

Re: Review Request 36972: One python_library per exported setup.py project

> On July 31, 2015, 10:57 p.m., Joshua Cohen wrote: > > I'm -1 to shipping without something under docs/ explaining this > > organizational structure. maybe a README.md under `src/main/python/apache` instead? - Bill --- This is an auto

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93805 --- I'm -1 to shipping without something under docs/ explaining this org

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/ --- (Updated July 31, 2015, 3:49 p.m.) Review request for Aurora, Bill Farner and B

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93799 --- Master (6cab9c5) is green with this patch. ./build-support/jenkins

Re: Review Request 36972: One python_library per exported setup.py project

> On July 31, 2015, 3:21 p.m., Brian Wickman wrote: > > I think it makes sense to split into two binary-exporting packages: one > > client-side and one server-side. The client side (aurora.client) should > > contain aurora client and aurora admin client. The server-side (reuse > > aurora.exec

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/ --- (Updated July 31, 2015, 3:30 p.m.) Review request for Aurora, Bill Farner and B

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93794 --- Master (6cab9c5) is green with this patch. ./build-support/jenkins

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93793 --- I think it makes sense to split into two binary-exporting packages:

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/ --- (Updated July 31, 2015, 3:03 p.m.) Review request for Aurora, Bill Farner and B

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93756 --- Master (d327773) is green with this patch. ./build-support/jenkins

Re: Review Request 36972: One python_library per exported setup.py project

> On July 31, 2015, 4:01 p.m., Maxim Khutornenko wrote: > > src/main/python/apache/aurora/admin/BUILD, lines 84-92 > > > > > > Are we completely abondoning the idea of having a non-kerberized client > > version with

Re: Review Request 36972: One python_library per exported setup.py project

> On July 31, 2015, 9:01 a.m., Maxim Khutornenko wrote: > > src/main/python/apache/aurora/admin/BUILD, lines 84-92 > > > > > > Are we completely abondoning the idea of having a non-kerberized client > > version with

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/ --- (Updated July 31, 2015, 11:03 a.m.) Review request for Aurora, Bill Farner and

Re: Review Request 36972: One python_library per exported setup.py project

> On July 31, 2015, 7:39 a.m., Joshua Cohen wrote: > > src/main/python/apache/thermos/bin/BUILD, line 40 > > > > > > Should this read from .auroraversion? Good catch! (leftover from my example email). Fixed. - Kevi

Re: Review Request 36972: One python_library per exported setup.py project

> On July 31, 2015, 10:19 a.m., Brian Wickman wrote: > > src/main/python/apache/aurora/common/BUILD, line 61 > > > > > > This is probably the toughest one to swallow. Pulling in aurora.common > > just for aurora_job

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/ --- (Updated July 31, 2015, 10:56 a.m.) Review request for Aurora, Bill Farner and

Re: Review Request 36972: One python_library per exported setup.py project

> On July 31, 2015, 4:01 p.m., Maxim Khutornenko wrote: > > src/main/python/apache/aurora/admin/BUILD, lines 84-92 > > > > > > Are we completely abondoning the idea of having a non-kerberized client > > version with

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93744 --- lgtm at a high level. src/main/python/apache/aurora/common/BUILD (

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93731 --- src/main/python/apache/aurora/admin/BUILD

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93722 --- Seems reasonable to me. We should definitely add some documentation

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93696 --- Master (d327773) is green with this patch. ./build-support/jenkins

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/ --- (Updated July 30, 2015, 8:37 p.m.) Review request for Aurora, Bill Farner and B

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93695 --- Master (d327773) is green with this patch. ./build-support/jenkins

Re: Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93693 --- LGTM once relevant tests come up green and NEWS entry is added. Nic

Review Request 36972: One python_library per exported setup.py project

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/ --- Review request for Aurora, Bill Farner and Brian Wickman. Bugs: AURORA-1381