Re: Review Request 61530: Enabled retries for `killTasks` in docker executor.

2017-08-10 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61530/#review182624 --- Ship it! Great job, great testing. Thanks a lot! - Alexander R

Re: Review Request 61530: Enabled retries for `killTasks` in docker executor.

2017-08-10 Thread Andrei Budnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61530/ --- (Updated Aug. 10, 2017, 4:14 p.m.) Review request for mesos and Alexander Rukle

Re: Review Request 61530: Enabled retries for `killTasks` in docker executor.

2017-08-10 Thread Alexander Rukletsov
> On Aug. 10, 2017, 12:07 p.m., Alexander Rukletsov wrote: > > src/docker/executor.cpp > > Lines 410-415 (original), 416-421 (patched) > > > > > > Let's add a comment explaining why are we doing retry / unblock on >

Re: Review Request 61530: Enabled retries for `killTasks` in docker executor.

2017-08-10 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61530/#review182573 --- src/docker/executor.cpp Lines 31 (patched)

Review Request 61530: Enabled retries for `killTasks` in docker executor.

2017-08-09 Thread Andrei Budnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61530/ --- Review request for mesos and Alexander Rukletsov. Bugs: MESOS-6743 https://