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 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 ---

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!

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.,