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

Hengzhe Guo commented on YUNIKORN-2908:
---------------------------------------

Sure I can repurpose this ticket to queue removal only. For this one I realized 
a caveat: completely unregistering the queue's metrics when it's actually 
removed is fine, but we may also want to remove guaranteed resource metrics 
when the queue is marked for deletion, before it's actually removed. This can 
be part of your complex change too

> metrics not removed when queue or queue's guaranteed/max resource config is 
> removed
> -----------------------------------------------------------------------------------
>
>                 Key: YUNIKORN-2908
>                 URL: https://issues.apache.org/jira/browse/YUNIKORN-2908
>             Project: Apache YuniKorn
>          Issue Type: Bug
>            Reporter: Hengzhe Guo
>            Assignee: Hengzhe Guo
>            Priority: Major
>
> 1. after a queue is removed, its metrics will continue to be reported by 
> prometheus. This is fine with metrics like allocated resource because they 
> will just be 0, but it won't make sense for guaranteed and max resources, 
> giving wrong impression that there are still resource given to the queue. I 
> propose to unregister all this queue's metrics when it's removed.
> 2. If queue is not removed but guaranteed or max resource config is removed, 
> or just a resource type is removed from the config, the metrics are also not 
> cleaned up. these metrics are only updated when there's a new valid value, 
> but not 'null' value. I propose to always delete all existing guaranteed and 
> max resources metrics of the queue then add back the new values, every time 
> we apply the configs.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to