You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Benjamin Bannier (JIRA)" <ji...@apache.org> on 2017/02/16 17:11:42 UTC

[jira] [Updated] (MESOS-6730) Reserve operation should validate reserved resource role against resource allocationInfo role

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

Benjamin Bannier updated MESOS-6730:
------------------------------------
    Sprint: Mesosphere Sprint 51

> Reserve operation should validate reserved resource role against resource allocationInfo role
> ---------------------------------------------------------------------------------------------
>
>                 Key: MESOS-6730
>                 URL: https://issues.apache.org/jira/browse/MESOS-6730
>             Project: Mesos
>          Issue Type: Bug
>            Reporter: Guangya Liu
>            Assignee: Benjamin Bannier
>             Fix For: 1.3.0
>
>
> When doing dynamic reservation validation, the current logic is make sure the reserved resources role is same as the framework role (see [src/master/validation.cpp|https://github.com/apache/mesos/blob/0228fa74c25f450478a6a5a42e1ca384c26db8bd/src/master/validation.cpp#L1539-L1544]):
> {code}
>   if (frameworkRole.isSome() && resource.role() != frameworkRole.get()) {
>       return Error(
>           "A reserve operation was attempted for a resource with role"
>           " '" + resource.role() + "', but the framework can only reserve"
>           " resources with role '" + frameworkRole.get() + "'");
>     }
> {code}
> With multi-role framework, we should validate reserved resource role same as resource allocation role.
> Please make sure distinguish dynamic reservation with framework and http endpoint. If dynamic reservation was triggered by a framework, then we need to do such validation. If done by the http endpoint, then no need to validate the roles.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)