-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69163/
-----------------------------------------------------------
(Updated Dec. 3, 2018, 2:20 p.m.)
Review request for mesos, Chun-Hung Hsiao, Gastón Kleiman, and James DeFelice.
Changes
-------
Addressed issues raised by Greg.
Bugs: MESOS-9293
https://issues.apache.org/jira/browse/MESOS-9293
Repository: mesos
Description
-------
This patch sets agent and/or resource provider ID operation status
update messages. This is not always possible, e.g., some operations
might fail validation so that no corresponding IDs can be extracted.
Since operations failing validation are currently directly rejected by
the master without going through a status update manager, they are not
retried either. If a master status update manager for operations is
introduced at a later point it should be possible to forward
acknowledgements for updates to the master's update manager (not agent
ID, not resource provider ID).
Diffs (updated)
-----
src/common/protobuf_utils.hpp 1662125ed3e47b179ee32d08c1d3af75553066ba
src/common/protobuf_utils.cpp a45607eed4c4bae5010bcc3f3ffeabd6d911062a
src/master/master.cpp 3b3824a67f46866cd64e32d7f9f92484b5891aa2
src/resource_provider/manager.cpp 6c81c430e9e1205d71982a7fa2bcd9aa15fc01b2
src/resource_provider/storage/provider.cpp
a22c82c442304979fbdec0fcb74543077751a135
src/slave/slave.cpp 858b78620e1ef33f3587d0bd95a684917aaf5bbb
src/tests/master_tests.cpp ef2c00101fc3d30c564a9ca34884dece2cdd2651
src/tests/mesos.hpp c08e7e6c1dbc3dd9eb980868e43368c0a423c3ee
Diff: https://reviews.apache.org/r/69163/diff/6/
Changes: https://reviews.apache.org/r/69163/diff/5-6/
Testing
-------
`make check`
Thanks,
Benjamin Bannier
