Re: Review Request 31659: Clean up end-to-end tests.

2015-03-03 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31659/#review75090 --- Ship it! Ship It! - Kevin Sweeney On March 3, 2015, 12:55 p.m.,

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31659/#review75049 --- Ship it! Master (782e3e7) is green with this patch. ./build-suppo

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-03 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31659/#review75048 --- Ship it! Just tried this out locally. So much faster, thanks for do

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31659/ --- (Updated March 3, 2015, 8:53 p.m.) Review request for Aurora, Kevin Sweeney and

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31659/ --- (Updated March 3, 2015, 8:55 p.m.) Review request for Aurora, Kevin Sweeney and

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31659/#review75045 --- Ship it! Ship It! - Maxim Khutornenko On March 3, 2015, 8:53 p.m

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-03 Thread Bill Farner
> On March 3, 2015, 6:10 p.m., Maxim Khutornenko wrote: > > src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh, line 130 > > > > > > Not directly related but adding --batch-size=2 would make it even > > faster. > >

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-03 Thread Bill Farner
> On March 3, 2015, 6:10 p.m., Maxim Khutornenko wrote: > > src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh, line 130 > > > > > > Not directly related but adding --batch-size=2 would make it even > > faster. Thi

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-03 Thread Bill Farner
> On March 3, 2015, 5:23 p.m., Maxim Khutornenko wrote: > > src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh, line 64 > > > > > > Is this statement still accurate now that ssh is dropped from the > > command? It i

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31659/#review74994 --- src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31659/#review74983 --- Ship it! src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-02 Thread Bill Farner
> On March 3, 2015, 3:38 a.m., Aurora ReviewBot wrote: > > Master (4cca6a6) is red with this patch. > > ./build-support/jenkins/build.sh > > > > src.test.python.apache.aurora.client.cli.cron > > . SUCCESS > > sr

Re: Review Request 31659: Clean up end-to-end tests.

2015-03-02 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31659/#review74881 --- Master (4cca6a6) is red with this patch. ./build-support/jenkins/b

Review Request 31659: Clean up end-to-end tests.

2015-03-02 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31659/ --- Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. Repository: aur