Commit 66e0b53c authored by xiechengsheng's avatar xiechengsheng

fix some typos

parent cd65afdf
...@@ -44,8 +44,8 @@ func balancedResourceScorer(requested, allocable *schedulercache.Resource, inclu ...@@ -44,8 +44,8 @@ func balancedResourceScorer(requested, allocable *schedulercache.Resource, inclu
// This to find a node which has most balanced CPU, memory and volume usage. // This to find a node which has most balanced CPU, memory and volume usage.
if includeVolumes && utilfeature.DefaultFeatureGate.Enabled(features.BalanceAttachedNodeVolumes) && allocatableVolumes > 0 { if includeVolumes && utilfeature.DefaultFeatureGate.Enabled(features.BalanceAttachedNodeVolumes) && allocatableVolumes > 0 {
volumeFraction := float64(requestedVolumes) / float64(allocatableVolumes) volumeFraction := float64(requestedVolumes) / float64(allocatableVolumes)
if cpuFraction > 1 || memoryFraction > 1 || volumeFraction > 1 { if cpuFraction >= 1 || memoryFraction >= 1 || volumeFraction >= 1 {
// if requested > capacity, the corresponding host should never be preferred. // if requested >= capacity, the corresponding host should never be preferred.
return 0 return 0
} }
// Compute variance for all the three fractions. // Compute variance for all the three fractions.
...@@ -57,8 +57,8 @@ func balancedResourceScorer(requested, allocable *schedulercache.Resource, inclu ...@@ -57,8 +57,8 @@ func balancedResourceScorer(requested, allocable *schedulercache.Resource, inclu
return int64((1 - variance) * float64(schedulerapi.MaxPriority)) return int64((1 - variance) * float64(schedulerapi.MaxPriority))
} }
if cpuFraction > 1 || memoryFraction > 1 { if cpuFraction >= 1 || memoryFraction >= 1 {
// if requested > capacity, the corresponding host should never be preferred. // if requested >= capacity, the corresponding host should never be preferred.
return 0 return 0
} }
// Upper and lower boundary of difference between cpuFraction and memoryFraction are -1 and 1 // Upper and lower boundary of difference between cpuFraction and memoryFraction are -1 and 1
...@@ -71,7 +71,7 @@ func balancedResourceScorer(requested, allocable *schedulercache.Resource, inclu ...@@ -71,7 +71,7 @@ func balancedResourceScorer(requested, allocable *schedulercache.Resource, inclu
func fractionOfCapacity(requested, capacity int64) float64 { func fractionOfCapacity(requested, capacity int64) float64 {
if capacity == 0 { if capacity == 0 {
return math.MaxFloat64 return 1
} }
return float64(requested) / float64(capacity) return float64(requested) / float64(capacity)
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment