----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18153/#review34735 -----------------------------------------------------------
src/main/python/apache/aurora/client/cli/options.py <https://reviews.apache.org/r/18153/#comment64979> This will throw a ValueError if the user didn't give a numeric value. - Mark Chu-Carroll On Feb. 14, 2014, 7:01 p.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18153/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2014, 7:01 p.m.) > > > Review request for Aurora, Mark Chu-Carroll and Brian Wickman. > > > Bugs: AURORA-207 > https://issues.apache.org/jira/browse/AURORA-207 > > > Repository: aurora > > > Description > ------- > > Added "sla get_job_uptime" command. > > Also, changed from ValueError to ArgumentTypeError in options.py as the > former is not handled properly by argparser and no custom messages were > reported. > > > Diffs > ----- > > src/main/python/apache/aurora/client/api/sla.py > b69d20818836fb6ae87b333e33008b27e013954f > src/main/python/apache/aurora/client/cli/options.py > 017f141a605eb6b522fcd7c4fe827eefdc626fca > src/main/python/apache/aurora/client/cli/sla.py > a3973df870567052083b1d5a524438191eb6d812 > src/test/python/apache/aurora/client/api/test_sla.py > 54f904c122b9e41d85dc7ad63fe3698cd0bf2923 > src/test/python/apache/aurora/client/cli/test_sla.py > 703d39e4566a30900f1e9bcca5871c4a426cc6c3 > > Diff: https://reviews.apache.org/r/18153/diff/ > > > Testing > ------- > > ./build-support/jenkins/build.sh > > > Thanks, > > Maxim Khutornenko > >