> On June 1, 2017, 11:53 p.m., David McLaughlin wrote: > > src/main/python/apache/aurora/executor/http_lifecycle.py > > Lines 37 (patched) > > <https://reviews.apache.org/r/59733/diff/1/?file=1739103#file1739103line39> > > > > Will this break for tasks that were deployed before the client was > > updated? Should we also inject a default here?
Good point! I definitely need to add a default to avoid this case. - Jordan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59733/#review176705 ----------------------------------------------------------- On June 1, 2017, 11:48 p.m., Jordan Ly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59733/ > ----------------------------------------------------------- > > (Updated June 1, 2017, 11:48 p.m.) > > > Review request for Aurora, David McLaughlin, Santhosh Kumar Shanmugham, > Stephan Erb, and Zameer Manji. > > > 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 > ----- > > 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/http_lifecycle.py > 9280bf29da9bda1691adbf3a4c34c4f3d4900517 > src/test/python/apache/aurora/client/cli/test_inspect.py > 4a23c5984c2d093e2f53e93aec71418f84b65928 > src/test/python/apache/aurora/executor/test_http_lifecycle.py > a967e3410a4d2dc2e1721f505a4d76da9209d177 > src/test/python/apache/aurora/executor/test_thermos_task_runner.py > 1b92667bceabc8ea1540122477a51cb58ea2ae36 > > > Diff: https://reviews.apache.org/r/59733/diff/1/ > > > Testing > ------- > > Ran unit and integration tests. > > Created and killed jobs with varying wait_escalation_secs values on the > Vagrant devcluster. > > > Thanks, > > Jordan Ly > >