You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Joseph Wu (JIRA)" <ji...@apache.org> on 2016/03/04 20:40:40 UTC

[jira] [Updated] (MESOS-4868) PersistentVolumeTests do not need to set up ACLs.

     [ https://issues.apache.org/jira/browse/MESOS-4868?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Joseph Wu updated MESOS-4868:
-----------------------------
    Description: 
The {{PersistentVolumeTest}} s have a custom helper for setting up ACLs in the {{master::Flags}}:
{code}
ACLs acls;
    hashset<string> roles;

    foreach (const FrameworkInfo& framework, frameworks) {
      mesos::ACL::RegisterFramework* acl = acls.add_register_frameworks();
      acl->mutable_principals()->add_values(framework.principal());
      acl->mutable_roles()->add_values(framework.role());

      roles.insert(framework.role());
    }

    flags.acls = acls;
    flags.roles = strings::join(",", roles);
{code}

This is no longer necessary with implicit roles.

  was:
The {{PersistentVolumeTest}}s have a custom helper for setting up ACLs in the {{master::Flags}}:
{code}
ACLs acls;
    hashset<string> roles;

    foreach (const FrameworkInfo& framework, frameworks) {
      mesos::ACL::RegisterFramework* acl = acls.add_register_frameworks();
      acl->mutable_principals()->add_values(framework.principal());
      acl->mutable_roles()->add_values(framework.role());

      roles.insert(framework.role());
    }

    flags.acls = acls;
    flags.roles = strings::join(",", roles);
{code}

This is no longer necessary with implicit roles.


> PersistentVolumeTests do not need to set up ACLs.
> -------------------------------------------------
>
>                 Key: MESOS-4868
>                 URL: https://issues.apache.org/jira/browse/MESOS-4868
>             Project: Mesos
>          Issue Type: Improvement
>          Components: technical debt, test
>            Reporter: Joseph Wu
>              Labels: mesosphere, newbie, test
>
> The {{PersistentVolumeTest}} s have a custom helper for setting up ACLs in the {{master::Flags}}:
> {code}
> ACLs acls;
>     hashset<string> roles;
>     foreach (const FrameworkInfo& framework, frameworks) {
>       mesos::ACL::RegisterFramework* acl = acls.add_register_frameworks();
>       acl->mutable_principals()->add_values(framework.principal());
>       acl->mutable_roles()->add_values(framework.role());
>       roles.insert(framework.role());
>     }
>     flags.acls = acls;
>     flags.roles = strings::join(",", roles);
> {code}
> This is no longer necessary with implicit roles.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)