You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by gi...@apache.org on 2018/05/07 21:17:44 UTC
[1/2] mesos git commit: Added MESOS-8876 to 1.5.1 CHANGELOG.
Repository: mesos
Updated Branches:
refs/heads/1.5.x 9343f4200 -> 1b813dcf0
Added MESOS-8876 to 1.5.1 CHANGELOG.
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/1b813dcf
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/1b813dcf
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/1b813dcf
Branch: refs/heads/1.5.x
Commit: 1b813dcf0c0bd31e46aa769c8fd86f11b339dad8
Parents: ecf80b9
Author: Gilbert Song <so...@gmail.com>
Authored: Mon May 7 14:12:45 2018 -0700
Committer: Gilbert Song <so...@gmail.com>
Committed: Mon May 7 14:15:47 2018 -0700
----------------------------------------------------------------------
CHANGELOG | 1 +
1 file changed, 1 insertion(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/1b813dcf/CHANGELOG
----------------------------------------------------------------------
diff --git a/CHANGELOG b/CHANGELOG
index ea3fce4..dbcd6a5 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -32,6 +32,7 @@ Release Notes - Mesos - Version 1.5.1 (WIP)
* [MESOS-8651] - Potential memory leaks in the `volume/sandbox_path` isolator.
* [MESOS-8742] - Agent resource provider config API calls should be idempotent.
* [MESOS-8787] - RP-related API should be experimental.
+ * [MESOS-8876] - Normal exit of Docker container using rexray volume results in TASK_FAILED.
Release Notes - Mesos - Version 1.5.0
[2/2] mesos git commit: Increased the timeout for waiting for
`reaped` to be invoked.
Posted by gi...@apache.org.
Increased the timeout for waiting for `reaped` to be invoked.
Previously after the container process is reaped by the Docker
executor, we will wait 3 seconds for `reaped` to be invoked.
However in some cases (e.g., launch a Docker container to use
an external rexray volume), there will be more than 3 seconds
after the container process exits and before the `docker run`
command returns (i.e., `reaped` invoked). So in this patch,
the timeout is increased to 60 seconds.
Review: https://reviews.apache.org/r/66947/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/ecf80b90
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/ecf80b90
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/ecf80b90
Branch: refs/heads/1.5.x
Commit: ecf80b90b9027acce17beecb9a725a90fa8325fa
Parents: 9343f42
Author: Qian Zhang <zh...@gmail.com>
Authored: Mon May 7 13:48:39 2018 -0700
Committer: Gilbert Song <so...@gmail.com>
Committed: Mon May 7 14:15:47 2018 -0700
----------------------------------------------------------------------
src/docker/executor.cpp | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/ecf80b90/src/docker/executor.cpp
----------------------------------------------------------------------
diff --git a/src/docker/executor.cpp b/src/docker/executor.cpp
index de462d9..df27083 100644
--- a/src/docker/executor.cpp
+++ b/src/docker/executor.cpp
@@ -330,11 +330,11 @@ public:
// lambda; ideally when the Docker issue mentioned above does
// not occur, `reaped` will be invoked (i.e., the `docker run`
// command returns) to get the actual exit status of the
- // container, so here we wait a few seconds for `reaped` to be
+ // container, so here we wait 60 seconds for `reaped` to be
// invoked. If `reaped` is not invoked within the timeout, that
// means we hit that Docker issue.
delay(
- Seconds(3),
+ Seconds(60),
self(),
&Self::reapedContainer,
container.pid.get());
@@ -343,10 +343,10 @@ public:
}));
} else {
// This is the case that the container process has already exited,
- // Similar to the above case, let's wait a few seconds for `reaped`
+ // Similar to the above case, let's wait 60 seconds for `reaped`
// to be invoked.
delay(
- Seconds(3),
+ Seconds(60),
self(),
&Self::reapedContainer,
None());