aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David McLaughlin" <da...@dmclaughlin.com>
Subject Re: Review Request 24852: Add command output tests for "job create", "job killall", "job kill"
Date Tue, 19 Aug 2014 17:47:12 GMT

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

Ship it!


Ship It!

- David McLaughlin


On Aug. 19, 2014, 3:22 p.m., Mark Chu-Carroll wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/24852/
> -----------------------------------------------------------
> 
> (Updated Aug. 19, 2014, 3:22 p.m.)
> 
> 
> Review request for Aurora, David McLaughlin and Maxim Khutornenko.
> 
> 
> Bugs: aurora-645
>     https://issues.apache.org/jira/browse/aurora-645
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Add tests to ensure that IO behavior of client commands is correct.
> 
> Improve IO behavior in the IO client, so that users get more information.
> Formorely, many commands succeeded silently, when it would be more helpful to actually
tell the user that you did something. (This shouldn't affect scripting, because in these commands,
the only result that's needed by a scripter is succeess/failure, which is carried by the exit
code.)
> 
> 
> Diffs
> -----
> 
>   src/main/python/apache/aurora/client/cli/context.py 0e10589c2e8ab6decedb12122a0d8c7a81285e18

>   src/main/python/apache/aurora/client/cli/jobs.py 109ce59e3e8dc1c6f4fb918522718cc47867020d

>   src/test/python/apache/aurora/client/cli/test_create.py ca635bd49acec9596db67839451101a59ba4d761

>   src/test/python/apache/aurora/client/cli/test_kill.py ee64908855a4960f44ce96c810e69dd105d2ce5d

>   src/test/python/apache/aurora/client/cli/test_restart.py 14a08e873058374a9bfe4ffbff8ae7279123f264

> 
> Diff: https://reviews.apache.org/r/24852/diff/
> 
> 
> Testing
> -------
> 
> Ran test suite.
> 
> 
> Thanks,
> 
> Mark Chu-Carroll
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message