why is this test hanging in CI but not locally?

Jose Nino · Jose Nino · commit bc382b02a303 · 2020-06-29T17:21:25.000-07:00
Signed-off-by: Jose Nino &lt;jnino@lyft.com&gt;
diff --git a/test/integration/server.h b/test/integration/server.h
@@ -198,11 +198,13 @@ class NotifyingAllocatorImpl : public Stats::AllocatorImpl {
 
   virtual void waitForCounterFromStringEq(const std::string& name, uint64_t value) {
     absl::MutexLock l(&mutex_);
-    ENVOY_LOG_MISC(trace, "waiting for {} to be {}", name, value);
+    ENVOY_LOG_MISC(error, "waiting for {} to be {}", name, value);
     while (getCounterLockHeld(name) == nullptr || getCounterLockHeld(name)->value() != value) {
+      ENVOY_LOG_MISC(error, "but {} is currently {}", name,
+                     getCounterLockHeld(name) == nullptr ? -1 : getCounterLockHeld(name)->value());
       condvar_.Wait(&mutex_);
     }
-    ENVOY_LOG_MISC(trace, "done waiting for {} to be {}", name, value);
+    ENVOY_LOG_MISC(error, "done waiting for {} to be {}", name, value);
   }
 
   virtual void waitForCounterFromStringGe(const std::string& name, uint64_t value) {

Original file line number	Diff line number	Diff line change
`@@ -198,11 +198,13 @@ class NotifyingAllocatorImpl : public Stats::AllocatorImpl {`
`198`	`198`
`199`	`199`	`virtual void waitForCounterFromStringEq(const std::string& name, uint64_t value) {`
`200`	`200`	`absl::MutexLock l(&mutex_);`
`201`		`- ENVOY_LOG_MISC(trace, "waiting for {} to be {}", name, value);`
	`201`	`+ ENVOY_LOG_MISC(error, "waiting for {} to be {}", name, value);`
`202`	`202`	`while (getCounterLockHeld(name) == nullptr \|\| getCounterLockHeld(name)->value() != value) {`
	`203`	`+ ENVOY_LOG_MISC(error, "but {} is currently {}", name,`
	`204`	`+ getCounterLockHeld(name) == nullptr ? -1 : getCounterLockHeld(name)->value());`
`203`	`205`	`condvar_.Wait(&mutex_);`
`204`	`206`	`}`
`205`		`- ENVOY_LOG_MISC(trace, "done waiting for {} to be {}", name, value);`
	`207`	`+ ENVOY_LOG_MISC(error, "done waiting for {} to be {}", name, value);`
`206`	`208`	`}`
`207`	`209`
`208`	`210`	`virtual void waitForCounterFromStringGe(const std::string& name, uint64_t value) {`