Comment 1 for bug 1347939

Revision history for this message
Ian Booth (wallyworld) wrote :

I found an unrelated deadlock in the login tests. It's very much timing related so it is plausible that underlying changes to state may lead to the deadlock occurring more frequently.

The problematic test is TestLoginRateLimited on loginSuite. This starts N login attempts, and then starts attempt N+1. The expectation in that N+1 will error out immediately with a try again error. But, all N+1 login attempts are racing via the rpc layer to reach the login on the server. And attempt N+1 may beat one of the N attempts to get there, thus one of the N attempts is the one to get the retry error. This stuff up the tests because attempt N+1 now blocks and the code to unblock the pending login attempts is never run.