[jira] [Comment Edited] (MESOS-4512) Render quota status consistently with other endpoints.

["Alexander Rukletsov (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MESOS-4512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15306933#comment-15306933 ] 

Alexander Rukletsov edited comment on MESOS-4512 at 5/30/16 8:29 PM:
---------------------------------------------------------------------

I meant the role in {{Resource}}, not the one on {{QuotaInfo}}. Printing '\*' there is misleading, because the resource is set aside for quota, but since technically it is not a reserved resource, {{Resource.role}} must be '\*'. 

Since we are deprecating {{model()}} functions, we can't really do better here and will have to print {{Resource}} proto. I'm fine with closing the ticket.


was (Author: alexr):
I meant the role in {{Resource}}, not the one on {{QuotaInfo}}. Printing '*' there is misleading, because the resource is set aside for quota, but since technically it is not a reserved resource, {{Resource.role}} must be '*'. 

Since we are deprecating {{model()}} functions, we can't really do better here and will have to print {{Resource}} proto. I'm fine with closing the ticket.

> Render quota status consistently with other endpoints.
> ------------------------------------------------------
>
>                 Key: MESOS-4512
>                 URL: https://issues.apache.org/jira/browse/MESOS-4512
>             Project: Mesos
>          Issue Type: Bug
>          Components: master
>            Reporter: Alexander Rukletsov
>            Assignee: Alexander Rukletsov
>              Labels: http, mesosphere
>
> Currently quota status endpoint returns a collection of {{QuotaInfo}} protos converted to JSON. An example response looks like this:
> {code:xml}
> {
>   "infos": [
>     {
>       "role": "role1",
>       "guarantee": [
>         {
>           "name": "cpus",
>           "role": "*",
>           "type": "SCALAR",
>           "scalar": { "value": 12 }
>         },
>         {
>           "name": "mem",
>           "role": "*",
>           "type": "SCALAR",
>           "scalar": { "value": 6144 }
>         }
>       ]
>     }
>   ]
> }
> {code}
> Presence of some fields, e.g. "role", is misleading. To address this issue and make the output more informative, we should probably introduce a  {{model()}} function for {{QuotaStatus}}.



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