Re: Review Request 29672: Fixing failed test.

2015-01-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29672/#review67104 --- Ship it! Master (5d8424c) is green with this patch. ./build-suppo

Re: Review Request 29672: Fixing failed test.

2015-01-07 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29672/ --- (Updated Jan. 7, 2015, 9:52 p.m.) Review request for Aurora and Kevin Sweeney.

Re: Review Request 29672: Fixing failed test.

2015-01-07 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29672/#review67101 --- Ship it! Ship It! - Kevin Sweeney On Jan. 7, 2015, 1:30 p.m., Ma

Re: Review Request 29672: Fixing failed test.

2015-01-07 Thread Maxim Khutornenko
> On Jan. 7, 2015, 9:48 p.m., Kevin Sweeney wrote: > > src/test/python/apache/aurora/client/commands/test_diff.py, line 82 > > > > > > While you're here mind changing these to kwargs form? That will make > > this test

Re: Review Request 29672: Fixing failed test.

2015-01-07 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29672/#review67099 --- src/test/python/apache/aurora/client/commands/test_diff.py