mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Klaus Ma" <>
Subject Re: Review Request 40759: [WIP] Command executor can overcommit the slave
Date Tue, 08 Dec 2015 04:03:10 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Dec. 8, 2015, 12:03 p.m.)

Review request for Ben Mahler, Ian Downes and Vinod Kone.


`make check` passed without `sudo`

Bugs: MESOS-1718

Repository: mesos


Currently we give a small amount of resources to the command executor, in addition to resources
used by the command task:

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.
          "cpus:" + stringify(DEFAULT_EXECUTOR_CPUS) + ";" +
          "mem:" + stringify(DEFAULT_EXECUTOR_MEM.megabytes())).get());

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.

Diffs (updated)

  include/mesos/mesos.proto 3a10c8a 
  src/docker/executor.cpp 4042cec 
  src/launcher/executor.cpp f90ea01 
  src/master/constants.hpp cc38dfc 
  src/master/master.hpp 4683fa5 
  src/master/master.cpp 953fa4f 
  src/slave/constants.hpp bcbb140 
  src/slave/slave.hpp b7586ce 
  src/slave/slave.cpp 9bd86e1 
  src/tests/containerizer/docker_containerizer_tests.cpp 3f199e6 
  src/tests/master_tests.cpp 865fa4a 
  src/tests/monitor_tests.cpp a848d14 
  src/tests/reservation_endpoints_tests.cpp c3833d0 
  src/tests/slave_recovery_tests.cpp c0e4ff7 
  src/tests/slave_tests.cpp 4975bea 



only `make` passwd for now.


Klaus Ma

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message