----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16130/#review30453 -----------------------------------------------------------
Ship it! Ship It! - Brian Wickman On Dec. 16, 2013, 4:45 p.m., Mark Chu-Carroll wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16130/ > ----------------------------------------------------------- > > (Updated Dec. 16, 2013, 4:45 p.m.) > > > Review request for Aurora, Jonathan Boulle and Brian Wickman. > > > Repository: aurora > > > Description > ------- > > First step towards aurora client v2! > > - Initial implementation of the noun/verb command and options processing > framework. > - Initial implementation of a command processing context for aurora commands. > - Implementations of a "Job" noun and "create" and "kill" verbs. > - Tests. > > Note: not all of the v1 tests for the create and kill verbs have been > migrated to the new > framework; command processing contexts need a bit more work to make it easy > to do the > appropriate mocking/stubbing to support them. They'll be in the next change. > > > Diffs > ----- > > src/main/python/twitter/aurora/BUILD.thirdparty > 87156eaa48a39689e18a3190fda487bcfe755771 > src/main/python/twitter/aurora/client/cli/BUILD PRE-CREATION > src/main/python/twitter/aurora/client/cli/__init__.py PRE-CREATION > src/main/python/twitter/aurora/client/cli/context.py PRE-CREATION > src/main/python/twitter/aurora/client/cli/jobs.py PRE-CREATION > src/test/python/twitter/aurora/client/cli/BUILD PRE-CREATION > src/test/python/twitter/aurora/client/cli/test_create.py PRE-CREATION > src/test/python/twitter/aurora/client/cli/test_kill.py PRE-CREATION > src/test/python/twitter/aurora/client/cli/util.py PRE-CREATION > src/test/python/twitter/aurora/client/commands/test_kill.py > 3649969c77f992688a7ddbe592eda8d4edb94036 > > Diff: https://reviews.apache.org/r/16130/diff/ > > > Testing > ------- > > > Thanks, > > Mark Chu-Carroll > >
