googleapis / nodejs-logging-bunyan

Node.js client integration between Stackdriver Logging and Bunyan.
https://cloud.google.com/logging/
Apache License 2.0
63 stars 34 forks source link

express middleware request logging middleware: should write request correlated log entries failed #620

Closed flaky-bot[bot] closed 2 years ago

flaky-bot[bot] commented 2 years ago

This test failed!

To configure my behavior, see the Flaky Bot documentation.

If I'm commenting on this issue too often, add the flakybot: quiet label and I will stop commenting.


commit: 75d7811c277e4de87a2232db3ce856669964cb45 buildURL: Build Status, Sponge status: failed

Test output
Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/workspace/build/system-test/test-middleware-express.js)
Error: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/workspace/build/system-test/test-middleware-express.js)
    at listOnTimeout (internal/timers.js:554:17)
    at processTimers (internal/timers.js:497:7)
flaky-bot[bot] commented 2 years ago

Looks like this issue is flaky. :worried:

I'm going to leave this open and stop commenting.

A human should fix and close this.


When run at the same commit (75d7811c277e4de87a2232db3ce856669964cb45), this test passed in one build (Build Status, Sponge) and failed in another build (Build Status, Sponge).

losalex commented 2 years ago

There is an issue with proto compilation which breaks the build and fixed in 2112. Waiting for fix to propagate, so tests will be back to normal

losalex commented 2 years ago

Fix checked in and latest tests are normal