| swest | corvus: took a stab at fixing the deadlock https://github.com/python-zk/kazoo/pull/650 | 09:56 |
|---|---|---|
| swest | corvus: it seems their travis CI setup is broken :/ | 10:00 |
| mordred | swest: :( | 13:35 |
| mordred | swest: it looks like *very* low activity - like - last commit merged in january | 13:36 |
| mordred | oh - well - looking back, a few commits with 6 months apart seems normal for them | 13:37 |
| mordred | actually - that's not true. it was active-ish until june 2020 (when corvus landed his most recent change there) then a few nits in dec | 13:38 |
| opendevreview | lotorev vitaly proposed zuul/zuul master: docs: Replace job to branches in job.branches definition https://review.opendev.org/c/zuul/zuul/+/803105 | 14:44 |
| opendevreview | lotorev vitaly proposed zuul/zuul master: docs: Replace job to branches in job.branches definition https://review.opendev.org/c/zuul/zuul/+/803105 | 14:45 |
| pabelanger[m] | just upgraded zuul to 4.7.0, and seeing a lot of the following errors in log | 14:46 |
| pabelanger[m] | 2021-07-31 14:44:03,076 ERROR zuul.zk.SemaphoreHandler: Semaphore /zuul/semaphores/ansible/ansible-test-cloud-integration-aws can not be released for 5fd176ec57dd41dcb71205066c240c6a-ansible-test-cloud-integration-aws-py36_2 because the semaphore is not held | 14:46 |
| pabelanger[m] | I don't believe I seen that before | 14:47 |
| pabelanger[m] | but checking | 14:47 |
| pabelanger[m] | I am wondering if that means we leaked the semaphore some how? | 14:47 |
| opendevreview | lotorev vitaly proposed zuul/zuul master: docs: Replace job to branches in job.branches definition https://review.opendev.org/c/zuul/zuul/+/803105 | 14:49 |
| opendevreview | lotorev vitaly proposed zuul/zuul master: docs: Replace job to branches in job.branches definition https://review.opendev.org/c/zuul/zuul/+/803105 | 14:52 |
| corvus | swest: kazoo change lgtm -- is it possible to add a test for that? | 15:26 |
| swest | corvus: yeah I was thinking about a simplified version of the code from the bug report. Only thing I don't like about that is the time.sleep() | 15:28 |
| corvus | swest: there are some sleeps in kazoo's tests, but maybe you could use a threading.Event | 15:33 |
| swest | Yep. Or just wait for the individual lock contenders to show up. I'll come up with something | 15:34 |
| corvus | swest: that should work | 15:35 |
| corvus | i like that better | 15:35 |
| *** jgwentworth is now known as melwitt | 16:13 | |
| opendevreview | James E. Blair proposed zuul/zuul master: Execute merge jobs via ZooKeeper https://review.opendev.org/c/zuul/zuul/+/802299 | 20:01 |
| corvus | felixedel: ^ i squashed my updates into that since you okay'd them | 20:02 |
| corvus | felixedel: i'm going to abandon https://review.opendev.org/802971 but see the comments there | 20:02 |
| opendevreview | lotorev vitaly proposed zuul/zuul master: docs: Clarify job.branches definition https://review.opendev.org/c/zuul/zuul/+/803105 | 20:10 |
| opendevreview | Paul Belanger proposed zuul/zuul master: WIP: fix tenant-conf-check for duplicate projects https://review.opendev.org/c/zuul/zuul/+/591868 | 20:50 |
| opendevreview | James E. Blair proposed zuul/zuul master: Execute merge jobs via ZooKeeper https://review.opendev.org/c/zuul/zuul/+/802299 | 21:01 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!