> On April 19, 2017, 6:31 p.m., Zameer Manji wrote: > > "Without this tasks will wait for watch_secs after the task moves to > > RUNNING extending the update time." > > > > I thought this was intentional. One can use this parameter to naturally > > slow down update speed. I was under the impression that `watch_secs` should > > be set to `0` if you want purely health check driven updates.
Race contion :). I have the same understanding as you. - Stephan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58524/#review172357 ----------------------------------------------------------- On April 19, 2017, 6:18 p.m., Santhosh Kumar Shanmugham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58524/ > ----------------------------------------------------------- > > (Updated April 19, 2017, 6:18 p.m.) > > > Review request for Aurora, Joshua Cohen, Stephan Erb, and Zameer Manji. > > > Repository: aurora > > > Description > ------- > > Tasks are gated on health-checks before they transistion into `RUNNING` > and hence it is possible for a task to stay in `STARTING` during the > watch duration of an instance's update. So include `STARTING` into the > possible states for a task when watching it after an update. Without > this tasks will wait for watch_secs after the task moves to RUNNING > extending the update time. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/updater/InstanceUpdater.java > c129896d8cd54abd2634e2a339c27921042b0162 > src/main/java/org/apache/aurora/scheduler/updater/StateEvaluator.java > c95943d242dc2f539778bdc9e071f342005e8de3 > src/test/java/org/apache/aurora/scheduler/updater/InstanceUpdaterTest.java > df1f8394b824dbb7b2745fcccdab5adaafdf6e6c > src/test/java/org/apache/aurora/scheduler/updater/OneWayJobUpdaterTest.java > 32a8c89d3e71395696fc613da96b871330891c42 > > > Diff: https://reviews.apache.org/r/58524/diff/2/ > > > Testing > ------- > > ./build-support/jenkins/build.sh > ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh > > > Thanks, > > Santhosh Kumar Shanmugham > >
