Skip to content

Commit

Permalink
Merge pull request #1546 from ingvagabund/sortNodesByUsage-extended
Browse files Browse the repository at this point in the history
sortNodesByUsage: drop extended resources as they are already counted in
  • Loading branch information
k8s-ci-robot authored Nov 13, 2024
2 parents da52983 + 67d3d52 commit d419816
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions pkg/framework/plugins/nodeutilization/nodeutilization.go
Original file line number Diff line number Diff line change
Expand Up @@ -385,14 +385,6 @@ func sortNodesByUsage(nodes []NodeInfo, ascending bool) {
}
}

// extended resources
for name := range nodes[i].usage {
if !nodeutil.IsBasicResource(name) {
ti = ti + nodes[i].usage[name].Value()
tj = tj + nodes[j].usage[name].Value()
}
}

// Return ascending order for HighNodeUtilization plugin
if ascending {
return ti < tj
Expand Down

0 comments on commit d419816

Please sign in to comment.