Thursday, 2023-11-09

opendevreviewMerged openstack/taskflow stable/2023.1: Fix parsing of zookeeper jobboard backend options  https://review.opendev.org/c/openstack/taskflow/+/88060505:24
opendevreviewMerged openstack/oslo.messaging stable/zed: Add is_admin to safe fields list for notifications  https://review.opendev.org/c/openstack/oslo.messaging/+/89642405:31
opendevreviewMerged openstack/oslo.messaging stable/yoga: Add is_admin to safe fields list for notifications  https://review.opendev.org/c/openstack/oslo.messaging/+/89642507:49
opendevreviewMerged openstack/oslo.messaging stable/xena: Add is_admin to safe fields list for notifications  https://review.opendev.org/c/openstack/oslo.messaging/+/89642609:10
opendevreviewThomas Goirand proposed openstack/oslo.cache master: Clean-up memcache connection sockets  https://review.opendev.org/c/openstack/oslo.cache/+/90015813:11
tkajinamhttps://releases.openstack.org/constraints/upper/master14:16
tkajinamugh... it seens the change messed up something14:16
fricklertkajinam: if you are talking about https://review.opendev.org/c/openstack/requirements/+/900517, that's because publishing on pypi is slower than zuul, so jobs run before they can find the new release on pypi14:34
fricklerbut this one looks real: TypeError: expected string or bytes-like object14:38
fricklerin nova and ironic, e.g. https://zuul.opendev.org/t/openstack/build/d063eda10bbc487eaad57ffa7476bec714:38
tkajinamyeah that error in ironic and nova looks real and we have to fix how they mock things14:39
tkajinamI reproduced the problem locally and the version argument is not a string but <_AutospecMagicMock name='objects.MyObj.VERSION' id='140525135731536'>14:39
tkajinamand thanks for the explanation about the slow pypi. let's see if we find further real failures after rechecks.14:40
tkajinamok I found the fix... let me report a bug and submit fixes14:54
*** carthaca_ is now known as carthaca15:02
opendevreviewTakashi Kajinami proposed openstack/oslo.cache master: Clean-up memcache connection sockets  https://review.opendev.org/c/openstack/oslo.cache/+/90015815:21
opendevreviewTakashi Kajinami proposed openstack/oslo.cache master: Clean-up memcache connection sockets  https://review.opendev.org/c/openstack/oslo.cache/+/90015815:22
opendevreviewThomas Goirand proposed openstack/oslo.cache master: Clean-up memcache connection sockets  https://review.opendev.org/c/openstack/oslo.cache/+/90015815:25
opendevreviewTakashi Kajinami proposed openstack/oslo.cache master: Clean-up memcache connection sockets  https://review.opendev.org/c/openstack/oslo.cache/+/90015815:35
tkajinamzigo, sorry seems our updates conflicted. See the updated version I just submitted. I think even your latest version does not work as intended ^^^15:35
zigotkajinam: Please overwrite my patch ! :)15:35
zigoI'm writing shit ...15:35
zigoYou're not writing a loop though ...15:36
tkajinamI believe it has while True at top15:37
zigoThanks for picking-it up. It's a real trouble in production, that one...15:37
zigoOh, indeed, got it.15:37
zigowhile True: with except break, got it.15:37
tkajinamthe loop ends when queue is empty because pop raises IndexError15:38
tkajinamzigo, do you mind reporting a bug ? I think it's very useful to have some explanation about the issue15:38
zigoOk.15:38
tkajinamhttps://bugs.launchpad.net/oslo.cache15:38
zigotkajinam: https://bugs.launchpad.net/oslo.cache/+bug/204312115:46
opendevreviewTakashi Kajinami proposed openstack/oslo.cache master: Clean-up memcache connection sockets  https://review.opendev.org/c/openstack/oslo.cache/+/90015815:48
tkajinamzigo, ok, thanks. added the tag to link the fix to that bug15:48
*** tosky_ is now known as tosky18:43

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!