[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16818377#comment-16818377 ] Hudson commented on YARN-7848: -- FAILURE: Integrated in Jenkins build Hadoop-trunk-Commit #164

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16815709#comment-16815709 ] Eric Yang commented on YARN-7848: - [~Jim_Brennan] Thank you for tips. [~ebadger] can you

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16815434#comment-16815434 ] Jim Brennan commented on YARN-7848: --- Thanks for updating [~eyang]!  lgtm.   I am +1 on p

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16814928#comment-16814928 ] Hadoop QA commented on YARN-7848: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16814900#comment-16814900 ] Eric Yang commented on YARN-7848: - [~Jim_Brennan] Thanks for the feedback.  Patch 004 clea

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16814627#comment-16814627 ] Jim Brennan commented on YARN-7848: --- Thanks [~eyang] for the update! Some comments: In

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16813627#comment-16813627 ] Eric Yang commented on YARN-7848: - [~ebadger] [~Jim_Brennan] The failed unit test is not r

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16812972#comment-16812972 ] Hadoop QA commented on YARN-7848: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote ||

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16812834#comment-16812834 ] Eric Yang commented on YARN-7848: - [~ebadger] [~Jim_Brennan] Thanks for the extra review t

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16812755#comment-16812755 ] Eric Badger commented on YARN-7848: --- Looked at the code with [~Jim_Brennan] and he's rig

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16812744#comment-16812744 ] Jim Brennan commented on YARN-7848: --- [~eyang], [~ebadger], it looks to me like the code

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16812730#comment-16812730 ] Eric Badger commented on YARN-7848: --- bq. Changing this can have dicey outcome and not ba

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16812722#comment-16812722 ] Eric Yang commented on YARN-7848: - {quote}Thanks for the update, Eric Yang! On the java si

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16812528#comment-16812528 ] Eric Badger commented on YARN-7848: --- Thanks for the update, [~eyang]! On the java side,

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16811375#comment-16811375 ] Hadoop QA commented on YARN-7848: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote ||

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16811188#comment-16811188 ] Eric Yang commented on YARN-7848: - Patch 002 added the force option to cmd file, and invok

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ 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 recen

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16802833#comment-16802833 ] Jim Brennan commented on YARN-7848: --- I think I agree with [~eyang] on the use of {{-f}}.

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16802178#comment-16802178 ] Eric Yang commented on YARN-7848: - [~ebadger] Docker rm will use stop signal then remove t

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16801943#comment-16801943 ] Eric Badger commented on YARN-7848: --- Thanks for the patch, [~eyang]! Correct me if I'm

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16801892#comment-16801892 ] Eric Yang commented on YARN-7848: - The failed unit test is not related to this patch. > F

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16801270#comment-16801270 ] Hadoop QA commented on YARN-7848: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote ||

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16801229#comment-16801229 ] Eric Yang commented on YARN-7848: - Back track to original discussion that if container stu

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16798427#comment-16798427 ] Eric Yang commented on YARN-7848: - [~ebadger] {quote}Additionally, how would this work wit

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16798387#comment-16798387 ] Eric Badger commented on YARN-7848: --- I don't like this idea. This is the Nodemanager com

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16797677#comment-16797677 ] Eric Yang commented on YARN-7848: - There are many scenarios that containers may not create

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16751820#comment-16751820 ] Eric Yang commented on YARN-7848: - [~ebadger] YARN-9074 does not force removal of containe

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16750499#comment-16750499 ] Eric Badger commented on YARN-7848: --- Does YARN-9074 force the removal of the containers?

[jira] [Commented] (YARN-7848) Force removal of docker containers that do not get removed on first try

[ https://issues.apache.org/jira/browse/YARN-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16750385#comment-16750385 ] Eric Yang commented on YARN-7848: - Do we still need this JIRA if YARN-9074 moved the docke