| opendevreview | Adam Harwell proposed openstack/tooz master: redis: Translate errors of group operations https://review.opendev.org/c/openstack/tooz/+/1007611 | 07:59 |
|---|---|---|
| opendevreview | Adam Harwell proposed openstack/tooz master: redis: Read groups without WATCH https://review.opendev.org/c/openstack/tooz/+/1007408 | 07:59 |
| opendevreview | Adam Harwell proposed openstack/tooz master: redis: Retry operations in one bounded layer https://review.opendev.org/c/openstack/tooz/+/1007940 | 07:59 |
| opendevreview | Adam Harwell proposed openstack/tooz master: redis: Recover after primary role changes https://review.opendev.org/c/openstack/tooz/+/1003880 | 07:59 |
| opendevreview | Adam Harwell proposed openstack/tooz master: coordination: Stop heartbeat retries during shutdown https://review.opendev.org/c/openstack/tooz/+/1009925 | 07:59 |
| opendevreview | Adam Harwell proposed openstack/tooz master: coordination: Stop heartbeat retries on shutdown https://review.opendev.org/c/openstack/tooz/+/1009925 | 08:00 |
| *** timburke_ is now known as timburke | 09:08 | |
| rm_work | tkajinam: heh I was going to message you once I really nailed down and retested these, but I do think a lot of them are ready… I was trying to do some final polish first before asking you to look | 17:15 |
| rm_work | I’ve been running with most of them in my deployment for a while now | 17:15 |
| rm_work | Addressing your comment, this one is the only patch I haven’t fully reviewed/tested yet because it just found that one during cleanup | 17:21 |
| rm_work | The other four we really do need and have had a bit of time to soak in our deployment | 17:22 |
| rm_work | Well… three of them. My deployment uses Sentinel so the 1003880 we don’t use but other people might run into it, we hit that issue before switching to sentinel | 17:23 |
| opendevreview | Adam Harwell proposed openstack/tooz master: coordination: Stop heartbeat retries on shutdown https://review.opendev.org/c/openstack/tooz/+/1009925 | 17:25 |
| opendevreview | Adam Harwell proposed openstack/tooz master: coordination: Stop heartbeat retries on shutdown https://review.opendev.org/c/openstack/tooz/+/1009925 | 17:34 |
| rm_work | OK that’s better I think | 17:38 |
| opendevreview | Adam Harwell proposed openstack/tooz master: redis: Translate errors of group operations https://review.opendev.org/c/openstack/tooz/+/1007611 | 17:47 |
| opendevreview | Adam Harwell proposed openstack/tooz master: redis: Read groups without WATCH https://review.opendev.org/c/openstack/tooz/+/1007408 | 17:47 |
| opendevreview | Adam Harwell proposed openstack/tooz master: redis: Retry operations in one bounded layer https://review.opendev.org/c/openstack/tooz/+/1007940 | 17:47 |
| opendevreview | Adam Harwell proposed openstack/tooz master: redis: Recover after primary role changes https://review.opendev.org/c/openstack/tooz/+/1003880 | 17:47 |
| rm_work | It’s all a stack now so a bit of rebase noise when I change the first one in the stack, sorry 😣 | 17:47 |
| rm_work | Found an annoying race failure while looking at this, posting a separate CR for it | 18:09 |
| rm_work | One thing I do love about LLM development is that when I find stuff like this, instead of “this sucks but I can’t spend hours tracking down a race” it basically can tell me exactly where it is and how to fix it and run a test suite 10,000 times to prove it failed 210 times before but 0 after the fix 😂 | 18:11 |
| rm_work | And it just handles it while I’m getting coffee | 18:11 |
| opendevreview | Adam Harwell proposed openstack/tooz master: ipc: Serialize shared memory lifecycle https://review.opendev.org/c/openstack/tooz/+/1009946 | 18:19 |
| tkajinam | it may be true for implementation but I've seen a lot, a lot of horrible test codes generated by AI and I'm hoping they improve for generating test code soon... | 18:21 |
| tkajinam | the ones you posted are much better, though | 18:22 |
| opendevreview | Adam Harwell proposed openstack/tooz master: redis: Read groups without WATCH https://review.opendev.org/c/openstack/tooz/+/1007408 | 18:43 |
| opendevreview | Adam Harwell proposed openstack/tooz master: redis: Retry operations in one bounded layer https://review.opendev.org/c/openstack/tooz/+/1007940 | 18:43 |
| opendevreview | Adam Harwell proposed openstack/tooz master: redis: Recover after primary role changes https://review.opendev.org/c/openstack/tooz/+/1003880 | 18:43 |
| rm_work | Addressed your other comments too I hope, double checking | 18:48 |
| tkajinam | no rush | 18:56 |
| tkajinam | just wanted to leave some comments while some contexts are loaded on my memory :-P | 18:56 |
| rm_work | Yeah I’d just really love to land some of these so I can stop tracking them… and because people will be running into this stuff if they try to use tooz lol | 19:30 |
| rm_work | Might want to backport at least to 2026.2 | 19:30 |
| rm_work | Maybe during PTG we can sync up, is there not an Oslo meetup? | 19:31 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!