fixing container state merge

daidaotong · Apr 16, 2020 · 711f25f · 711f25f
1 parent 7a43f9d
commit 711f25f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/vertical-pod-autoscaler/pkg/recommender/model/aggregate_container_state.go b/vertical-pod-autoscaler/pkg/recommender/model/aggregate_container_state.go
@@ -157,7 +157,8 @@ func (a *AggregateContainerState) MergeContainerState(other *AggregateContainerS
 	a.AggregateCPUUsage.Merge(other.AggregateCPUUsage)
 	a.AggregateMemoryPeaks.Merge(other.AggregateMemoryPeaks)
 
-	if !other.FirstSampleStart.IsZero() && other.FirstSampleStart.Before(a.FirstSampleStart) {
+	if a.FirstSampleStart.IsZero() ||
+		(!other.FirstSampleStart.IsZero() && other.FirstSampleStart.Before(a.FirstSampleStart)) {
 		a.FirstSampleStart = other.FirstSampleStart
 	}
 	if other.LastSampleStart.After(a.LastSampleStart) {