Monday, 2020-08-17

*** brinzhang has joined #openstack-oslo00:51
*** brinzhang_ has joined #openstack-oslo02:34
*** brinzhang has quit IRC02:37
*** dave-mccowan has quit IRC02:44
*** brinzhang0 has joined #openstack-oslo03:04
*** brinzhang_ has quit IRC03:07
*** Jorhson has joined #openstack-oslo03:23
*** Jorhson has quit IRC03:31
*** brinzhang_ has joined #openstack-oslo05:04
*** brinzhang0 has quit IRC05:06
*** brinzhang0 has joined #openstack-oslo05:54
*** brinzhang_ has quit IRC05:57
*** hberaud has joined #openstack-oslo07:36
*** dtantsur|afk is now known as dtantsur07:55
*** tosky has joined #openstack-oslo07:56
*** brinzhang_ has joined #openstack-oslo08:04
*** brinzhang0 has quit IRC08:07
*** Jorhson has joined #openstack-oslo09:02
*** tkajinam has quit IRC09:55
*** Luzi has joined #openstack-oslo09:59
*** brinzhang_ has quit IRC10:04
*** brinzhang_ has joined #openstack-oslo10:04
*** brinzhang_ has quit IRC10:06
*** brinzhang_ has joined #openstack-oslo10:06
*** tkajinam has joined #openstack-oslo11:27
*** brinzhang0 has joined #openstack-oslo11:34
*** brinzhang_ has quit IRC11:37
*** dave-mccowan has joined #openstack-oslo11:39
*** raildo has joined #openstack-oslo11:49
*** brinzhang_ has joined #openstack-oslo12:14
*** brinzhang0 has quit IRC12:17
*** hberaud has quit IRC12:20
*** hberaud has joined #openstack-oslo12:21
*** lbragstad has quit IRC12:32
*** dave-mccowan has quit IRC12:32
*** dhellmann has joined #openstack-oslo12:35
*** hberaud has quit IRC12:36
*** hberaud has joined #openstack-oslo12:39
*** dave-mccowan has joined #openstack-oslo12:42
*** lbragstad has joined #openstack-oslo12:43
*** hberaud has quit IRC13:08
*** kgiusti has joined #openstack-oslo13:11
openstackgerritMerged openstack/oslo.versionedobjects master: zuul: port the legacy multinode grenade job  https://review.opendev.org/74518313:16
*** Luzi has quit IRC13:26
openstackgerritLuigi Toscano proposed openstack/oslo.versionedobjects stable/ussuri: zuul: port the legacy multinode grenade job  https://review.opendev.org/74650613:28
*** dtantsur has quit IRC13:32
*** brinzhang0 has joined #openstack-oslo13:33
*** hberaud has joined #openstack-oslo13:34
*** brinzhang0 has quit IRC13:35
*** brinzhang0 has joined #openstack-oslo13:35
*** brinzhang_ has quit IRC13:36
*** dtantsur has joined #openstack-oslo13:38
*** hberaud has quit IRC13:43
*** hberaud has joined #openstack-oslo13:57
*** moguimar has joined #openstack-oslo14:02
*** tkajinam has quit IRC14:06
*** brinzhang has joined #openstack-oslo14:25
*** brinzhang0 has quit IRC14:27
*** brinzhang_ has joined #openstack-oslo14:34
*** brinzhang has quit IRC14:37
*** purplerbot has quit IRC15:00
*** purplerbot has joined #openstack-oslo15:01
moguimarbnemec, meeting?15:02
bnemec#startmeeting oslo15:02
bnemecCourtesy ping for bnemec, smcginnis, moguimar, johnsom, stephenfin, bcafarel, kgiusti, jungleboyj15:02
bnemec#link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting15:02
openstackMeeting started Mon Aug 17 15:02:46 2020 UTC and is due to finish in 60 minutes.  The chair is bnemec. Information about MeetBot at http://wiki.debian.org/MeetBot.15:02
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:02
*** openstack changes topic to " (Meeting topic: oslo)"15:02
openstackThe meeting name has been set to 'oslo'15:02
bnemecI hate Mondays. :-P15:02
hberaudo/ (here with a really unustable internet conection which back and worth)15:03
johnsomo/15:03
*** brinzhang0 has joined #openstack-oslo15:04
moguimaro/15:04
kgiustio/15:04
bnemec#topic Red flags for/from liaisons15:06
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)"15:06
moguimarnone from Barbican15:06
smcginniso/15:06
johnsomNothing to report from Octavia today15:06
moguimarI have a hard stop at half past the hour, so this is going to be a quick meeting (at least on my side)15:06
*** brinzhang_ has quit IRC15:06
bnemecLast week was pretty quiet, so not much to report from our side either.15:07
bnemecmoguimar: Sounds good15:07
bnemec#topic Releases15:07
*** openstack changes topic to "Releases (Meeting topic: oslo)"15:07
bnemecI think I said I was going to do these last week and then didn't. :-/15:08
bnemecWhich may have contributed to the quiet week since nothing new got released. ;-)15:08
bnemecSo we'll need to get that done this week. Nothing much else to say on this topic.15:09
bnemec#topic Action items from last meeting15:09
*** openstack changes topic to "Action items from last meeting (Meeting topic: oslo)"15:09
bnemec"bnemec to reply to rpc ping thread with results of meeting discussion"15:10
bnemecDone15:10
bnemecAnd it spawned a bunch of good discussion, so that was a good action. :-)15:10
bnemec#topic  zuulv3 migration15:10
*** openstack changes topic to "zuulv3 migration (Meeting topic: oslo)"15:11
bnemecI saw a patch merged earlier.15:11
bnemecLooks like we just have zmq, which we don't care about and are retiring, and pbr left on the list.15:11
bnemec#link https://etherpad.opendev.org/p/goal-victoria-native-zuulv3-migration15:11
toskypbr is mostly done15:12
toskythe new jobs have been merged, there is just one patch (now running again) to remove the legacy jobs (now unused)15:12
bnemecSo basically we've just got https://review.opendev.org/#/c/745192/ for the migration to be complete?15:12
toskycorrect - and the retirement of the zmq repo for real15:13
toskybonus point for the patches being backported, but that's not just part of the stretch goal15:13
bnemecYeah, but at least in that case if something breaks with the legacy job, nobody cares. :-)15:13
toskysummary: oslo is in a good shape15:13
bnemecCool, thanks tosky and everyone else who has helped out with this!15:14
*** jungleboyj has joined #openstack-oslo15:14
bnemec#topic oslo.messaging ping endpoint15:15
*** openstack changes topic to "oslo.messaging ping endpoint (Meeting topic: oslo)"15:15
hberaudis this discussion is related to the ping endpoint?15:16
bnemecAs I mentioned, there was quite a bit more discussion on this on the list last week.15:16
kgiustiYep - an internal ping endpoint provided by the library itself for every RPC server instance15:17
bnemecBecause of the way the Nova service healthchecking works, I'm more inclined to go ahead with this change after getting some more details about the use cases involved.15:17
hberaudack15:18
bnemecEssentially Nova is only checking communication in one direction. This would allow operators to check both sides of the bus.15:18
kgiustiHrmm... I got the sense that the nova folks were not all that thrilled about introducing this traffic on the message bus.15:18
bnemecAs I said on the list, I'd still eventually like us to have proper healthchecks for all services so that we don't need external stuff like this, but since there's been little movement on that front this is a decent stopgap.15:19
bnemeckgiusti: True, but as I noted there's no way to keep operators off the rabbit bus if they want to be there.15:19
bnemecI'd rather give them a well-defined, consistent endpoint than have them each implementing their own version of it.15:19
bnemecAccording to ttx, OVH is already doing this successfully.15:20
kgiustibnemec: I'd like to at least reserve the ping target address - using "ping" as the RPC method will break existing projects.15:21
bnemecI mean, we can play whack-a-mole with all of the potential healthcheck gaps in services, like the one we discovered in Nova as part of this discussion.15:21
bnemecBut that's a long and frustrating path to go down.15:21
bnemeckgiusti: That's fair.15:21
bnemecI think it's reasonable to start talking about the best way to implement this, as opposed to whether to implement it at all.15:22
kgiustibnemec: +115:22
kgiustibnemec: as the patch stands now I'm pretty sure it will break stuff15:22
bnemecThe weird thing is it passed tempest.15:22
bnemecBut like I said, maybe it just broke something tempest doesn't cover.15:22
kgiustibnemec: possibly - I'm just repeating Nova's claim that they use "ping" address internally15:23
bnemecYeah, I don't know much about that. We'll need to work with them to make sure we don't overlap here.15:24
kgiustibnemec: I'd also like to have a ping client added to the oslo.messaging package at some point15:24
bnemecThat seems reasonable.15:24
bnemecIf oslo.messaging provided the client for doing the pings then it would mitigate a lot of the danger of having ops sending messages on the rabbit bus.15:24
kgiustibnemec: yeah, worth a follow up patch.15:25
bnemecBoth ends of the transaction would still be under our control.15:25
bnemecOkay, I like that.15:25
kgiustibnemec: but I'm going to review the patch again and -1 until we address the "ping" concern.15:25
kgiustimy action for this week :)15:25
bnemecSounds good!15:26
bnemec#action kgiusti to review the ping endpoint patch again15:26
bnemecDo you also want to raise the suggestion about adding a client?15:26
bnemecMaybe on the list so it's clear that we aren't blocking the current patch on that?15:26
kgiustibnemec: sure will do15:27
bnemecGreat, thanks!15:27
bnemecOkay, we have one more topic, but moguimar is coming up on time I believe. Is there anything you wanted to cover before you go?15:28
moguimaryup15:29
moguimarjust a review15:29
moguimarhttps://review.opendev.org/#/c/746130/15:29
moguimarthis is a first patch and I'll reenable memcached functional tests in the next one15:29
moguimarjust need some more pushing15:29
openstackgerritClark Boylan proposed openstack/pbr master: Increase integration test timeout  https://review.opendev.org/74654715:29
bnemecThat looks pretty easy.15:30
bnemecApproved.15:30
bnemecI saw the discussion about that last week and I agree with the reasoning to not force a Python version.15:31
bnemecThanks for doing that!15:31
moguimarthanks15:31
bnemec#topic enable oslo.messaging heartbeat fix by default?15:31
*** openstack changes topic to "enable oslo.messaging heartbeat fix by default? (Meeting topic: oslo)"15:31
moguimarsee ya15:31
*** moguimar has quit IRC15:31
bnemecWe tabled this one last week to wait for hberaud's input.15:32
bnemecThis is about the messaging heartbeat fix using native threads that we put behind an option to let affected users try it out without causing issues for everyone.15:33
bnemecIt's been in for a while now and I haven't heard any complaints, so I think it may be time to try turning it on by default.15:33
hberaudAFAIK we don't got bad feedback since these changes have been introduced so +115:33
bnemecOkay, I think that was the consensus last week as well.15:34
hberaudI'll submit a patch15:34
bnemecInitially I think we want to just flip the default for the option and deprecate the opt.15:34
hberaudack15:34
bnemecThat way if it does break someone they can turn it off again.15:34
bnemecThen in a cycle or two we can remove the option completely.15:35
hberaudmake sense15:35
bnemec#action hberaud to send patch to enable heartbeat fix by default15:35
hberaudI'll put a debtcollector warn too to higlight the future removal15:35
bnemechberaud: There's functionality in oslo.config to mark an opt as deprecated for removal.15:36
hberaudah yes right15:36
bnemecI guess we'll need a release note and everything too.15:37
hberaudyep15:37
bnemec#topic Weekly Wayward Review15:39
bnemec#link https://review.opendev.org/#/c/67130915:39
*** openstack changes topic to "Weekly Wayward Review (Meeting topic: oslo)"15:39
bnemecI just WIP'd that because it's actually in merge conflict.15:39
bnemecThat's not a super useful action though. Let's see if we can find a patch that we can make progress on.15:40
bnemecI think we're waiting for extra testing on https://review.opendev.org/#/c/695527/15:40
hberaudbnemec: I'll try to spend time on it tomorrow15:40
bnemecOkay, there we go: https://review.opendev.org/#/c/73448815:41
bnemecApproved. :-)15:41
hberaudthanks15:41
bnemechberaud: There is a comment on https://review.opendev.org/#/c/734453/ that would be nice to address before merging that.15:42
toskywe mentioned https://review.opendev.org/#/c/745192/ before; it failed, but there is a potential fix: https://review.opendev.org/74654715:42
bnemectosky: Approved15:43
openstackgerritHervĂ© Beraud proposed openstack/openstack-doc-tools master: Use unittest.mock instead of mock  https://review.opendev.org/73445315:43
hberaudbnemec: ^^15:44
bnemecOkay, we've cleaned up some old patches, so that should do it for this section.15:44
bnemec#topic Open discussion15:44
*** openstack changes topic to "Open discussion (Meeting topic: oslo)"15:44
bnemechberaud: Thanks!15:44
bnemecAnd that one's approved too now. :-)15:45
bnemecMerge all the things!15:45
hberaudbnemec: thanks15:45
hberaud:)15:45
bnemecOkay, was there anything else before we call it this week?15:45
hberaudI just came back from PTO so nope... nothing on my side15:45
bnemecSounds like we're good then.15:48
bnemecThanks for joining everyone!15:48
bnemec#endmeeting15:48
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo"15:48
hberaudbnemec: thanks15:48
openstackMeeting ended Mon Aug 17 15:48:39 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:48
openstackMinutes:        http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-08-17-15.02.html15:48
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-08-17-15.02.txt15:48
openstackLog:            http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-08-17-15.02.log.html15:48
*** hberaud has quit IRC15:52
*** hberaud has joined #openstack-oslo15:55
openstackgerritMerged openstack/openstack-doc-tools master: Use unittest.mock instead of mock  https://review.opendev.org/73445316:07
openstackgerritMerged openstack/oslo.cache master: Drop hard choice on py38 for functional tests  https://review.opendev.org/74613016:24
*** brinzhang_ has joined #openstack-oslo16:38
*** brinzhang0 has quit IRC16:42
*** dtantsur is now known as dtantsur|afk16:59
openstackgerritMerged openstack/oslo.context master: drop mock from lower-constraints  https://review.opendev.org/73448817:11
*** brinzhang0 has joined #openstack-oslo17:37
*** brinzhang_ has quit IRC17:40
*** hoonetorg has joined #openstack-oslo18:29
openstackgerritMerged openstack/pbr master: Increase integration test timeout  https://review.opendev.org/74654718:58
*** smcginnis has quit IRC19:19
*** smcginnis has joined #openstack-oslo19:21
*** cognifloyd has joined #openstack-oslo19:39
*** kgiusti has left #openstack-oslo20:37
*** raildo has quit IRC21:22
openstackgerritMerged openstack/pbr master: Cleanup old legacy devstack-gate jobs  https://review.opendev.org/74519221:34
*** brinzhang_ has joined #openstack-oslo21:44
*** brinzhang0 has quit IRC21:47
*** rcernin has joined #openstack-oslo22:33
*** tosky has quit IRC22:57
*** tkajinam has joined #openstack-oslo23:07
*** brinzhang0 has joined #openstack-oslo23:09
*** brinzhang_ has quit IRC23:12
*** Dmitrii-Sh has quit IRC23:37
*** Dmitrii-Sh has joined #openstack-oslo23:39

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