[YUNIKORN-1609] Make API factory and Informer timeouts configurable

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Component/s: documentation, shim - kubernetes
• Labels: pull-request-available
• Target Version: 1.3.0
• Assignee: Eli Schiff
• Reporter: Eli Schiff
• Created: 27/Feb/23 17:40
• Updated: 20/Jun/23 14:41
• Resolved: 28/Feb/23 22:28

Description

Right now there are a few places I was able to find with a hard coded 30 second timeout. It would be nice if those timeouts were user configurable. 

These are the 2 places in the scheduler that need to be updated to fix the 30 second timeout:

https://github.com/apache/yunikorn-k8shim/blob/master/pkg/client/apifactory.go#L217

https://github.com/apache/yunikorn-k8shim/blob/master/pkg/client/apifactory.go#L224

 

Context:

Yunikorn slack thread

I was seeing error/warning logs like `nodes recovery failed {"error": "timeout waiting for condition"}`, `scheduler recovery failed {"error": "timeout waiting for condition"}`, and `Failed to sync informers    {"error": "timeout waiting for condition"}`.

Issue Links

• links to: GitHub Pull Request #271
• links to: GitHub Pull Request #543