[YUNIKORN-541] Unexpected WARN log in node.go#refreshAvailableResource

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.11
• Component/s: core - scheduler
• Labels: log pull-request-available
• Assignee: Tao Yang
• Reporter: Tao Yang
• Created: 07/Feb/21 09:11
• Updated: 21/Jan/22 21:48
• Resolved: 07/Jan/22 01:07

Description

We have found lots of unexpected WARN log:  

WARN    objects/node.go:169 Node update triggered over allocated node ...

For this problem, I think below if-condition in node.go#refreshAvailableResource is opposite:

	// check if any quantity is negative: a nil resource is all 0's
	if resources.StrictlyGreaterThanOrEquals(sn.availableResource, nil) {
		log.Logger().Warn("Node update triggered over allocated node",
			zap.String("available", sn.availableResource.String()),
			zap.String("total", sn.totalResource.String()),
			zap.String("occupied", sn.occupiedResource.String()),
			zap.String("allocated", sn.allocatedResource.String()))
	}

Issue Links

• links to: GitHub Pull Request #248