mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From James Peach <jpe...@apache.org>
Subject Re: Review Request 62644: Added resources to the TaskStatus message.
Date Tue, 10 Oct 2017 00:08:43 GMT

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

(Updated Oct. 10, 2017, 12:08 a.m.)


Review request for mesos, Jie Yu and Qian Zhang.


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


Repository: mesos


Description (updated)
-------

Added a TaskResourcesLimitation field to the `TaskStatus`
message to convey specific information about a resource
limit that has been violated by a container.

This field propagates the resources from the `ContainerLimitation`
message sent by isolators on the agent, and is populated for the
following reasons:

  * `REASON_CONTAINER_LIMITATION`
  * `REASON_CONTAINER_LIMITATION_DISK`
  * `REASON_CONTAINER_LIMITATION_MEMORY`


Diffs (updated)
-----

  include/mesos/mesos.proto 1bfcc5bb9091075c6fa1bc36e2cd23afe2e0e0ec 
  include/mesos/v1/mesos.proto d742adbbf24692d2d82aad51316d767af93942d5 
  src/common/protobuf_utils.hpp ff0fd01b7a20f597ec6cd916f0bd9c41baa5fd51 
  src/common/protobuf_utils.cpp 04f44f6f63e431c17ec67e234c8da58e7945294d 
  src/slave/slave.cpp 2e05637ed10a39eb3f4ce953506b1cb5d50caf3c 


Diff: https://reviews.apache.org/r/62644/diff/2/

Changes: https://reviews.apache.org/r/62644/diff/1-2/


Testing
-------

make check (Fedora 26)


Thanks,

James Peach


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