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

(Updated Oct. 17, 2016, 12:12 p.m.)


Review request for mesos, Adam B, Kapil Arya, and Till Toenshoff.


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


Repository: mesos


Description (updated)
-------

Standarize the way in which authorization is perfomed by enabling
all actions suffixed with `_WITH_ROLE` and `_WITH_PRINCIPAL` to
perform authorization using whole protobuf messages instead of
forcing a single field from the message.

While this change is still in the deprecation cycle, the original
value is also set.


Diffs
-----

  3rdparty/libprocess/Makefile.am 71319891a451bd1d565210dcce2fb61fc69e1f61 
  3rdparty/libprocess/configure.ac 1644035d7fe1a45c798b13dbfb3c66f74466f2a2 
  3rdparty/libprocess/include/process/deferred.hpp 
ee902a6c8de48f17f7401ee7b3814d92eabd65f8 
  3rdparty/libprocess/src/openssl.cpp 8e4e0490a84c46e8f4842cf567f55a3c988ad445 
  3rdparty/libprocess/src/process.cpp 18a8e206f6f297157d246a94f374311be67cd782 
  3rdparty/libprocess/src/tests/CMakeLists.txt 
66597fd3021bc7c3c53cb0f38416edc85ace5a9b 
  3rdparty/libprocess/src/tests/future_tests.cpp 
7c411c7be1849119fe0b070622dbe4488fa11b7a 
  3rdparty/libprocess/src/tests/http_tests.cpp 
533104c93dd1eaf67bf3752163d2e0cad090078d 
  3rdparty/libprocess/src/tests/limiter_tests.cpp 
b80b1da214f97b50aa7b61b79bbf683fd01116aa 
  3rdparty/libprocess/src/tests/metrics_tests.cpp 
d7fdb06060b273e16be27a263b5ee268842aa25c 
  3rdparty/libprocess/src/tests/process_tests.cpp 
a4af54aa78c162bc0ecbe2f25796d8c9b12e0f31 
  3rdparty/libprocess/src/tests/reap_tests.cpp 
30518dee6c2fb904a607c7a457a5ec7366aab818 
  3rdparty/libprocess/src/tests/subprocess_tests.cpp 
483ac1c5540d54921de1c1488ccb5e63b17ac481 
  3rdparty/libprocess/src/tests/time_tests.cpp 
08ddb56f1789f400b8cd072c53e885c759f13ddc 
  3rdparty/stout/Makefile.am 4e10ae25a6c8a2d01d7b98b04ccf06a216611938 
  3rdparty/stout/README.md 693e3a3e76300f7fa6c6f6518497edf44cc9f6a2 
  3rdparty/stout/configure.ac cbb0fdb0e1584ace7bbd6bf496f0f55b9846ce0b 
  3rdparty/stout/include/Makefile.am 1eb9c146d3eebacaea32e0f724d145f0c1dd676f 
  3rdparty/stout/include/stout/flags/flags.hpp 
794a598169684c8474184b47de533498f3b234e0 
  3rdparty/stout/include/stout/gtest.hpp 
b2f75b6c706df9de68edbac86a1e2dec32a574ed 
  3rdparty/stout/include/stout/mac.hpp 7ed5f38ae4f93e267406254cb2acff201faffe45 
  3rdparty/stout/include/stout/os.hpp 96e8621b198a3ec4cce78e0a6ff5f271eda05ff1 
  3rdparty/stout/include/stout/os/linux.hpp 
bf12e0b0577810c64cc8276ff0d987524327ffcd 
  3rdparty/stout/include/stout/os/posix/killtree.hpp 
f47f0d85f88bf69b5a9ae74e32d4f41c9f4a985a 
  3rdparty/stout/include/stout/os/posix/shell.hpp 
70a9184e85ba322f1cd4ce5d12b963cd4b3ad500 
  3rdparty/stout/include/stout/os/raw/environment.hpp 
b3e82ac8071b41748aeb098b7d5fcc210a1d3c43 
  3rdparty/stout/include/stout/os/wait.hpp 
5f6483c0d754dd1c1f5fdcfc542a0cc05c0a8df5 
  3rdparty/stout/include/stout/os/windows/rmdir.hpp 
4437484c068e9ef046e0be14683c97db447f2da1 
  3rdparty/stout/include/stout/os/windows/shell.hpp 
17e3d564564abebf1d558b7a7a277aef3c87e5ae 
  3rdparty/stout/include/stout/windows/os.hpp 
7ca0b5dc9793369ea142684e3614e8f33cac64b6 
  3rdparty/stout/tests/CMakeLists.txt e52aa62479a345bbcee70047109ebd80b8f9813d 
  3rdparty/stout/tests/flags_tests.cpp 201b16a6c040e32f8a14fa7a2520c86997c363ad 
  3rdparty/stout/tests/ip_tests.cpp b5a206fe5fdcdadaa383698f505d87ddb2c6806d 
  3rdparty/stout/tests/mac_tests.cpp d4560e75ae4e03d1765c599298e88842381690b9 
  3rdparty/stout/tests/os/process_tests.cpp 
4cb3b5fab389492bdc1258a27e821e60aef19dc8 
  3rdparty/stout/tests/os/rmdir_tests.cpp 
9aa4059d589e84f3c377163b1d6d2a278d4130b6 
  3rdparty/stout/tests/os_tests.cpp 0b7ee0723b6a608d6f110fa8ac16e0fd7b75ddea 
  CHANGELOG 2e0da96d055c7a4c228726c628d94f3414b2be86 
  configure.ac 015255ec1876a51b9eb2cf488b375af90f73c722 
  docs/app-framework-development-guide.md 
de92c7570cd0d0ac8639ce50a79e5158844ac53c 
  docs/configuration.md c83a58eb6884c8d8c37880a745e04cf0b789ebdc 
  docs/contributors.yaml aa49acd188fada6659626d6cb9224c0dfc671f0f 
  docs/design-docs.md 7fafd09c7178559fedfa037faee0debfc29a927c 
  docs/docker-containerizer.md bab84dc2b0ce104b3ec59aaf0ef800b418a6517c 
  docs/executor-http-api.md a3b8d6f7cff4f182c8fb77dc216d82eca1d30e7c 
  docs/home.md 1c6b191bd194a9100ce1ad4bf5aff62a20ed8f41 
  docs/oversubscription.md ec7194638003d90a1a7a6eab722fb6ea9ff56732 
  docs/scheduler-http-api.md 958cfc52496b1585564e019f9ce8b06a623f2d5c 
  docs/shared-resources.md 29e43389c530f816118a32dcd7c2f1b82b0af431 
  docs/upgrades.md 8e45fdbb485829930f9b918d1d6f24893228e873 
  include/mesos/authorizer/authorizer.hpp 
cb365c7d8d088f2810bde11b72dc20843a18fa51 
  include/mesos/authorizer/authorizer.proto 
b6a9f142eecbdfd59210872a92e3126f04de334c 
  include/mesos/docker/v2.proto 9b5bfb5d03ae8115c1b20a8c640640d6dbaa257d 
  include/mesos/executor.hpp 68cba9e3c411eb9f396b49adcc9d71137a5d948c 
  include/mesos/log/log.hpp d92d7a0b982c1c4bc0fdedde19fe012a6523a224 
  include/mesos/master/master.proto 3553c683c17004ac1831ec90271aa8584c950e53 
  include/mesos/mesos.proto 0017d3d9d46433b391703025f611ce437bbc7ebe 
  include/mesos/resources.hpp f569c931ff7db8d51dfd7c96f4f2addab05df85d 
  include/mesos/slave/container_logger.hpp 
939974736f9eb744c83036e074718d2a1eba8b0a 
  include/mesos/slave/containerizer.proto 
94e8cb4c37aa2a06c59726773834812c5eb660d8 
  include/mesos/type_utils.hpp 88c7bcf7e8c3053e6e8bb78c67bf93b8276ac053 
  include/mesos/v1/master/master.proto 022b491b7d5c49c5aeddf4ffc97c148f55629c95 
  include/mesos/v1/mesos.proto def33ef5e446576c86da0498e8a24e2e2de17918 
  include/mesos/v1/resources.hpp f60ab794a0c7c24885c49cc47b798c363e3279e7 
  src/CMakeLists.txt e60d34ca4b714bb237d5864b4682280f11b147a6 
  src/Makefile.am 3bcc0f2dfc2c4f71841bd6d161f39e0e919fc0d7 
  src/authorizer/local/authorizer.cpp f1dff65d973fc84f4171f68fd0391a2343a96965 
  src/cli/execute.cpp 600542b5dcdc5dc1fd171ef2d236968406eb0ab4 
  src/cli_new/.gitignore 3ce5f74748dae6ac60170a1f0c73191fc26177d9 
  src/cli_new/README.md 0e60515b71192ce1a544711948a5c17a6f9002af 
  src/cli_new/activate 65df76f4590caf8160435ccbc1d6b199b115f7f8 
  src/cli_new/bin/config.py 274f8c63b0c642637f17aa2e3c8c4a8a5a059e37 
  src/cli_new/bin/main.py bbfb52c894540158c70e0f50ebb8a277b692d54d 
  src/cli_new/bin/mesos c5152a2ebf8704c804bb4f39e46580a512aecdea 
  src/cli_new/bootstrap 6d62e9adf1d543ed00a3a2cf2484edf1c33ee443 
  src/cli_new/deactivate 3dbe16765cad6f10523b1a3824608005e81b0944 
  src/cli_new/lib/mesos/__init__.py f4fc3f18af5641a4a87143adaba81e62334ccffb 
  src/cli_new/lib/mesos/docopt.py 86a4e9c74326fb80cc59487113f07358dd96960d 
  src/cli_new/lib/mesos/exceptions.py c78f4a97debdb89cc9972c89c5c1ea249836713b 
  src/cli_new/lib/mesos/plugins/__init__.py 
e743772afce3aad753be3cd2a923a4331bafc31c 
  src/cli_new/lib/mesos/plugins/base.py 
61b15428b029cec9ec7c9d89ab949959c3dc5b88 
  src/cli_new/lib/mesos/util.py 87d2a65e78f04209566c1434b489b941d570ee01 
  src/cli_new/mesos.bash_completion 21211819760c3a49566d32d4113116c9355182a5 
  src/cli_new/pip-requirements.txt 9cc8d096ac353f4ed2d40000318f2d0e439a6edf 
  src/cli_new/pylint.config c398220db063d249e6c62cf7e8cd6757e7860630 
  src/common/http.cpp fb8454a97ad78c306d17b4222c14d68b3598175d 
  src/common/protobuf_utils.hpp 80c66c4f7d52fc184e7553d173195ba7714971d4 
  src/common/protobuf_utils.cpp 7362b875ce1ffca6bc6376169a11494bdb9cf062 
  src/common/resources.cpp 4bb9beffcb3509f4226b4985e05eccec01412d0f 
  src/common/type_utils.cpp c6cf4f1d4acd040d21434221e5469ca911404a39 
  src/executor/executor.cpp 1d47b52d89eedee59d160badd6313d34e3bdb6d2 
  src/health-check/health_checker.cpp 96ae1a733ff3d211b84d0893b4603873af1c89f0 
  src/java/jni/org_apache_mesos_state_AbstractState.cpp 
0f12f4f7763727da51edb72f8e16cc520a783e42 
  src/java/src/org/apache/mesos/Executor.java 
078c0c7ea3ad68e9b4b40802c3d6b94084cbb69b 
  src/launcher/default_executor.cpp af4a97f7de5f2157aa65fdab742455b0683c40a4 
  src/launcher/executor.cpp 3e95d6029bea0ce6e0dfb39c24b795fe98d90d13 
  src/launcher/posix/executor.hpp d057ff6211a77e681ee80599d2ec5851a2fa6785 
  src/launcher/posix/executor.cpp da0081c0e470aebb16d2e78031d276f5d7d2c726 
  src/linux/cgroups.hpp cfce09cb57501f2c988a8d997d7c6150280ed53d 
  src/linux/fs.hpp da49c9ebfa938d169152ed3b6e4df7378711b013 
  src/linux/fs.cpp 913e23317291db164fe6bdf77f3eca146dedec9b 
  src/local/local.cpp 257179443827ffdfc946ed655a6840fcea70d454 
  src/log/CMakeLists.txt 8a92de4b0f9f69164930bd28c527f3a92ea0d17e 
  src/log/replica.cpp d596e617d4b4eab91245e0a88a3b9479fc75b813 
  src/master/allocator/sorter/drf/metrics.cpp 
15aab32db5ca1a7a14080e9bbb7c65283be3ec20 
  src/master/allocator/sorter/drf/sorter.cpp 
4ff153963229f2ecc73e05ba80f895c23e6a870e 
  src/master/http.cpp bb9c87327dfe2161a6f1fd4cded72aa9a5ffaf66 
  src/master/main.cpp 9d2fd92dbc2fc5af166edec907adbdb541eb0dd3 
  src/master/master.hpp 881f0d6127c4d6a7fccb1677fb3129310777f737 
  src/master/master.cpp 3c6b18ead44cd5f2978093f5415e974cfcbfa714 
  src/master/quota_handler.cpp d87d6c6266ba596c4e6bd36124f5a360428fe171 
  src/master/validation.cpp f690a9eacd278b51a52f5588dbeea377df074435 
  src/master/weights_handler.cpp c240fb2980a7e957136d11ddd3f43ee4761a07dc 
  src/slave/constants.hpp 6c381f06365b9deb84f43cdd101a2d2e5d826f57 
  src/slave/container_loggers/lib_logrotate.hpp 
d6e4f95e63825c01cb5fc05847ec249efc1348d2 
  src/slave/container_loggers/lib_logrotate.cpp 
53698d339f0f4d2dc916b53239ca0c36bbebcd42 
  src/slave/container_loggers/sandbox.hpp 
e0aeb32a9ec83af049af8a10010b819c1d8b25d8 
  src/slave/container_loggers/sandbox.cpp 
cc263ebef7e0c3e778fabafa49faa6dd315adc45 
  src/slave/containerizer/docker.cpp 8ec4c0a25335fb1b36cb2ab82577f6d3e2f7f008 
  src/slave/containerizer/mesos/containerizer.hpp 
c4fea8b56c39d5a363f6ea80bd109fd2d3db52d9 
  src/slave/containerizer/mesos/containerizer.cpp 
eac70d955e08142a2d054039d610a3d516b1b57e 
  src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp 
c2fcc0b21c5b1e51cb38b61245d4bbd3856a9512 
  src/slave/containerizer/mesos/isolators/filesystem/linux.cpp 
df16b8fee6799a69c7d96f33a5049bd9787c48f5 
  src/slave/containerizer/mesos/isolators/filesystem/posix.cpp 
270d2aa6e06f323bfb6eee3b703a24a600a55871 
  src/slave/containerizer/mesos/isolators/linux/capabilities.hpp 
c3afe2054bb206e9b2823c008435f78e70ce7d63 
  src/slave/containerizer/mesos/isolators/linux/capabilities.cpp 
60d22aa877c1ab62a08222e5efe8800e337684da 
  src/slave/containerizer/mesos/isolators/network/cni/cni.cpp 
7c35c3056326a8a135e4ad944ac741cda96cab99 
  
src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/main.cpp
 f8d38c6f6694e9eb9dc3f4f600b6a1976dd9c66a 
  
src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.hpp
 7fad707a240234e35828917aea1bc79f42fe130e 
  
src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp
 2ff8b0e76a11b6f6c98b839d3ac91a81e41285f5 
  src/slave/containerizer/mesos/isolators/network/cni/spec.hpp 
ccd511ec14810dcc1020dec5e1641141f3a319b4 
  src/slave/containerizer/mesos/launch.hpp 
cfbf79029dd2d0b74b015e6e051bd2d429321825 
  src/slave/containerizer/mesos/launch.cpp 
8a30ff8bd6f9263d68a4344b79f2374a2ae53c04 
  src/slave/containerizer/mesos/linux_launcher.cpp 
0305d14c1f791c93edcd3b32786b483b15f40a2d 
  src/slave/containerizer/mesos/provisioner/appc/fetcher.cpp 
f5e90527e6ace45a7745ca1ba1a9eeef938d2a68 
  src/slave/containerizer/mesos/provisioner/docker/local_puller.cpp 
817e30c5d6d6a4b011193e3209301fc3cdf88b06 
  src/slave/containerizer/mesos/provisioner/docker/metadata_manager.cpp 
395c36b0f6b8f8e037681ec5f2df99e83a7cf155 
  src/slave/containerizer/mesos/provisioner/docker/paths.hpp 
232c027f8f96da0cb30b957bce4607d3695050d2 
  src/slave/containerizer/mesos/provisioner/docker/paths.cpp 
dea37566cbe0313f84f2bac27e29752a980b4394 
  src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 
b06ddff68a8d2df13abb838b03a8e73d4e273c31 
  src/slave/containerizer/mesos/provisioner/docker/store.cpp 
e192f86a1848b373f3aa73d29688a96375cac313 
  src/slave/flags.hpp ec619f84c7f3be81db8ce8536936bf7d4207efa2 
  src/slave/flags.cpp 87d9e4632321134192bb0a67f1b91db7d89f539b 
  src/slave/http.cpp a32aca437cffd52bc2bcde859eedddca2038e3f1 
  src/slave/main.cpp 219914d594ace009201d9caf8209ca3dd4eaa1a5 
  src/slave/slave.hpp d8c5873bb7380b8449f8f344e85689519c467251 
  src/slave/slave.cpp 6bd9b49c3bbdb973a0d03552ae8fe55b33371083 
  src/tests/api_tests.cpp f0bafd1d5207a687ca0489b05ed1acdc2a6d875d 
  src/tests/authorization_tests.cpp 5d7e17b67821357b8cb538798acc883945c8f8fd 
  src/tests/container_logger_tests.cpp 1bb94a8461e481983f25a44737e4011ed5fc4b1f 
  src/tests/containerizer.hpp 940c4146f4e854a6b1b9ccaba5687e76d5723cba 
  src/tests/containerizer.cpp 27c29b40f3241807fb22b27f0d9ca64b3e602a8f 
  src/tests/containerizer/cgroups_isolator_tests.cpp 
b5797574ca2bfe3a005c80a3ba6bb6965c54cc95 
  src/tests/containerizer/fs_tests.cpp ccdd5967d01abfe4fa7746e3588ff93c6e0fe7e4 
  src/tests/containerizer/linux_capabilities_isolator_tests.cpp 
edb46659324c4c6345606cfa4c19f4fce05c59fe 
  src/tests/containerizer/linux_filesystem_isolator_tests.cpp 
f17ed4437a5e1366f85803ce7e29bee24162504c 
  src/tests/containerizer/nested_mesos_containerizer_tests.cpp 
c690b41f3852de035e5cc6378e7fc44d2c9a6fb9 
  src/tests/containerizer/provisioner_appc_tests.cpp 
a999fc7991da805dfbcdf5659fcfb762aee5b2b9 
  src/tests/containerizer/provisioner_docker_tests.cpp 
d8fb7abb55410cfd0835f59bfc0f35566aeeae37 
  src/tests/containerizer/rootfs.hpp 6bc3835cbb62536ec933ef38c9e15138b8611e5f 
  src/tests/containerizer/volume_image_isolator_tests.cpp 
f14a0bd1c44c069c2784638132a08072a2343309 
  src/tests/default_executor_tests.cpp 92e6b9f5fb80811c94632de3bb20c8e6d2e895ff 
  src/tests/environment.cpp 970136f98c601c4fbc71e8ffbf506bebe89c3658 
  src/tests/executor_http_api_tests.cpp 
a9f1a7b0498acd541c6f58ad1388da49c9951e22 
  src/tests/health_check_tests.cpp 1d1676d7259bf52cfb1e499954fa815fe7e37522 
  src/tests/hierarchical_allocator_tests.cpp 
2e979d784b8e6cdacebac78a67498b5f4d023540 
  src/tests/master_maintenance_tests.cpp 
6917272f2de7a09bf4de7e932994655f4e54d3da 
  src/tests/master_validation_tests.cpp 
da43f990e3b8f61e27b22d551cd3e07638c7ff37 
  src/tests/mesos.hpp 9309b5a985c0d7136a2ee5aa1598b4fee6194816 
  src/tests/oversubscription_tests.cpp b356fb62a4e068bc171a75a76001c6d0e76af92a 
  src/tests/persistent_volume_tests.cpp 
6289009fe9ed0a57ba5eff46dbbe0633a75d2616 
  src/tests/resources_tests.cpp 6a12783c26f359dda835b4866b299a8fcfb3f972 
  src/tests/scheduler_http_api_tests.cpp 
6390f2eb53d8bb97b98e64ec6e0f795abc3f3c7f 
  src/tests/slave_recovery_tests.cpp 703948f7a6861a4401ee45ce9cae2644106083f3 
  src/tests/slave_tests.cpp 93b81d3e5b90d8036409e943f598c865fe335bcc 
  src/tests/utils.cpp fc004a9c567898ffbc38a42cc2340dd57347a829 
  src/v1/resources.cpp 46cc00f2f453f5eb4ddc4b0b9b89be2bd89f05d9 
  src/webui/master/static/agent_executor.html 
95e921d1c3ca0f005e3eb3f79eb0d9a70a3b3f91 
  src/webui/master/static/framework.html 
6297cf932838ed32c8def1022efda9ef5a053d37 
  src/webui/master/static/home.html 07f862f69802bef8fda2d0a94e33c3b463ddcd4c 
  src/webui/master/static/js/app.js c32177aa23c962f2bdf03d98272fb6d21a565382 
  src/webui/master/static/js/controllers.js 
3dead4f20e2dbeec8447e22ad287dd6ed2378345 
  src/webui/master/static/js/jquery.pailer.js 
537fe002166167748a6357a50f4b7dd04b1f4257 
  src/zookeeper/group.cpp 1df4b54883d451fe36f132af45f2f7c77f3f221b 
  support/mesos-gtest-runner.py 457211a7721a1595a8efaf51a10041da0494bdfe 
  support/mesos-style.py 3a5bec81950ccf8083cbd73577c49b2fb3b910f8 
  support/windows-build.bat b3b6d0bbe877cf3794a5c616374ffce8c1b1552c 

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


Testing
-------

make check


Thanks,

Alexander Rojas

Reply via email to