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 Tue, 07 Feb 2017 10:26:07 GMT

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

(Updated Feb. 7, 2017, 11:26 a.m.)

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


Add link to `value`'s deprecation ticket.

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