| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix race condition in the sink https://review.opendev.org/c/openstack/designate/+/814290 | 01:49 |
|---|---|---|
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix race condition in the sink https://review.opendev.org/c/openstack/designate/+/814290 | 01:52 |
| eandersson | johnsom is there a bug for the sink bug? | 01:56 |
| eandersson | ozzzo: I wrote a new, better, stronger patch :D if you have time can you test that one as well? | 01:58 |
| ozzzo | ok I'll try it tomorrow | 01:58 |
| eandersson | Thanks! | 02:00 |
| eandersson | There is still a small chance for a race condition with this code, but it should be very small. | 02:00 |
| eandersson | This was my hacky way of reproducing this https://paste.openstack.org/show/810063/ | 02:02 |
| johnsom | eandersson: I don’t remember and I can’t look right now. | 02:23 |
| eandersson | No worries. | 02:33 |
| eandersson | I'll see if I can find one. | 02:34 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix race condition in the sink https://review.opendev.org/c/openstack/designate/+/814290 | 02:34 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix race condition in the sink https://review.opendev.org/c/openstack/designate/+/814290 | 02:37 |
| eandersson | I actually don't see one. https://bugs.launchpad.net/designate/+bug/1940976 This one is a different issue | 02:40 |
| eandersson | ozzzo: Would you be able to create a bug for this so that I can attach it to the patch, if not I can do it for you. | 02:40 |
| ozzzo | I think I can; I'll have to ask my supervisor | 02:46 |
| ozzzo | They're very happy to have the problem fixed, but I still haven't received permission to engage with the community, so at this point I can only talk in general terms from my home computer | 02:47 |
| eandersson | I was in that boat for many years | 02:49 |
| eandersson | You can tell them that this would probably have gone unfixed for many years (it's a really old bug) if you hadn't reported it. | 03:30 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix race condition in the sink when deleting records https://review.opendev.org/c/openstack/designate/+/814290 | 03:44 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix race condition in the sink when deleting records https://review.opendev.org/c/openstack/designate/+/814290 | 03:45 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix race condition in the sink when deleting records https://review.opendev.org/c/openstack/designate/+/814290 | 05:35 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix race condition in the sink when deleting records https://review.opendev.org/c/openstack/designate/+/814290 | 05:36 |
| opendevreview | Arkady Shtempler proposed openstack/designate-tempest-plugin master: Adding negative test for a Primary user https://review.opendev.org/c/openstack/designate-tempest-plugin/+/799628 | 08:01 |
| opendevreview | Arkady Shtempler proposed openstack/designate-tempest-plugin master: Stabilizing “PTR record” tests suite and adding new test cases https://review.opendev.org/c/openstack/designate-tempest-plugin/+/794708 | 09:51 |
| opendevreview | Arkady Shtempler proposed openstack/designate-tempest-plugin master: New test cases for blacklist test suite https://review.opendev.org/c/openstack/designate-tempest-plugin/+/798715 | 10:05 |
| opendevreview | Arkady Shtempler proposed openstack/designate-tempest-plugin master: New test cases for quota tests suite https://review.opendev.org/c/openstack/designate-tempest-plugin/+/799079 | 10:14 |
| *** dkehn_ is now known as dkehn | 11:44 | |
| ozzzo | eandersson: I got permission to open a bug, but the logs will be a bit goofy because I have to obscure IPs and domain names | 13:41 |
| eandersson | That is fine ozzzo | 14:45 |
| ozzzo | eandersson: here's the bug: https://bugs.launchpad.net/kolla-ansible/+bug/1947765 | 16:19 |
| -opendevstatus- NOTICE: Both Gerrit and Zuul services are being restarted briefly for minor updates, and should return to service momentarily; all previously running builds will be reenqueued once Zuul is fully started again | 17:00 | |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix race condition in the sink when deleting records https://review.opendev.org/c/openstack/designate/+/814290 | 19:39 |
| ozzzo | eandersson: the new patch also works | 20:18 |
| opendevreview | Michael Johnson proposed openstack/designate-tempest-plugin master: Expand Designate RBAC testing - zones https://review.opendev.org/c/openstack/designate-tempest-plugin/+/807400 | 20:37 |
| eandersson | Awesome! Thank you ozzzo | 20:53 |
| eandersson | johnsom https://review.opendev.org/c/openstack/designate/+/814290 is ready for review btw | 21:24 |
| johnsom | ack | 22:00 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!