Merge pull request #170 from aleksandra-malinowska/find-unneeded-scope

Change scope of findUnneeded metric
This commit is contained in:
MaciekPytel 2017-07-07 16:47:33 +02:00 committed by GitHub
commit 61ee171ec4
1 changed files with 1 additions and 2 deletions

View File

@ -252,14 +252,13 @@ func (a *StaticAutoscaler) RunOnce(currentTime time.Time) errors.AutoscalerError
} }
if a.ScaleDownEnabled { if a.ScaleDownEnabled {
unneededStart := time.Now()
pdbs, err := pdbLister.List() pdbs, err := pdbLister.List()
if err != nil { if err != nil {
glog.Errorf("Failed to list pod disruption budgets: %v", err) glog.Errorf("Failed to list pod disruption budgets: %v", err)
return errors.ToAutoscalerError(errors.ApiCallError, err) return errors.ToAutoscalerError(errors.ApiCallError, err)
} }
unneededStart := time.Now()
// In dry run only utilization is updated // In dry run only utilization is updated
calculateUnneededOnly := a.lastScaleUpTime.Add(a.ScaleDownDelay).After(time.Now()) || calculateUnneededOnly := a.lastScaleUpTime.Add(a.ScaleDownDelay).After(time.Now()) ||
a.lastScaleDownFailedTrial.Add(a.ScaleDownTrialInterval).After(time.Now()) || a.lastScaleDownFailedTrial.Add(a.ScaleDownTrialInterval).After(time.Now()) ||