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

(Updated Oct. 6, 2014, 2:54 p.m.)


Review request for Aurora, David McLaughlin and Zameer Manji.


Changes
-------

Apologies - I've got 5 different changes open in RB, and I accidentally based 
this diff against the wrong one. I'll have the correct diff uploaded in a 
couple of minutes.


Summary (updated)
-----------------

Improve handling of unknown errors in the aurora client.


Bugs: aurora-779
    https://issues.apache.org/jira/browse/aurora-779


Repository: aurora


Description
-------

Improve handling of unknown errors in the aurora client.

Instead of dumping the stack on the user's terminal, or
absorbing the error and generating a brief error
message, the client now writes detailed information about
the error is written into an error log file, and the
user is given a clean error message referring them to that
file for details.


Diffs
-----

  src/main/python/apache/aurora/client/cli/__init__.py 
e0c3050151bca1128ed7e476ec5133407a20f6c2 
  src/main/python/apache/aurora/client/cli/context.py 
f639af7de93a069b278dc494b6f92a2f6b10de9c 
  src/main/python/apache/aurora/client/cli/options.py 
9a81a1d7d04c6648e94ff117429278317a9dbed8 
  src/main/python/apache/aurora/client/cli/standalone_client.py 
fd2232bc8b7e5caf487291f657a50184391a8c69 
  src/main/python/apache/aurora/client/cli/update.py 
b4dd792dc12f19424c620f4d91748113e272f0c9 
  src/test/python/apache/aurora/client/cli/test_api_from_cli.py 
78f21d2f20cf71fa2dfe0614885d44d2948decd2 
  src/test/python/apache/aurora/client/cli/test_create.py 
6e55188bdfc576506848605debb391288e696fe3 
  src/test/python/apache/aurora/client/cli/test_supdate.py 
2782fee2867c6ef79349240299de082f07f7967a 
  src/test/python/apache/aurora/client/cli/util.py 
e1ee884c06f3bc22bcd9e908ff61af9459a0b535 

Diff: https://reviews.apache.org/r/26328/diff/


Testing
-------

New unit test.


Thanks,

Mark Chu-Carroll

Reply via email to