Merge pull request #38917 from foxyriver/if-statement-must-be-true

Automatic merge from submit-queue (batch tested with PRs 38426, 38917, 38891, 38935) if statement must be true **What this PR does / why we need it**: if len(metrics.Items)==0, the function would been returned. so the statement if len(metrics.Items) > 0 is redudant, it must be true. **Special notes for your reviewer**: **Release note**: ```release-note ```

Merge pull request #38917 from foxyriver/if-statement-must-be-true
d373d1c4 · Kubernetes Submit Queue · GitHub · 9484212b · 69c76d83 · d373d1c4
Commit d373d1c4 authored Dec 19, 2016 by Kubernetes Submit Queue Committed by GitHub Dec 19, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 4 deletions

metrics_client.go pkg/controller/podautoscaler/metrics/metrics_client.go +1 -4

No files found.
--- a/pkg/controller/podautoscaler/metrics/metrics_client.go
+++ b/pkg/controller/podautoscaler/metrics/metrics_client.go
@@ -122,10 +122,7 @@ func (h *HeapsterMetricsClient) GetResourceMetric(resource v1.ResourceName, name
 		}
 	}

-	timestamp := time.Time{}
-	if len(metrics.Items) > 0 {
-		timestamp = metrics.Items[0].Timestamp.Time
-	}
+	timestamp := metrics.Items[0].Timestamp.Time

 	return res, timestamp, nil
 }