[YUNIKORN-2788] Node.AddAllocation() isn't error checked

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.6.0
• Labels: pull-request-available
• Target Version: 1.6.0
• Assignee: Craig Condit
• Reporter: Craig Condit
• Created: 06/Aug/24 21:18
• Updated: 07/Aug/24 16:13
• Resolved: 07/Aug/24 16:13

Description

The AddAllocation() method is sometimes called in contexts where the allocation can fail, and others where it should not. This leads to silent corruption of values in certain cases. To fix this, we should split this method into TryAddAllocation(), using the existing semantics, and a new AddAllocation() method which cannot fail.

Issue Links

• links to: GitHub Pull Request #932