[Lldb-commits] [lldb] [lldb] Added an exponential algorithm for the sleep time in connect_to_debug_monitor() (PR #118222)

[Pavel Labath via lldb-commits]

https://github.com/labath approved this pull request.

Yeah, that's the general idea, although 16 attempts @ 262s sounds a bit too much (that's like almost three times as long as the first version with 30*3s). I worry that the tests will take too long to time out if they not able to establish a connection due to some other reason (like, imagine a bad patch causes lldb-server to never accept a connection). I'd suggest starting out with 10attempts at 77 seconds. That's still higher than the initial ~60 seconds, and I really hope that should be enough since doubling the port range sort of doubles the number of attempts as well (by halving the chance of collision).

https://github.com/llvm/llvm-project/pull/118222