Re: Review Request 31966: Add client support for including messages when changing update state.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31966/#review76333 --- Ship it! Master (ee1a13a) is green with this patch. ./build-suppo

Re: Review Request 32014: Adding more logging into MaintenanceController.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32014/#review76332 --- src/main/java/org/apache/aurora/scheduler/state/MaintenanceControll

Re: Review Request 31966: Add client support for including messages when changing update state.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31966/ --- (Updated March 13, 2015, 2:12 a.m.) Review request for Aurora, Maxim Khutornenk

Re: Review Request 31966: Add client support for including messages when changing update state.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31966/#review76331 --- Master (ee1a13a) is red with this patch. ./build-support/jenkins/b

Re: Review Request 31966: Add client support for including messages when changing update state.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31966/ --- (Updated March 13, 2015, 1:58 a.m.) Review request for Aurora, Maxim Khutornenk

Re: Review Request 32014: Adding more logging into MaintenanceController.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32014/#review76327 --- Ship it! Master (ee1a13a) is green with this patch. ./build-suppo

Review Request 32014: Adding more logging into MaintenanceController.

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

Re: Review Request 32012: Improve TaskDetector performance by memoizing regular expressions

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32012/#review76324 --- Master (ee1a13a) is red with this patch. ./build-support/jenkins/b

Re: Review Request 32012: Improve TaskDetector performance by memoizing regular expressions

> On March 13, 2015, 1:10 a.m., Brian Wickman wrote: > > @ReviewBot retry If you believe you have discovered a flaky test (which i assume you're asking the bot to try again), can you please file a ticket? - Bill --- This is an automati

Re: Review Request 32012: Improve TaskDetector performance by memoizing regular expressions

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32012/#review76322 --- @ReviewBot retry - Brian Wickman On March 13, 2015, 12:47 a.m., B

Re: Review Request 32012: Improve TaskDetector performance by memoizing regular expressions

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32012/#review76320 --- Master (ee1a13a) is red with this patch. ./build-support/jenkins/b

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/#review76319 --- whoops, forgot that I added os.path.exists filtering -- will update

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/#review76318 --- Master (ee1a13a) is red with this patch. ./build-support/jenkins/b

Re: Review Request 31966: Add client support for including messages when changing update state.

> On March 12, 2015, 4:23 p.m., Maxim Khutornenko wrote: > > src/main/python/apache/aurora/client/cli/update.py, lines 123-127 > > > > > > Mind sorting them alphabetically? > > Bill Farner wrote: > Done. Spoke to

Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/ --- Review request for Aurora, Joe Smith and Zameer Manji. Repository: aurora Des

Review Request 32012: Improve TaskDetector performance by memoizing regular expressions

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32012/ --- Review request for Aurora, Joe Smith and Zameer Manji. Repository: aurora Des

Re: Review Request 31966: Add client support for including messages when changing update state.

> On March 12, 2015, 4:23 p.m., Maxim Khutornenko wrote: > > src/main/python/apache/aurora/client/api/__init__.py, line 157 > > > > > > Spacing seems off here and below. Fixed. > On March 12, 2015, 4:23 p.m., Maxim

Re: Review Request 31913: Added 'none' host maintenance grouping function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31913/#review76307 --- Thanks, it's now on master. - Maxim Khutornenko On March 12, 2015

Re: Review Request 31913: Added 'none' host maintenance grouping function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31913/#review76305 --- Ship it! +10 :) - Joe Smith On March 12, 2015, 4:13 p.m., David

Re: Review Request 31913: Added 'none' host maintenance grouping function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31913/#review76304 --- Ship it! Thanks! - Maxim Khutornenko On March 12, 2015, 11:13 p.

Re: Review Request 31913: Added 'none' host maintenance grouping function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31913/#review76303 --- Ship it! Master (f62e0f1) is green with this patch. ./build-suppo

Re: Review Request 31913: Added 'none' host maintenance grouping function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31913/ --- (Updated March 12, 2015, 11:13 p.m.) Review request for Aurora and Maxim Khutor

Re: Review Request 31913: Added 'none' host maintenance grouping function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31913/#review76298 --- src/test/python/apache/aurora/client/test_base.py

Re: Review Request 31913: Added 'none' host maintenance grouping function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31913/#review76283 --- src/test/python/apache/aurora/client/test_base.py

Re: Review Request 31966: Add client support for including messages when changing update state.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31966/#review76257 --- Ship it! Ship It! - Zameer Manji On March 11, 2015, 6:27 p.m., B

Re: Review Request 31966: Add client support for including messages when changing update state.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31966/#review76243 --- Ship it! src/main/python/apache/aurora/client/api/__init__.py

Re: Review Request 31820: Support HTTP Basic auth and shiro.ini configuration

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31820/#review75926 --- Ship it! lgtm! src/main/java/org/apache/aurora/scheduler/http/api

Re: Review Request 31821: Making preemptor asynchronous. Part 1 - extracting slot finder.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31821/ --- (Updated March 10, 2015, 5:26 p.m.) Review request for Aurora, Bill Farner and