[ https://issues.apache.org/jira/browse/YUNIKORN-93?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17100883#comment-17100883 ]
Kinga Marton commented on YUNIKORN-93: -------------------------------------- Thank you [~wilfreds] for the review! Before updating the PR I would like to discuss/share my opinion on a few possible cases: * *if {{capacity}} or {{used}} is nil*: instead of playing with +INF, I would return nil, because we don't have enough data to calculate the percentage, . Since this is a percentage value, what would be the meaning of +INF here? 100%? I think the clear solution is to just not calculate it and in the web ui will be shown as {{'n/a'}} * *all zeros for {{used}}*: this will result in all zeros for the calculated capacity as well * *all zeros for {{capacity}}*: However, this is not a valid case, because the config validator will [return an error if the total of max capacities = 0|https://github.com/apache/incubator-yunikorn-core/blob/master/pkg/common/configs/configvalidator.go#L99]. we should handle this case as well, so in this case my opinion is that we should return zero resource. * *{{capacity}} set to 0, usage is larger than 0*: I this this should not be a valid case, but I would return zero resource because of the 0 capacity setting * *no {{capacity}} set, but {{usage}} is set*: if no capacity is set but we have resource usage I would return an empty resource, because we cannot calculate the absolute resource usage, since we don’t know what would mean 100%, so the web ui will show n/a (I would not make here an exception for the case when the usage = 0, however we know that this would mean 0%) * *overflow case*: {{used}} = maxInt64, {{available}} is 10. Is this a valid scenario? Do we allow ho have higher usage than the configured max capacity? Note: above I am using capacity for simplicity, but that is max capacity, since the abs value calculation is maxValue/usage*100 [~wilfreds], [~wwei] what do you think? > Queue maximum capacity cleanup > ------------------------------ > > Key: YUNIKORN-93 > URL: https://issues.apache.org/jira/browse/YUNIKORN-93 > Project: Apache YuniKorn > Issue Type: Improvement > Reporter: Kinga Marton > Assignee: Kinga Marton > Priority: Major > Labels: pull-request-available > Attachments: queuesResponse.json > > > Right now the absolute used capacity is hardcoded to 20%. > The usage bar is rendered by this value, but currently, it is hardcoded. > Note, both capacity/max capacity could be 0. -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@yunikorn.apache.org For additional commands, e-mail: issues-h...@yunikorn.apache.org