Re: Review Request 35527: Making observer polling interval configurable.

2015-06-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35527/#review88127 --- Ship it! Master (b09adc6) is green with this patch.

Re: Review Request 35535: Use Shiro identity if available in audit logs.

2015-06-16 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35535/#review88139 --- Ship it! Ship It! - Maxim Khutornenko On June 16, 2015, 9:54

Re: Review Request 35535: Use Shiro identity if available in audit logs.

2015-06-16 Thread Maxim Khutornenko
On June 16, 2015, 9:54 p.m., Joshua Cohen wrote: src/main/java/org/apache/aurora/auth/UnsecureAuthModule.java, line 34 https://reviews.apache.org/r/35535/diff/1/?file=986082#file986082line34 Should this move to UnsecureSessionContext? +1 - Maxim

Re: Review Request 35535: Use Shiro identity if available in audit logs.

2015-06-16 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35535/#review88136 --- Ship it!

Re: Review Request 35527: Making observer polling interval configurable.

2015-06-16 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35527/#review88130 ---

Review Request 35535: Use Shiro identity if available in audit logs.

2015-06-16 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35535/ --- Review request for Aurora, Joshua Cohen and Maxim Khutornenko. Bugs:

Re: Review Request 35527: Making observer polling interval configurable.

2015-06-16 Thread Maxim Khutornenko
On June 16, 2015, 9:33 p.m., Joshua Cohen wrote: src/test/python/apache/aurora/tools/test_thermos_observer_entry_point.py, lines 27-30 https://reviews.apache.org/r/35527/diff/1/?file=985959#file985959line27 There's a way to set this up so that only these expected properties are

Re: Review Request 35527: Making observer polling interval configurable.

2015-06-16 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35527/ --- (Updated June 16, 2015, 10:18 p.m.) Review request for Aurora, Joshua Cohen

Re: Review Request 35527: Making observer polling interval configurable.

2015-06-16 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35527/#review88144 --- Ship it!

Re: Review Request 35535: Use Shiro identity if available in audit logs.

2015-06-16 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35535/ --- (Updated June 16, 2015, 3:06 p.m.) Review request for Aurora, Joshua Cohen and

Review Request 35527: Making observer polling interval configurable.

2015-06-16 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35527/ --- Review request for Aurora, Joshua Cohen and Brian Wickman. Bugs: AURORA-1351

Re: Review Request 35527: Making observer polling interval configurable.

2015-06-16 Thread Maxim Khutornenko
On June 16, 2015, 10:25 p.m., Brian Wickman wrote: src/test/python/apache/thermos/observer/test_task_observer.py, lines 37-38 https://reviews.apache.org/r/35527/diff/2/?file=986119#file986119line37 while ugly, should probably change pass to time.sleep(0.01) so that MainThread

Re: Review Request 35527: Making observer polling interval configurable.

2015-06-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35527/#review88153 --- Ship it! Master (ea2c9ad) is green with this patch.

Re: Review Request 35527: Making observer polling interval configurable.

2015-06-16 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35527/#review88154 --- Ship it! Ship It! - Joshua Cohen On June 16, 2015, 10:18 p.m.,

Review Request 35498: Compute SLA stats for non-prod jobs

2015-06-16 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35498/ --- Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. Repository:

Re: Review Request 35498: Compute SLA stats for non-prod jobs

2015-06-16 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35498/ --- (Updated June 16, 2015, 11:07 a.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 35498: Compute SLA stats for non-prod jobs

2015-06-16 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35498/#review88042 --- src/main/java/org/apache/aurora/scheduler/sla/SlaModule.java (line

Re: Review Request 32973: Allow config of the /quitquitquit and /abortabortabort endpoints.

2015-06-16 Thread Brian Brazil
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32973/ --- (Updated June 16, 2015, 9:12 a.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 35498: Compute SLA stats for non-prod jobs

2015-06-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35498/#review88044 --- Ship it! Master (b09adc6) is green with this patch.

Re: Review Request 32973: Allow config of the /quitquitquit and /abortabortabort endpoints.

2015-06-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32973/#review88053 --- Ship it! Master (b09adc6) is green with this patch.