[ 
https://issues.apache.org/jira/browse/MESOS-999?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14258777#comment-14258777
 ] 

Nishant Suneja commented on MESOS-999:
--------------------------------------

As for test plan, I would ideally want to somehow delay the launch of the 
executor process by >= registration_timeout. This would ensure that the current 
code will lead to destruction of the container. 
But, with this fix, we should not see the container getting destroyed, because 
our timer starts only after the successful launch of the container.

I have the fix ready. Have to write a test case now. 

> Slave should wait() and start executor registration timeout after launch 
> -------------------------------------------------------------------------
>
>                 Key: MESOS-999
>                 URL: https://issues.apache.org/jira/browse/MESOS-999
>             Project: Mesos
>          Issue Type: Bug
>          Components: isolation
>    Affects Versions: 0.18.0
>            Reporter: Ian Downes
>            Assignee: Nishant Suneja
>            Priority: Minor
>
> The current code will start launch a container and wait on it before the 
> launch is complete. We should do this only after the container has 
> successfully launched. Likewise for the executor registration timeout.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to