mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <>
Subject Re: Review Request 56178: Enabled the authorizer to work with MULTI_ROLE frameworks.
Date Mon, 06 Feb 2017 15:44:23 GMT

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

(Updated Feb. 6, 2017, 4:44 p.m.)

Review request for mesos, Adam B, Alexander Rojas, and Benjamin Mahler.


Added a comment on deprecation.

Bugs: MESOS-7022

Repository: mesos


This updates the local authorizer so that MULTI_ROLE frameworks can be

For non-MULTI_ROLE frameworks we continue to support use of the
deprecated 'value' field in the authorization request's 'Object';
however for MULTI_ROLE frameworks the 'value' field will not be set,
and authorizers still relying on it should be updated to instead use
the object's 'framework_info' field to extract roles to authorize
against from.

Diffs (updated)

  src/authorizer/local/authorizer.cpp b98e1fcdf2ee5ec1f6ac0be6f8accdefaa390a09 
  src/master/master.cpp 98c39b279e7b9830d02efc8ec6a4469afc15d62a 



Tested on various configurations in internal CI.


Benjamin Bannier

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