Re: Review Request 22408: Log SlaveID without newline.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22408/ --- (Updated June 9, 2014, 6:45 p.m.) Review request for Aurora, Maxim Khutornenko

Review Request 22408: Log SlaveID without newline.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22408/ --- Review request for Aurora, Maxim Khutornenko and Brian Wickman. Repository: aur

Re: Review Request 22407: Add transport to common package.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22407/#review45187 --- Ship it! Ship It! - Kevin Sweeney On June 9, 2014, 6:23 p.m., Br

Review Request 22407: Add transport to common package.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22407/ --- Review request for Aurora and Kevin Sweeney. Repository: aurora Description -

Re: Review Request 22167: Adding SLA check into perform_maintenance_hosts command.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22167/#review45175 --- Ping, Brian/Joe. - Maxim Khutornenko On June 9, 2014, 11:10 p.m.,

Re: Review Request 22280: Implement a TRequestsClient as a prelude to kerberization.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22280/#review45174 --- Ship it! Ship It! - Kevin Sweeney On June 9, 2014, 4:42 p.m., Br

Re: Review Request 22280: Implement a TRequestsClient as a prelude to kerberization.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22280/ --- (Updated June 9, 2014, 11:42 p.m.) Review request for Aurora, Kevin Sweeney and

Re: Review Request 22167: Adding SLA check into perform_maintenance_hosts command.

> On June 9, 2014, 8:43 p.m., Mark Chu-Carroll wrote: > > src/main/python/apache/aurora/client/commands/maintenance.py, line 72 > > > > > > This help line is unclear: what does "time interval for the percentage > > of

Re: Review Request 22167: Adding SLA check into perform_maintenance_hosts command.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22167/ --- (Updated June 9, 2014, 11:10 p.m.) Review request for Aurora, Joe Smith, Mark C

Re: Review Request 22167: Adding SLA check into perform_maintenance_hosts command.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22167/#review45164 --- Ship it! Ship It! - Mark Chu-Carroll On June 9, 2014, 6:08 p.m.,

Re: Review Request 22167: Adding SLA check into perform_maintenance_hosts command.

> On June 9, 2014, 4:43 p.m., Mark Chu-Carroll wrote: > > src/main/python/apache/aurora/client/commands/maintenance.py, line 72 > > > > > > This help line is unclear: what does "time interval for the percentage > > of

Re: Review Request 22167: Adding SLA check into perform_maintenance_hosts command.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22167/ --- (Updated June 9, 2014, 10:08 p.m.) Review request for Aurora, Joe Smith, Mark C

Re: Review Request 22167: Adding SLA check into perform_maintenance_hosts command.

> On June 9, 2014, 8:43 p.m., Mark Chu-Carroll wrote: > > src/main/python/apache/aurora/admin/admin_util.py, line 88 > > > > > > You've got "parse_hosts", "parst_host_file", "parse_hosts_optional", > > and "parse_host_

Re: Review Request 22280: Implement a TRequestsClient as a prelude to kerberization.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22280/#review45138 --- Ship it! Ship It! - Kevin Sweeney On June 9, 2014, 1:56 p.m., Br

Re: Review Request 22263: Add a short form of help for v1 client, so that merged help isn't unreasonably long.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22263/#review45137 --- Ship it! Ship It! - Brian Wickman On June 5, 2014, 2:02 p.m., Ma

Re: Review Request 22280: Implement a TRequestsClient as a prelude to kerberization.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22280/ --- (Updated June 9, 2014, 8:56 p.m.) Review request for Aurora, Kevin Sweeney and

Re: Review Request 22167: Adding SLA check into perform_maintenance_hosts command.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22167/#review45131 --- src/main/python/apache/aurora/admin/admin_util.py

Re: Review Request 22167: Adding SLA check into perform_maintenance_hosts command.

> On June 9, 2014, 7:17 p.m., Mark Chu-Carroll wrote: > > src/main/python/apache/aurora/admin/admin_util.py, line 14 > > > > > > Are you sure you uploaded the correct diff? In your reply to the > > review, you said you

Re: Review Request 22263: Add a short form of help for v1 client, so that merged help isn't unreasonably long.

> On June 6, 2014, 3:38 a.m., David McLaughlin wrote: > > Ship It! wickman, ping? - Mark --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22263/#review44881 ---

Re: Review Request 22243: Variety of help fixes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22243/ --- (Updated June 9, 2014, 4:13 p.m.) Review request for Aurora, David McLaughlin a

Re: Review Request 22281: Fixing broken increase_quota command.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22281/#review45122 --- Ship it! Ship It! - Kevin Sweeney On June 6, 2014, 11:01 a.m., M

Re: Review Request 21943: Accept a job key for config loading.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21943/#review45119 --- Ship it! schweet - Joe Smith On May 29, 2014, 11:21 a.m., Antoin

Re: Review Request 22281: Fixing broken increase_quota command.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22281/#review45116 --- Ship it! Ship It! - Joe Smith On June 6, 2014, 11:01 a.m., Maxim

Re: Review Request 22243: Variety of help fixes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22243/#review45114 --- Ship it! Ship It! - David McLaughlin On June 4, 2014, 6:39 p.m.,

Re: Review Request 22280: Implement a TRequestsClient as a prelude to kerberization.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22280/#review45110 --- src/main/python/apache/aurora/common/transport.py

Re: Review Request 22334: Fixing quota check for non-prod/prod job transition.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22334/#review45113 --- Ship it! Ship It! - Brian Wickman On June 6, 2014, 11:32 p.m., M

Re: Review Request 22243: Variety of help fixes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22243/#review45112 --- Ship it! modulo checkstyle nit src/main/python/apache/aurora/clie

Re: Review Request 21943: Accept a job key for config loading.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21943/#review45111 --- Ship it! ef yeah! - Brian Wickman On May 29, 2014, 6:21 p.m., An

Re: Review Request 22167: Adding SLA check into perform_maintenance_hosts command.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22167/#review45109 --- src/main/python/apache/aurora/admin/admin_util.py

Re: Review Request 22280: Implement a TRequestsClient as a prelude to kerberization.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22280/#review45108 --- Ship it! Approach looks good to me. - Mark Chu-Carroll On June 9

Re: Review Request 22280: Implement a TRequestsClient as a prelude to kerberization.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22280/ --- (Updated June 9, 2014, 6:42 p.m.) Review request for Aurora, Kevin Sweeney and

Re: Review Request 22167: Adding SLA check into perform_maintenance_hosts command.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22167/#review45082 --- Ping. - Maxim Khutornenko On June 4, 2014, 1:46 a.m., Maxim Khuto