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

(Updated Dec. 15, 2015, 9:18 p.m.)


Review request for mesos, Jie Yu, Michael Park, and Neil Conway.


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


Repository: mesos


Description
-------

Added 'CreateVolume' and 'DestroyVolume' ACL support to the authorizer.


Diffs
-----

  include/mesos/authorizer/authorizer.hpp 
ec3d3e9c9bae461c8cd136f17cfda58e0cbd4e1b 
  src/authorizer/local/authorizer.hpp 965658b53d21734a2dbf3713d02d44d728fd6cca 
  src/authorizer/local/authorizer.cpp 1ac6a41d20454515a3ef2b84e296494a4592b3ea 
  src/tests/authorization_tests.cpp 32beeea3584f093bcac24e0c160235de0e3abb28 
  src/tests/mesos.hpp 2429ac5cbcd9c1a3949c11de94b542108a3c13d8 
  src/tests/mesos.cpp a501f04dfbf9c93c2196e83b512519e925a67c97 

Diff: https://reviews.apache.org/r/40168/diff/


Testing
-------

This is the third in a chain of 7 patches. New tests were added to 
`authorization_tests.cpp` in order to test the authorizer's handling of both 
successful and failed authorization attempts using `ACL::CreateVolume` and 
`ACL::DestroyVolume`. `make check` was used to test after all patches were 
applied.


Thanks,

Greg Mann

Reply via email to