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 https://reviews.apache.org/r/31659/diff/1/?file=882656#file882656line130 Not directly related but adding --batch-size=2 would make it even faster. This crossed my

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.

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 https://reviews.apache.org/r/31659/diff/1/?file=882656#file882656line64 Is this statement still accurate now that ssh is dropped from the command? It is not!

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

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

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

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.,

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:

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.

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