----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26417/#review55664 -----------------------------------------------------------
Ship it! Ship It! - Zameer Manji On Oct. 7, 2014, 10:12 a.m., Mark Chu-Carroll wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26417/ > ----------------------------------------------------------- > > (Updated Oct. 7, 2014, 10:12 a.m.) > > > Review request for Aurora, David McLaughlin and Zameer Manji. > > > Bugs: aurora-803 > https://issues.apache.org/jira/browse/aurora-803 > > > Repository: aurora > > > Description > ------- > > Instead of just exiting with an error code, return a valid JSON > structure when a user calls "aurora job status", and no job matching > the jobspec is running. > > Also, remove a bunch of "pants" labels from build files, which generated > warnings while building the tests. > > > Diffs > ----- > > src/main/python/apache/aurora/client/cli/context.py > 0816ac02846c30e0beab5c0339aca38f15c4ae10 > src/main/python/apache/aurora/client/cli/jobs.py > 103fb53cb5101d3e025d8712e3887bccdfbb6aeb > src/test/python/apache/aurora/client/cli/BUILD > 8ce6bd5b7faa1579372fb6935180ea982af64af8 > src/test/python/apache/aurora/client/cli/test_status.py > bd30b11022ee0d1fb38da9f6efd4d9c5923b7d13 > > Diff: https://reviews.apache.org/r/26417/diff/ > > > Testing > ------- > > Ran all unit tests; added new test for the empty job list json case. > > > Thanks, > > Mark Chu-Carroll > >