You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by kl...@apache.org on 2018/10/26 23:49:40 UTC
[mesos] branch master updated: Fixed bug in 'execute.cpp' with
tty-based tasks and no 'containerInfo'.
This is an automated email from the ASF dual-hosted git repository.
klueska pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git
The following commit(s) were added to refs/heads/master by this push:
new 4fa080a Fixed bug in 'execute.cpp' with tty-based tasks and no 'containerInfo'.
4fa080a is described below
commit 4fa080a7eefd47697b9fab934d19c73f092c78f8
Author: Kevin Klues <kl...@gmail.com>
AuthorDate: Fri Oct 26 16:49:32 2018 -0700
Fixed bug in 'execute.cpp' with tty-based tasks and no 'containerInfo'.
Previously, we could only launch tasks using the '--tty' flag if they
had a backing docker image (or some other combination of other flags set
that would cause the task to have a 'containerInfo' created for it).
This commit makes sure that if '--tty' is passed, that a containerInfo
gets created and its TTYInfo field gets populated.
Review: https://reviews.apache.org/r/69103/
---
src/cli/execute.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/cli/execute.cpp b/src/cli/execute.cpp
index eb5c564..c436332 100644
--- a/src/cli/execute.cpp
+++ b/src/cli/execute.cpp
@@ -810,7 +810,8 @@ static Result<ContainerInfo> getContainerInfo(
// Mesos containerizer supports 'appc' and 'docker' images.
if (containerizer == "mesos") {
- if (appcImage.isNone() &&
+ if (!tty &&
+ appcImage.isNone() &&
dockerImage.isNone() &&
effective_capabilities.isNone() &&
bounding_capabilities.isNone() &&