pubsub Topic ordered messages: should pass the acceptance tests failed

[flaky-bot]

Note: #1715 was also for this test, but it was closed more than 10 days ago. So, I didn't mark it flaky.

---

commit: 146c028
buildURL: Build Status, Sponge
status: failed

Test output

Timeout of 600000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/workspace/build/system-test/pubsub.js)
Error: Timeout of 600000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/workspace/build/system-test/pubsub.js)
    at listOnTimeout (internal/timers.js:554:17)
    at processTimers (internal/timers.js:497:7)

[flaky-bot]

Looks like this issue is flaky. 😟

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

A human should fix and close this.

---

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

[flaky-bot]

Oops! Looks like this issue is still flaky. It failed again. 😬

I reopened the issue, but a human will need to close it again.

---

commit: bc7e915
buildURL: Build Status, Sponge
status: failed

Test output

Expected "message5642" but received "message654" for key "CCC"
Error: Expected "message5642" but received "message654" for key "CCC"
    at Subscription. (build/system-test/pubsub.js:236:41)
        -> /workspace/system-test/pubsub.ts:344:17
    at Subscriber. (build/src/subscription.js:255:44)
        -> /workspace/src/subscription.ts:331:38
    at /workspace/build/src/lease-manager.js:207:53
        -> /workspace/src/lease-manager.ts:243:47
    at processTicksAndRejections (internal/process/task_queues.js:79:11)

[feywind]

Yeah, it seems there's a legit flaky test thing here still.

[feywind]

This happened only once, and a year ago, and I don't see anything super obviously wrong in the test itself. :| I'm just going to chalk it up to chance unless it happens again.