-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65939/
-----------------------------------------------------------
(Updated May 5, 2018, 2:14 p.m.)
Review request for mesos, Benjamin Bannier, Benjamin Mahler, Kapil Arya,
Michael Park, and Meng Zhu.
Changes
-------
Updated to fit semantics of "allocated".
Summary (updated)
-----------------
Updated role endpoints for detailed, hierarchical accounting.
Bugs: MESOS-8069, MESOS-8789 and MESOS-8790
https://issues.apache.org/jira/browse/MESOS-8069
https://issues.apache.org/jira/browse/MESOS-8789
https://issues.apache.org/jira/browse/MESOS-8790
Repository: mesos
Description (updated)
-------
When listing role "a", resources allocated to role "a/b" are added to
those allocated to role "a". These changes affect both, the "/roles"
endpoint as well as the V1 HTTP Call "GET_ROLES".
Adds dynamic hierarchical role tracking to the master.
Deprecates the `resources` member of the `Role` Message in mesos.proto
(V0 + V1), replacing it by `allocated` and `offered`. Follows that
deprecation on role-related endpoints while adding resource accounting
for `[total_]allocated` and `[total_]offered` to "/roles" as well as
"GET_ROLES".
Diffs (updated)
-----
CHANGELOG 0254044808bc04014803ad36aebc999b4ccea816
include/mesos/master/master.proto 54f84120728eea7995422b9c356ed67e5b054623
include/mesos/mesos.proto 463e6adcdf14115b2f21270bb1fd9c45e1b67cc3
include/mesos/v1/master/master.proto 12f019d8d0902e212d4a3706fe2310b514d0d183
include/mesos/v1/mesos.proto 8eaad9c4b2a9cdd527922a004b0f7dd0dc58a7f1
src/master/http.cpp 77cf47a898a7b4fc3b548373fcf0f6232e39a369
src/master/master.hpp 76e7763a99972f686af9d7eed08b6b6b1db23b0f
src/master/master.cpp 3b5d2eba3f602f68a6bb1e00444b01fb58a1bfc2
src/master/quota_handler.cpp 21bafd0064e9397f88185eaf687a58f85da94e2c
src/master/weights_handler.cpp 1053652804a8fc6af31a3b8a9f632f836c897fa9
Diff: https://reviews.apache.org/r/65939/diff/5/
Changes: https://reviews.apache.org/r/65939/diff/4-5/
Testing
-------
make check
Thanks,
Till Toenshoff
