15:00:07 <hberaud> #startmeeting oslo
15:00:08 <openstack> Meeting started Mon May 17 15:00:07 2021 UTC and is due to finish in 60 minutes.  The chair is hberaud. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:10 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:13 <hberaud> Courtesy ping for hberaud, stephenfin, moguimar, jungleboyj, bnemec, johnsom, bcafarel, kgiusti, gmann, valleedelisle, sboyron, damani
15:00:13 <openstack> The meeting name has been set to 'oslo'
15:00:17 <hberaud> #link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting
15:00:21 <hberaud> Will just wait a couple minutes for folks.
15:00:25 <sboyron> o/
15:00:36 <jungleboyj> o/
15:01:06 <bnemec> o/
15:01:55 <johnsom> o/
15:02:40 <kgiusti> o/
15:03:25 <bcafarel> o/
15:04:07 <hberaud> #topic Red flags for/from liaisons
15:04:20 <johnsom> Nothing to report from the Octavia/Designate side
15:04:41 <jungleboyj> Nothing that I am aware of from Cinder.
15:04:56 <bcafarel> I was away last week but I did not hear major complaints from Neutron
15:05:08 <hberaud> Nothing that I'm aware of from Oslo
15:05:53 <johnsom> lol
15:06:43 <hberaud> #topic Releases liaison
15:07:31 <moguimar> late me o/
15:09:06 <hberaud> We released a couple of bugfix versions for stable series and independent projects (oslo.log, tooz, etc...). I don't have much more topics to speak about here.
15:09:10 <hberaud> moguimar: o/
15:09:19 <hberaud> #topic Security liaison
15:09:35 <hberaud> bnemec: The floor is yours
15:10:07 <bnemec> Nothing from me. We didn't get tapped in the last round of security threads on the list. :-)
15:10:19 <hberaud> thank you bnemec
15:10:25 <hberaud> #topic TaCT SIG liaison
15:10:34 <hberaud> moguimar, sboyron: The floor is yours
15:11:02 <moguimar> no updates
15:11:29 <hberaud> thank you moguimar
15:11:32 <hberaud> #topic Action items from last meeting
15:11:46 <hberaud> No actions
15:12:00 <hberaud> #topic Weekly Wayward Wallaby Review
15:12:35 <hberaud> https://review.opendev.org/c/openstack/oslo.messaging/+/787384
15:12:46 <hberaud> Just need a final +W
15:12:54 <hberaud> (queens backport)
15:13:01 <moguimar> checking it
15:13:47 <hberaud> Could be worth to land it before the next round of EOL branches (pike, rocky, queens)
15:14:15 <hberaud> thank moguimar
15:14:28 <moguimar> do you know why didin't they do msg_id="" in the original patch?
15:15:03 <moguimar> instead of name=msg_id -> name=""
15:15:09 <hberaud> yes, to avoid failover issue IIRC
15:15:09 <moguimar> feels too hardcoded
15:15:20 <hberaud> Ah I see
15:15:50 <hberaud> Nope I don't remember
15:16:23 <hberaud> That's a cherry-pick
15:16:24 <moguimar> ok, +W
15:16:31 <hberaud> Thanks
15:16:36 <bnemec> You can't have a default on the first parameter without also setting one for the second.
15:16:38 <moguimar> was it also backported to other branches?
15:16:46 <hberaud> Yes
15:16:58 <moguimar> I can see only queens and pike
15:17:09 <hberaud> If I'm right it was merged since stein
15:17:10 <moguimar> and original patch March 2020
15:17:29 <hberaud> let me double check too
15:21:44 <hberaud> so, by using `git branch --remotes --contains 3a5de89` I can find it on stein/train/ussuri/victoria/wallaby
15:21:47 <hberaud> and master
15:23:09 <hberaud> It was also backported to rocky https://review.opendev.org/c/openstack/oslo.messaging/+/713153
15:23:14 <hberaud> so queens LGTM
15:23:57 <moguimar> ouky
15:24:13 <hberaud> thanks moguimar
15:24:24 <hberaud> #topic Courtesy ping
15:26:05 <hberaud> Just a friendly reminder about the future courtesy ping list, next meeting I'll use the new one so don't forget to put your name in if you want to continue to receive notifications https://wiki.openstack.org/wiki/Meetings/Oslo
15:26:18 <hberaud> #topic Open discussion
15:26:40 <bnemec> I've seen a bunch of pings about https://bugs.launchpad.net/oslo.db/+bug/1896916 this morning.
15:26:40 <openstack> Launchpad bug 1896916 in oslo.db "duplicate key error for mysql >=8.0.19 are not parsed correctly" [Undecided,New]
15:26:47 <ajya> o/
15:26:50 <hberaud> Anything else that you want to discuss today?
15:27:00 <ajya> yes, I have Q about duplicate key thing
15:27:07 <bnemec> I think there's just a regex that probably needs to be updated, based on previous similar bugs.
15:27:08 <ajya> Are there plans to fix it as it has been opened for a while?
15:27:44 <hberaud> ajya: I don't know
15:27:53 <bnemec> Looks like it has been open for a while.
15:27:57 <hberaud> yes
15:28:57 <hberaud> Without too digging either, I think that some code around here will fix the issue => https://opendev.org/openstack/oslo.db/src/branch/master/oslo_db/sqlalchemy/exc_filters.py
15:29:09 <bnemec> zzzeek might have some input. It's possible this has already come up with sqlalchemy too.
15:29:37 <hberaud> Yes that what I was thinking too
15:29:39 <bnemec> If we can avoid writing a new regex and reuse one from somewhere else that would be ideal. :-)
15:30:04 <ajya> should we propose a patch or somebody else will take this?
15:30:36 <hberaud> ajya: AFAIK nobody work on that, so feel free to propose a patch
15:30:51 <ajya> hberaud: ack
15:32:43 <hberaud> Also, if the problem is already fixed on sqlalchemy this implies that the version is supported by openstack/requirements
15:33:41 <hberaud> and I far as I know the latest upgrade isn't yet landed https://review.opendev.org/c/openstack/requirements/+/788339
15:34:09 <hberaud> so we still rely on sqlalchemy <1.4
15:35:45 <hberaud> ajya: before submitting a patch can you do some tests with sqlalchemy > 1.4?
15:35:57 <ajya> hberaud: ok, will check
15:35:59 <hberaud> and see if that fix the problem?
15:37:07 <hberaud> I would argue that the requirement bump will be merged soon, so, possibly as said bnemec that will allow us to avoid duplicate code
15:37:22 <hberaud> *duplicated
15:38:00 <hberaud> Anything else?
15:38:43 <hberaud> ajya: let us know if that helped you
15:38:55 <ajya> ok, thank you
15:39:13 <hberaud> ajya: thank you
15:41:00 <hberaud> OK, thanks everyone. Let's wrap up.
15:41:03 <hberaud> #endmeeting