Re: Review Request 26458: Adding wait loop into host_drain status monitoring.

2014-10-08 Thread Tobias Weingartner
://reviews.apache.org/r/26458/#comment96280 Will this abort aurora_admin? If so, writing these to the failed set is more desirable, along with continuing. - Tobias Weingartner On Oct. 8, 2014, 9:40 p.m., Maxim Khutornenko wrote

Re: Review Request 20285: Improve documentation and testing for host maintenance API

2014-05-21 Thread Tobias Weingartner
, would it make sense to use a list of lists? src/main/python/apache/aurora/admin/host_maintenance.py https://reviews.apache.org/r/20285/#comment77900 s/give/given - Tobias Weingartner On May 21, 2014, 3:51 a.m., Joe Smith wrote

Re: Review Request 19694: Add maintenance commands into aurora_admin

2014-03-26 Thread Tobias Weingartner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19694/#review38643 --- Ship it! Ship It! - Tobias Weingartner On March 26, 2014, 6:20

Re: Review Request 17752: Add task noun, supporting run and ssh verbs.

2014-02-11 Thread Tobias Weingartner
possibly root, what is the use-case for this? - Tobias Weingartner On Feb. 7, 2014, 8:44 p.m., Mark Chu-Carroll wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17752

Re: Review Request 17042: Add a web interface to the zookeeper instance.

2014-02-09 Thread Tobias Weingartner
# point web browser at 192.168.33.2:8080 Thanks, Tobias Weingartner

Re: Review Request 16983: Fix maintenance command documentation and host/filename logic.

2014-01-16 Thread Tobias Weingartner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16983/#review32081 --- Ship it! Ship It! - Tobias Weingartner On Jan. 16, 2014, 9:17

Re: Review Request 16615: Client command hooks doc

2014-01-13 Thread Tobias Weingartner
On Jan. 13, 2014, 7:27 p.m., Kevin Sweeney wrote: In the interest of avoiding a proliferation of broken documentation links/links to drafts, would everyone be all right with committing this as-is and continuing to iterate in later reviews? Normally I prefer to review small changes in