[YUNIKORN-2637] finalizePods should ignore pods like registerPods does

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.5.2, 1.6.0
• Component/s: shim - kubernetes
• Labels: pull-request-available
• Target Version: 1.5.2, 1.6.0
• Assignee: Wilfred Spiegelenburg
• Reporter: Wilfred Spiegelenburg
• Created: 21/May/24 05:54
• Updated: 07/Jun/24 13:36
• Resolved: 07/Jun/24 13:36

Description

The initialisation code is a two step process for pods: first list all pods and add them to the system in registerPods(). This returns a list of pods processed.

The second step happens after event handlers are turned on and nodes have been cleaned up etc. During the second step pods from the first step are checked and removed. However pods that were already in a terminated state in step 1 get removed again. Although the step should be idempotent this is unneeded. When iterating over the existing pods any pod in a terminal state should be skipped.

Issue Links

• links to: GitHub Pull Request #847