[Lldb-commits] [lldb] [lldb][target] Add progress report for wait-attaching to process (PR #144768)

[Chelsea Cassanova via lldb-commits]

chelcassanova wrote:

@labath 

> Well.. IIUC, this test tries to attach to a process called "a.out" -- and hopes that the process doesn't appear.

That's pretty much it, I just need to wait-attach to _some_ process to trigger the progress report I'm expecting and then drop the process once we've triggered that report and check that it was emitted correctly.

> If you run the test suite in parallel, you'll have any number of processes like that going around all the time. Not only will this test fail, but it will mess up the state of a random other test.

Great point, I hadn't considered this. I'm also a little surprised this seems to specifically affect ARM Ubuntu here 🤔 .

> I guess you could just wait for a sufficiently unique process name...

This is probably the right way to go for this, I'll update the test to do this.

@DavidSpickett 

> Though I cannot reproduce the failure, so it could be to do with the high load when run on the bots. Some process being deprioritised.

With this and Pavel's comment about the generic process name it's probably how this test works on the bots. I'm not sure if there's a way to preview how this test would run on the bots before actually landing (outside of running it locally or having the GH pre-commit checks), but I'll try relanding and changing the process name to wait on for now since I don't think I have another way to ensure the test would be fine on the bots.

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