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

Eric Badger commented on YARN-7848:
-----------------------------------

We talked about the use of {{-f}} in our most recent docker meeting and all 
came to agreement that this was appropriate for all removals. [~eyang]'s 
explanation above does a good job summarizing why. 

I do have one ask for the patch though. The current patch changes 
{{get_docker_rm_command}} to always give it the {{-f}} flag. I think it would 
be beneficial to have this as an option passed into the function so that we 
don't accidentally reuse this function in the future and unknowingly use {{rm 
-f}} when we aren't expecting to



> Force removal of docker containers that do not get removed on first try
> -----------------------------------------------------------------------
>
>                 Key: YARN-7848
>                 URL: https://issues.apache.org/jira/browse/YARN-7848
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Eric Badger
>            Assignee: Eric Yang
>            Priority: Major
>              Labels: Docker
>         Attachments: YARN-7848.001.patch
>
>
> After the addition of YARN-5366, containers will get removed after a certain 
> debug delay. However, this is a one-time effort. If the removal fails for 
> whatever reason, the container will persist. We need to add a mechanism for a 
> forced removal of those containers.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to