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

Saad Ur Rahman edited comment on YUNIKORN-1213 at 5/18/22 9:55 PM:
-------------------------------------------------------------------

Hi [~wwei] 

I am on the last piece of configuring the health check settings from the 
configs and I have the tests completed. How can I get the partition number for 
Health Check in question? Currently, my code is going for partition 0:

[https://github.com/surahman/yunikorn-core/blob/4855591dcd608e377de691c51ff5d7a297d42f6f/pkg/scheduler/health_checker.go#L45-L63]

 

+Edit:+
I am reworking the logic for default checks. I will be switching to 
_*disabled*_ instead of _*enabled*_ due to a corner case I have found. The 
issue is that the YAML parser treats missing entries as false and 0 for the 
_*enabled*_ and _*period*_ keys.


was (Author: surahman):
Hi [~wwei] 

I am on the last piece of configuring the health check settings from the 
configs and I have the tests completed. How can I get the partition number for 
Health Check in question? Currently, my code is going for partition 0:

https://github.com/surahman/yunikorn-core/blob/4855591dcd608e377de691c51ff5d7a297d42f6f/pkg/scheduler/health_checker.go#L45-L63

 

> The interval of the background health checker needs to be configurable
> ----------------------------------------------------------------------
>
>                 Key: YUNIKORN-1213
>                 URL: https://issues.apache.org/jira/browse/YUNIKORN-1213
>             Project: Apache YuniKorn
>          Issue Type: Improvement
>          Components: core - scheduler
>            Reporter: Weiwei Yang
>            Priority: Major
>
> YUNIKORN-1107 adds a background running health checker to verify the 
> scheduler data correctness in the fixed time interval 30s: 
> https://github.com/apache/yunikorn-core/blob/3ba91fb8a41c0fd0dd6243326e583dea5167199f/pkg/scheduler/health_checker.go#L34.
>  We need to make this configurable, either let the user set a longer/shorter 
> interval, or completely disable it.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

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

Reply via email to