[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16615042#comment-16615042
]
Hudson commented on YARN-8706:
--
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #14958 (See
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16614226#comment-16614226
]
Chandni Singh commented on YARN-8706:
-
[~eyang] Thanks for looking at it. Yes I should not have
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16614215#comment-16614215
]
Eric Yang commented on YARN-8706:
-
[~csingh] +1 for addendum patch 001. It would be nice to not replace
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16607866#comment-16607866
]
Chandni Singh commented on YARN-8706:
-
Thanks [~eyang], [~ebadger], and [~shaneku...@gmail.com]
>
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16607841#comment-16607841
]
Hudson commented on YARN-8706:
--
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #14907 (See
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16607809#comment-16607809
]
Eric Yang commented on YARN-8706:
-
+1 for patch 004. I will commit shortly.
> DelayedProcessKiller is
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16604930#comment-16604930
]
Hadoop QA commented on YARN-8706:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16604800#comment-16604800
]
Chandni Singh commented on YARN-8706:
-
I had deprecated {{DockerStopCommand}} because of which there
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16604796#comment-16604796
]
Hadoop QA commented on YARN-8706:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16604745#comment-16604745
]
Eric Badger commented on YARN-8706:
---
Thanks for the update, [~csingh]. +1 (non-binding) pending Hadoop
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16603773#comment-16603773
]
Hadoop QA commented on YARN-8706:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16603613#comment-16603613
]
Eric Badger commented on YARN-8706:
---
{quote}
Either we implement native code that accepts name of the
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16603480#comment-16603480
]
Chandni Singh commented on YARN-8706:
-
{quote}Additionally, for non-privileged containers, we don't
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16599280#comment-16599280
]
Chandni Singh commented on YARN-8706:
-
{quote}
Additionally, for non-privileged containers, we don't
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16599088#comment-16599088
]
Eric Badger commented on YARN-8706:
---
bq. In DockerLinuxContainerRuntime, reapContainer() calls
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16598976#comment-16598976
]
Chandni Singh commented on YARN-8706:
-
Thanks for the review [~ebadger]
{quote}Both reapContainer()
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16598932#comment-16598932
]
Eric Badger commented on YARN-8706:
---
Thanks for the patch, [~csingh]!
Both {{reapContainer()}} and
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596887#comment-16596887
]
Eric Badger commented on YARN-8706:
---
I think the default sleep delay between STOPSIGNAL and SIGKILL is
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596883#comment-16596883
]
Eric Yang commented on YARN-8706:
-
{quote}Why is this specific to docker containers? Other types of
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596637#comment-16596637
]
Chandni Singh commented on YARN-8706:
-
{quote}I am not entirely sure about globally identical killing
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595851#comment-16595851
]
Eric Yang commented on YARN-8706:
-
{quote}But then we have redundant configs for no reason. And we would
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595623#comment-16595623
]
Eric Badger commented on YARN-8706:
---
bq. If this is setup properly, code only needs to ensure
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595612#comment-16595612
]
Eric Yang commented on YARN-8706:
-
[~csingh] We can arrange it as NM_SLEEP_DELAY_BEFORE_SIGKILL_MS to be
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595603#comment-16595603
]
Chandni Singh commented on YARN-8706:
-
{quote}
Docker stop already covers sending the custom signal,
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595585#comment-16595585
]
Eric Yang commented on YARN-8706:
-
[~ebadger] suggested solution of discover STOPSIGNAL and perform this
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595562#comment-16595562
]
Shane Kumpf commented on YARN-8706:
---
Seems like a reasonable solution to me. {{docker stop}} has been a
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595535#comment-16595535
]
Eric Badger commented on YARN-8706:
---
bq. I can work on it, if there aren't any concerns?
Sounds good to
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595515#comment-16595515
]
Chandni Singh commented on YARN-8706:
-
Thanks [~shaneku...@gmail.com] and [~ebadger] for the
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595507#comment-16595507
]
Shane Kumpf commented on YARN-8706:
---
Thanks for reporting this, [~csingh]. I know several of us
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595410#comment-16595410
]
Chandni Singh commented on YARN-8706:
-
{quote}
Really it would be better if we didn't send the kill
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595170#comment-16595170
]
Eric Badger commented on YARN-8706:
---
Really it would be better if we didn't send the kill from docker
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16594348#comment-16594348
]
Chandni Singh commented on YARN-8706:
-
I can see 2 ways for addressing this:
Approach 1:
1. Deprecate
[
https://issues.apache.org/jira/browse/YARN-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16592013#comment-16592013
]
Chandni Singh commented on YARN-8706:
-
Seems that {{YarnConfiguration.NM_DOCKER_STOP_GRACE_PERIOD}} is
33 matches
Mail list logo