mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From James Peach <jpe...@apache.org>
Subject Review Request 62641: Removed multiple reasons from ContainerTermination.
Date Thu, 28 Sep 2017 00:20:16 GMT

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

Review request for mesos, Jie Yu and Qian Zhang.


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


Repository: mesos


Description
-------

Since the ContainerLimitation is no longer ever populated with more than
one TaskStatus.Reason, make it optional rather than repeated.


Diffs
-----

  include/mesos/slave/containerizer.proto 84f9ca765fe6e29ddd2f7956ba0976e10b21d685 
  src/slave/containerizer/mesos/containerizer.cpp 4d5dc13f363f5d8886983d7dd06a5cecc177c345

  src/slave/slave.cpp 75e2e25c438637e2b1731f9fd01b6fe297687c50 
  src/tests/containerizer/io_switchboard_tests.cpp bf7917ac72edff22561545a807bd17b610aec796



Diff: https://reviews.apache.org/r/62641/diff/1/


Testing
-------

make check (Fedora 26)


Thanks,

James Peach


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