mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Greg Mann" <>
Subject Re: Review Request 39989: [5/5] Added framework authorization for dynamic reservation.
Date Mon, 30 Nov 2015 20:48:27 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Nov. 30, 2015, 8:48 p.m.)

Review request for mesos, Adam B, Jie Yu, Michael Park, and Till Toenshoff.


Addressed comments, refactored validation into master's authorization helpers.

Bugs: MESOS-3062

Repository: mesos


Added framework authorization for dynamic reservation.
Note: this review is continued from

Diffs (updated)

  src/master/master.cpp b918ae4a0e7dc3cd41165fc4b683ae7b6f031821 
  src/tests/reservation_tests.cpp 15d180f92ec0aea99e6f3a7d0b505c62bd207b61 



This is the fifth in a chain of 5 patches. New reservation tests were added to `reservation_tests.cpp`
to validate the authentication of framework reserve and unreserve operations using ACLs. `make
check` was run to test after all patches were applied.


Greg Mann

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