[mesos] branch master updated: Replaced a set of single '*' role in SCRIPT_TEST ACL with ANY.

[bmahler]

This is an automated email from the ASF dual-hosted git repository.

bmahler pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git


The following commit(s) were added to refs/heads/master by this push:
     new 2030964  Replaced a set of single '*' role in SCRIPT_TEST ACL with ANY.
2030964 is described below

commit 203096466435dac62b89f3d16822160fab2dee78
Author: Andrei Sekretenko <asekrete...@mesosphere.io>
AuthorDate: Thu Jun 13 14:47:25 2019 -0400

    Replaced a set of single '*' role in SCRIPT_TEST ACL with ANY.
    
    This patch changes the roles set in the ACL used by SCRIPT_TESTs for the
    `register_frameworks` action from the '*' role to `ANY`. This is needed
    for adding and removing roles via the UPDATE_FRAMEWORK call in
    `ExamplesTest`s.
    
    Review: https://reviews.apache.org/r/70814/
---
 src/tests/script.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/tests/script.cpp b/src/tests/script.cpp
index 2e50861..038519c 100644
--- a/src/tests/script.cpp
+++ b/src/tests/script.cpp
@@ -157,7 +157,7 @@ void execute(const string& script)
 
     mesos::ACL::RegisterFramework* register_ = acls.add_register_frameworks();
     
register_->mutable_principals()->add_values(DEFAULT_CREDENTIAL.principal());
-    register_->mutable_roles()->add_values("*");
+    register_->mutable_roles()->set_type(mesos::ACL::Entity::ANY);
 
     // Allow agents with any principal or no principal to register.
     // Currently the agents in the example tests don't have authentication