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 Tue, 17 Nov 2015 17:07:42 GMT

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

(Updated Nov. 17, 2015, 5:07 p.m.)

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


Fixed authorization ordering, added new test.

Bugs: MESOS-3062

Repository: mesos


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

Diffs (updated)

  src/master/master.cpp 39a0e730b230cee73e30d831ee67d9207359ae28 
  src/tests/reservation_tests.cpp ac664ebb49e74aa28551f427ea8f39ac9ce0cfb3 



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