[
https://issues.apache.org/jira/browse/MESOS-1718?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15020884#comment-15020884
]
Klaus Ma edited comment on MESOS-1718 at 11/22/15 2:02 PM:
-----------------------------------------------------------
Thanks for your input :). Also agree to ask master to setup TaskInfo & its
ExecutorInfo; the info will send to master when register & re-register.
For executor's resource, if we used additional resources, it's hard to decide
which kind of resources should be used: revocable or non-revocable; so I'm
thinking to move some of resources from task to executor. As a result, command
line executor's resource will be consistent with task's, and will not consume
more resources.
was (Author: klaus1982):
Thanks for your input :). Also agree to ask master to setup TaskInfo & its
ExecutorInfo; the info will send to master when register & re-register.
> Command executor can overcommit the slave.
> ------------------------------------------
>
> Key: MESOS-1718
> URL: https://issues.apache.org/jira/browse/MESOS-1718
> Project: Mesos
> Issue Type: Bug
> Components: slave
> Reporter: Benjamin Mahler
> Assignee: Ian Downes
>
> Currently we give a small amount of resources to the command executor, in
> addition to resources used by the command task:
> https://github.com/apache/mesos/blob/0.20.0-rc1/src/slave/slave.cpp#L2448
> {code: title=}
> ExecutorInfo Slave::getExecutorInfo(
> const FrameworkID& frameworkId,
> const TaskInfo& task)
> {
> ...
> // Add an allowance for the command executor. This does lead to a
> // small overcommit of resources.
> executor.mutable_resources()->MergeFrom(
> Resources::parse(
> "cpus:" + stringify(DEFAULT_EXECUTOR_CPUS) + ";" +
> "mem:" + stringify(DEFAULT_EXECUTOR_MEM.megabytes())).get());
> ...
> }
> {code}
> This leads to an overcommit of the slave. Ideally, for command tasks we can
> "transfer" all of the task resources to the executor at the slave / isolation
> level.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
