Test failure in test_oversubscribing_leases

[mrocklin]

This showed up in a recent PR. I copy-pasted the traceback and then restarted the job.
Unfortunately my attempt to copy-paste the trace turned out to be incomplete. My apologies. Hopefully this helps though.

cc @fjetter

>   assert not sem.acquire(timeout=0.05)
E   assert not True
E    +  where True = <bound method Semaphore.acquire of <distributed.semaphore.Semaphore object at 0x0000027BF23DE438>>(timeout=0.05)
E    +    where <bound method Semaphore.acquire of <distributed.semaphore.Semaphore object at 0x0000027BF23DE438>> = <distributed.semaphore.Semaphore object at 0x0000027BF23DE438>.acquire

distributed\tests\test_semaphore.py:396: AssertionError
---------------------------- Captured stderr call -----------------------------
distributed.worker - WARNING -  Compute Failed
Function:  observe_state
args:      ()
kwargs:    {'sem': <distributed.semaphore.Semaphore object at 0x0000027BF23DE438>}
Exception: AssertionError('assert not True\n +  where True = <bound method Semaphore.acquire of <distributed.semaphore.Semaphore object at 0x0000027BF23DE438>>(timeout=0.05)\n +    where <bound method Semaphore.acquire of <distributed.semaphore.Semaphore object at 0x0000027BF23DE438>> = <distributed.semaphore.Semaphore object at 0x0000027BF23DE438>.acquire',)

[mrocklin]

It's coming up here as well. (I didn't restart the job this time)

https://github.com/dask/distributed/pull/3650/checks?check_run_id=593854022

Pinging also @lr4d

[fjetter]

Looking into it

[mrocklin]

Thanks!

…

On Fri, Apr 17, 2020 at 8:31 AM Florian Jetter ***@***.***> wrote: Looking into it — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#3717 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACKZTHNTBLY3QM65WIHEDLRNBY6PANCNFSM4MKJ63HQ> .

[mrocklin]

I think that this is still happening. I reran these tests about an hour ago, which I think is after the recent merge of #3720

https://github.com/dask/distributed/pull/3650/checks?check_run_id=596031832

[lr4d]

@mrocklin in the traceback I see > assert not sem.acquire(timeout=0.05), whereas this was changed in #3720 at https://github.com/dask/distributed/pull/3720/files#diff-dee863c4f303f33e8e88eb708f00130bL396