-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59733/
-----------------------------------------------------------
(Updated June 6, 2017, 5:47 p.m.)
Review request for Aurora, David McLaughlin, Santhosh Kumar Shanmugham, Stephan
Erb, and Zameer Manji.
Changes
-------
Added default for `escalation_wait_secs` in `http_lifecycle.py` if the field is
not present in the config.
Default is 5 seconds (the current value)
Added the ability to specify maximum stop timeout for the executor through the
flag `stop_timeout_in_secs`.
Default is 120 seconds (the current value).
Added test to ensure the new executor flag is used when creating the executor,
and that the stop_timeout is
used when killing a task.
Added/updated documentation.
Bugs: AURORA-1931
https://issues.apache.org/jira/browse/AURORA-1931
Repository: aurora
Description
-------
We have some services that require more than the current 10 seconds given to
gracefully shutdown (they need to close resources, finish requests, etc).
We would like to be able to configure the amount of time we wait between each
stage of the graceful shutdown sequence. See this
[proposal](https://docs.google.com/document/d/1Sl-KWNyt1j0nIndinqfJsH3pkUY5IYXfGWyLHU2wacs/edit?usp=sharing)
for a more in-depth
analysis.
Diffs (updated)
-----
RELEASE-NOTES.md c73643d669363a56e530c2d8c79d6ea06cc9415b
docs/reference/task-lifecycle.md cf1b6794dca92756cee4a54f00f33fd3b1a481ea
src/main/python/apache/aurora/config/schema/base.py
b2692a648645a195a24491e4978fb833c6c20be8
src/main/python/apache/aurora/executor/aurora_executor.py
81461cb49ac223f3bdfa59e8c59e150a07771dea
src/main/python/apache/aurora/executor/bin/thermos_executor_main.py
c6c08983fb2d204afedd419798135d667169bce4
src/main/python/apache/aurora/executor/http_lifecycle.py
9280bf29da9bda1691adbf3a4c34c4f3d4900517
src/test/python/apache/aurora/client/cli/test_inspect.py
4a23c5984c2d093e2f53e93aec71418f84b65928
src/test/python/apache/aurora/executor/bin/test_thermos_executor_entry_point.py
38deae69dce7d88dab87f70e0128e0b86091fdef
src/test/python/apache/aurora/executor/test_http_lifecycle.py
a967e3410a4d2dc2e1721f505a4d76da9209d177
src/test/python/apache/aurora/executor/test_thermos_executor.py
e628ccd21f0588d0c76001b8f336ffa844c3ec35
src/test/python/apache/aurora/executor/test_thermos_task_runner.py
1b92667bceabc8ea1540122477a51cb58ea2ae36
Diff: https://reviews.apache.org/r/59733/diff/2/
Changes: https://reviews.apache.org/r/59733/diff/1-2/
Testing
-------
Ran unit and integration tests.
Created and killed jobs with varying wait_escalation_secs values on the Vagrant
devcluster.
Thanks,
Jordan Ly
