-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17724/#review33849
-----------------------------------------------------------

Ship it!



src/main/python/apache/aurora/client/api/command_runner.py
<https://reviews.apache.org/r/17724/#comment63463>

    cls.EnvironmentError



src/test/python/apache/aurora/client/commands/test_run.py
<https://reviews.apache.org/r/17724/#comment63464>

    revert


- Kevin Sweeney


On Feb. 6, 2014, 1:40 p.m., Mark Chu-Carroll wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17724/
> -----------------------------------------------------------
> 
> (Updated Feb. 6, 2014, 1:40 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Brian Wickman.
> 
> 
> Bugs: aurora-128
>     https://issues.apache.org/jira/browse/aurora-128
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Add an exception catch to provide a better error message when distributed 
> command runner fails due to un-specified executor parameters in the slave 
> configurations.
> 
> 
> Diffs
> -----
> 
>   src/main/python/apache/aurora/client/api/command_runner.py 
> f7605f40b5c4ec864a1bae68649f282268260528 
>   src/main/python/apache/aurora/client/commands/run.py 
> 332bded632ee4cfc0a986540239cb4cc296807ea 
>   src/main/python/apache/aurora/client/commands/ssh.py 
> d1499ec1e9e910bc32f5f9f4b050c53316348503 
>   src/test/python/apache/aurora/client/commands/test_run.py 
> abcc275d57373038227d3805c16b17b967f92048 
> 
> Diff: https://reviews.apache.org/r/17724/diff/
> 
> 
> Testing
> -------
> 
> Manual testing on the command line.
> 
> 
> Thanks,
> 
> Mark Chu-Carroll
> 
>

Reply via email to