Re: Review Request 18602: Make http signaler timeout secs configurable via environment variable.

2014-02-28 Thread Jim Youll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18602/#review35810 --- would it be very hard to make that a command line / config file var

Re: Review Request 18602: Make http signaler timeout secs configurable via environment variable.

2014-02-28 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18602/#review35816 --- Ship it! Ship It! - Brian Wickman On Feb. 28, 2014, 2:37 a.m.,

Re: Review Request 18602: Make http signaler timeout secs configurable via environment variable.

2014-02-28 Thread Kevin Sweeney
On Feb. 28, 2014, 10:37 a.m., Jim Youll wrote: would it be very hard to make that a command line / config file var rather than an env var? Then it would be set just like all other configurable params in a config file or with a command line override, and would be more sustainable. I

Re: Review Request 18602: Make http signaler timeout secs configurable via environment variable.

2014-02-28 Thread Jim Youll
On Feb. 28, 2014, 6:37 p.m., Jim Youll wrote: would it be very hard to make that a command line / config file var rather than an env var? Then it would be set just like all other configurable params in a config file or with a command line override, and would be more sustainable.

Review Request 18602: Make http signaler timeout secs configurable via environment variable.

2014-02-27 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18602/ --- Review request for Aurora, Mark Chu-Carroll, Maxim Khutornenko, and Brian

Re: Review Request 18602: Make http signaler timeout secs configurable via environment variable.

2014-02-27 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18602/ --- (Updated Feb. 27, 2014, 5:45 p.m.) Review request for Aurora, Mark

Re: Review Request 18602: Make http signaler timeout secs configurable via environment variable.

2014-02-27 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18602/#review35748 --- src/main/python/apache/aurora/common/http_signaler.py

Re: Review Request 18602: Make http signaler timeout secs configurable via environment variable.

2014-02-27 Thread Kevin Sweeney
On Feb. 27, 2014, 6:09 p.m., Maxim Khutornenko wrote: src/main/python/apache/aurora/common/http_signaler.py, line 52 https://reviews.apache.org/r/18602/diff/3/?file=506638#file506638line52 Isn't it still the same problem here? Attempt to use %f for a string? I'll go ahead and

Re: Review Request 18602: Make http signaler timeout secs configurable via environment variable.

2014-02-27 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18602/ --- (Updated Feb. 27, 2014, 6:37 p.m.) Review request for Aurora, Mark

Re: Review Request 18602: Make http signaler timeout secs configurable via environment variable.

2014-02-27 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18602/#review35755 --- Ship it! Ship It! - Maxim Khutornenko On Feb. 28, 2014, 2:37