Address comments for issue506

aws · Jan 11, 2017 · 6d41ad1 · 6d41ad1
1 parent 4a7798f
commit 6d41ad1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/agent/stats/engine.go b/agent/stats/engine.go
@@ -402,8 +402,8 @@ func (engine *DockerStatsEngine) getContainerMetricsForTask(taskArn string) ([]*
 			continue
 		}
 
-		if !container.statsQueue.resetThresholdElapsed(queueResetThreshold) &&
-			!container.statsQueue.enoughDatapointsInBuffer() {
+		if !container.statsQueue.enoughDatapointsInBuffer() &&
+			!container.statsQueue.resetThresholdElapsed(queueResetThreshold) {
 			seelog.Debug("Stats not ready for container %s", dockerID)
 			continue
 		}

diff --git a/agent/tcs/client/client_test.go b/agent/tcs/client/client_test.go
@@ -157,7 +157,6 @@ func TestPublishMetricsRequest(t *testing.T) {
 	cs.Close()
 }
 func TestPublishMetricsOnceEmptyStatsError(t *testing.T) {
-
 	cs := clientServer{
 		statsEngine: &emptyStatsEngine{},
 	}