mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chun-Hung Hsiao <chhs...@mesosphere.io>
Subject Re: Review Request 63555: Publish resource provider resources before container launch or update.
Date Thu, 07 Dec 2017 09:33:20 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63555/
-----------------------------------------------------------

(Updated Dec. 7, 2017, 9:33 a.m.)


Review request for mesos, Gilbert Song, Jie Yu, Joseph Wu, and Jan Schlicht.


Changes
-------

Addressed Jie's comments.


Bugs: MESOS-7550
    https://issues.apache.org/jira/browse/MESOS-7550


Repository: mesos


Description
-------

`Slave::publishAllocatedResources()` will compute the total allocated
resources for all currently running executor containers, and takes an
`extra` argument for resources that will be used by the executor that
is about to launch, then sums them up and asks the resource provider
manager to publish the resources.


Diffs (updated)
-----

  src/internal/devolve.hpp 17ab76e32dae1f4f57de43b323c8ffd67bb17ba8 
  src/internal/devolve.cpp 289c6e37415f4a289fe34cd303329b095f9348cd 
  src/internal/evolve.hpp d796f328de3c614e510b75021c405563d413c18e 
  src/internal/evolve.cpp f46f8641cae0f030103d82e7694ddf60300fdd63 
  src/resource_provider/manager.cpp 8d8b2f1396230d6edba590b8b7e9b4ca51366efe 
  src/slave/slave.hpp fc762fb1e8aeb57b0d7ad551d960ec4be06e8e33 
  src/slave/slave.cpp dd830d93f8a08720231e3d4a6421f7a058ba6093 
  src/tests/mesos.hpp 3a9b1fbd4301c6cccbc770329ff71ed8ff7e86a2 
  src/tests/slave_tests.cpp 25cfd4730c3d77d1886745204b13ca3cb140620c 


Diff: https://reviews.apache.org/r/63555/diff/3/

Changes: https://reviews.apache.org/r/63555/diff/2-3/


Testing (updated)
-------


Thanks,

Chun-Hung Hsiao


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