*** brinzhang has joined #openstack-oslo | 00:51 | |
*** brinzhang_ has joined #openstack-oslo | 02:34 | |
*** brinzhang has quit IRC | 02:37 | |
*** dave-mccowan has quit IRC | 02:44 | |
*** brinzhang0 has joined #openstack-oslo | 03:04 | |
*** brinzhang_ has quit IRC | 03:07 | |
*** Jorhson has joined #openstack-oslo | 03:23 | |
*** Jorhson has quit IRC | 03:31 | |
*** brinzhang_ has joined #openstack-oslo | 05:04 | |
*** brinzhang0 has quit IRC | 05:06 | |
*** brinzhang0 has joined #openstack-oslo | 05:54 | |
*** brinzhang_ has quit IRC | 05:57 | |
*** hberaud has joined #openstack-oslo | 07:36 | |
*** dtantsur|afk is now known as dtantsur | 07:55 | |
*** tosky has joined #openstack-oslo | 07:56 | |
*** brinzhang_ has joined #openstack-oslo | 08:04 | |
*** brinzhang0 has quit IRC | 08:07 | |
*** Jorhson has joined #openstack-oslo | 09:02 | |
*** tkajinam has quit IRC | 09:55 | |
*** Luzi has joined #openstack-oslo | 09:59 | |
*** brinzhang_ has quit IRC | 10:04 | |
*** brinzhang_ has joined #openstack-oslo | 10:04 | |
*** brinzhang_ has quit IRC | 10:06 | |
*** brinzhang_ has joined #openstack-oslo | 10:06 | |
*** tkajinam has joined #openstack-oslo | 11:27 | |
*** brinzhang0 has joined #openstack-oslo | 11:34 | |
*** brinzhang_ has quit IRC | 11:37 | |
*** dave-mccowan has joined #openstack-oslo | 11:39 | |
*** raildo has joined #openstack-oslo | 11:49 | |
*** brinzhang_ has joined #openstack-oslo | 12:14 | |
*** brinzhang0 has quit IRC | 12:17 | |
*** hberaud has quit IRC | 12:20 | |
*** hberaud has joined #openstack-oslo | 12:21 | |
*** lbragstad has quit IRC | 12:32 | |
*** dave-mccowan has quit IRC | 12:32 | |
*** dhellmann has joined #openstack-oslo | 12:35 | |
*** hberaud has quit IRC | 12:36 | |
*** hberaud has joined #openstack-oslo | 12:39 | |
*** dave-mccowan has joined #openstack-oslo | 12:42 | |
*** lbragstad has joined #openstack-oslo | 12:43 | |
*** hberaud has quit IRC | 13:08 | |
*** kgiusti has joined #openstack-oslo | 13:11 | |
openstackgerrit | Merged openstack/oslo.versionedobjects master: zuul: port the legacy multinode grenade job https://review.opendev.org/745183 | 13:16 |
---|---|---|
*** Luzi has quit IRC | 13:26 | |
openstackgerrit | Luigi Toscano proposed openstack/oslo.versionedobjects stable/ussuri: zuul: port the legacy multinode grenade job https://review.opendev.org/746506 | 13:28 |
*** dtantsur has quit IRC | 13:32 | |
*** brinzhang0 has joined #openstack-oslo | 13:33 | |
*** hberaud has joined #openstack-oslo | 13:34 | |
*** brinzhang0 has quit IRC | 13:35 | |
*** brinzhang0 has joined #openstack-oslo | 13:35 | |
*** brinzhang_ has quit IRC | 13:36 | |
*** dtantsur has joined #openstack-oslo | 13:38 | |
*** hberaud has quit IRC | 13:43 | |
*** hberaud has joined #openstack-oslo | 13:57 | |
*** moguimar has joined #openstack-oslo | 14:02 | |
*** tkajinam has quit IRC | 14:06 | |
*** brinzhang has joined #openstack-oslo | 14:25 | |
*** brinzhang0 has quit IRC | 14:27 | |
*** brinzhang_ has joined #openstack-oslo | 14:34 | |
*** brinzhang has quit IRC | 14:37 | |
*** purplerbot has quit IRC | 15:00 | |
*** purplerbot has joined #openstack-oslo | 15:01 | |
moguimar | bnemec, meeting? | 15:02 |
bnemec | #startmeeting oslo | 15:02 |
bnemec | Courtesy ping for bnemec, smcginnis, moguimar, johnsom, stephenfin, bcafarel, kgiusti, jungleboyj | 15:02 |
bnemec | #link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting | 15:02 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
*** openstack changes topic to " (Meeting topic: oslo)" | 15:02 | |
openstack | The meeting name has been set to 'oslo' | 15:02 |
bnemec | I hate Mondays. :-P | 15:02 |
hberaud | o/ (here with a really unustable internet conection which back and worth) | 15:03 |
johnsom | o/ | 15:03 |
*** brinzhang0 has joined #openstack-oslo | 15:04 | |
moguimar | o/ | 15:04 |
kgiusti | o/ | 15:04 |
bnemec | #topic Red flags for/from liaisons | 15:06 |
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)" | 15:06 | |
moguimar | none from Barbican | 15:06 |
smcginnis | o/ | 15:06 |
johnsom | Nothing to report from Octavia today | 15:06 |
moguimar | I 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 IRC | 15:06 | |
bnemec | Last week was pretty quiet, so not much to report from our side either. | 15:07 |
bnemec | moguimar: Sounds good | 15:07 |
bnemec | #topic Releases | 15:07 |
*** openstack changes topic to "Releases (Meeting topic: oslo)" | 15:07 | |
bnemec | I think I said I was going to do these last week and then didn't. :-/ | 15:08 |
bnemec | Which may have contributed to the quiet week since nothing new got released. ;-) | 15:08 |
bnemec | So 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 meeting | 15: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 |
bnemec | Done | 15:10 |
bnemec | And it spawned a bunch of good discussion, so that was a good action. :-) | 15:10 |
bnemec | #topic zuulv3 migration | 15:10 |
*** openstack changes topic to "zuulv3 migration (Meeting topic: oslo)" | 15:11 | |
bnemec | I saw a patch merged earlier. | 15:11 |
bnemec | Looks 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-migration | 15:11 |
tosky | pbr is mostly done | 15:12 |
tosky | the new jobs have been merged, there is just one patch (now running again) to remove the legacy jobs (now unused) | 15:12 |
bnemec | So basically we've just got https://review.opendev.org/#/c/745192/ for the migration to be complete? | 15:12 |
tosky | correct - and the retirement of the zmq repo for real | 15:13 |
tosky | bonus point for the patches being backported, but that's not just part of the stretch goal | 15:13 |
bnemec | Yeah, but at least in that case if something breaks with the legacy job, nobody cares. :-) | 15:13 |
tosky | summary: oslo is in a good shape | 15:13 |
bnemec | Cool, thanks tosky and everyone else who has helped out with this! | 15:14 |
*** jungleboyj has joined #openstack-oslo | 15:14 | |
bnemec | #topic oslo.messaging ping endpoint | 15:15 |
*** openstack changes topic to "oslo.messaging ping endpoint (Meeting topic: oslo)" | 15:15 | |
hberaud | is this discussion is related to the ping endpoint? | 15:16 |
bnemec | As I mentioned, there was quite a bit more discussion on this on the list last week. | 15:16 |
kgiusti | Yep - an internal ping endpoint provided by the library itself for every RPC server instance | 15:17 |
bnemec | Because 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 |
hberaud | ack | 15:18 |
bnemec | Essentially Nova is only checking communication in one direction. This would allow operators to check both sides of the bus. | 15:18 |
kgiusti | Hrmm... I got the sense that the nova folks were not all that thrilled about introducing this traffic on the message bus. | 15:18 |
bnemec | As 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 |
bnemec | kgiusti: True, but as I noted there's no way to keep operators off the rabbit bus if they want to be there. | 15:19 |
bnemec | I'd rather give them a well-defined, consistent endpoint than have them each implementing their own version of it. | 15:19 |
bnemec | According to ttx, OVH is already doing this successfully. | 15:20 |
kgiusti | bnemec: I'd like to at least reserve the ping target address - using "ping" as the RPC method will break existing projects. | 15:21 |
bnemec | I 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 |
bnemec | But that's a long and frustrating path to go down. | 15:21 |
bnemec | kgiusti: That's fair. | 15:21 |
bnemec | I 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 |
kgiusti | bnemec: +1 | 15:22 |
kgiusti | bnemec: as the patch stands now I'm pretty sure it will break stuff | 15:22 |
bnemec | The weird thing is it passed tempest. | 15:22 |
bnemec | But like I said, maybe it just broke something tempest doesn't cover. | 15:22 |
kgiusti | bnemec: possibly - I'm just repeating Nova's claim that they use "ping" address internally | 15:23 |
bnemec | Yeah, I don't know much about that. We'll need to work with them to make sure we don't overlap here. | 15:24 |
kgiusti | bnemec: I'd also like to have a ping client added to the oslo.messaging package at some point | 15:24 |
bnemec | That seems reasonable. | 15:24 |
bnemec | If 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 |
kgiusti | bnemec: yeah, worth a follow up patch. | 15:25 |
bnemec | Both ends of the transaction would still be under our control. | 15:25 |
bnemec | Okay, I like that. | 15:25 |
kgiusti | bnemec: but I'm going to review the patch again and -1 until we address the "ping" concern. | 15:25 |
kgiusti | my action for this week :) | 15:25 |
bnemec | Sounds good! | 15:26 |
bnemec | #action kgiusti to review the ping endpoint patch again | 15:26 |
bnemec | Do you also want to raise the suggestion about adding a client? | 15:26 |
bnemec | Maybe on the list so it's clear that we aren't blocking the current patch on that? | 15:26 |
kgiusti | bnemec: sure will do | 15:27 |
bnemec | Great, thanks! | 15:27 |
bnemec | Okay, 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 |
moguimar | yup | 15:29 |
moguimar | just a review | 15:29 |
moguimar | https://review.opendev.org/#/c/746130/ | 15:29 |
moguimar | this is a first patch and I'll reenable memcached functional tests in the next one | 15:29 |
moguimar | just need some more pushing | 15:29 |
openstackgerrit | Clark Boylan proposed openstack/pbr master: Increase integration test timeout https://review.opendev.org/746547 | 15:29 |
bnemec | That looks pretty easy. | 15:30 |
bnemec | Approved. | 15:30 |
bnemec | I saw the discussion about that last week and I agree with the reasoning to not force a Python version. | 15:31 |
bnemec | Thanks for doing that! | 15:31 |
moguimar | thanks | 15: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 | |
moguimar | see ya | 15:31 |
*** moguimar has quit IRC | 15:31 | |
bnemec | We tabled this one last week to wait for hberaud's input. | 15:32 |
bnemec | This 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 |
bnemec | It'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 |
hberaud | AFAIK we don't got bad feedback since these changes have been introduced so +1 | 15:33 |
bnemec | Okay, I think that was the consensus last week as well. | 15:34 |
hberaud | I'll submit a patch | 15:34 |
bnemec | Initially I think we want to just flip the default for the option and deprecate the opt. | 15:34 |
hberaud | ack | 15:34 |
bnemec | That way if it does break someone they can turn it off again. | 15:34 |
bnemec | Then in a cycle or two we can remove the option completely. | 15:35 |
hberaud | make sense | 15:35 |
bnemec | #action hberaud to send patch to enable heartbeat fix by default | 15:35 |
hberaud | I'll put a debtcollector warn too to higlight the future removal | 15:35 |
bnemec | hberaud: There's functionality in oslo.config to mark an opt as deprecated for removal. | 15:36 |
hberaud | ah yes right | 15:36 |
bnemec | I guess we'll need a release note and everything too. | 15:37 |
hberaud | yep | 15:37 |
bnemec | #topic Weekly Wayward Review | 15:39 |
bnemec | #link https://review.opendev.org/#/c/671309 | 15:39 |
*** openstack changes topic to "Weekly Wayward Review (Meeting topic: oslo)" | 15:39 | |
bnemec | I just WIP'd that because it's actually in merge conflict. | 15:39 |
bnemec | That's not a super useful action though. Let's see if we can find a patch that we can make progress on. | 15:40 |
bnemec | I think we're waiting for extra testing on https://review.opendev.org/#/c/695527/ | 15:40 |
hberaud | bnemec: I'll try to spend time on it tomorrow | 15:40 |
bnemec | Okay, there we go: https://review.opendev.org/#/c/734488 | 15:41 |
bnemec | Approved. :-) | 15:41 |
hberaud | thanks | 15:41 |
bnemec | hberaud: There is a comment on https://review.opendev.org/#/c/734453/ that would be nice to address before merging that. | 15:42 |
tosky | we mentioned https://review.opendev.org/#/c/745192/ before; it failed, but there is a potential fix: https://review.opendev.org/746547 | 15:42 |
bnemec | tosky: Approved | 15:43 |
openstackgerrit | Hervé Beraud proposed openstack/openstack-doc-tools master: Use unittest.mock instead of mock https://review.opendev.org/734453 | 15:43 |
hberaud | bnemec: ^^ | 15:44 |
bnemec | Okay, we've cleaned up some old patches, so that should do it for this section. | 15:44 |
bnemec | #topic Open discussion | 15:44 |
*** openstack changes topic to "Open discussion (Meeting topic: oslo)" | 15:44 | |
bnemec | hberaud: Thanks! | 15:44 |
bnemec | And that one's approved too now. :-) | 15:45 |
bnemec | Merge all the things! | 15:45 |
hberaud | bnemec: thanks | 15:45 |
hberaud | :) | 15:45 |
bnemec | Okay, was there anything else before we call it this week? | 15:45 |
hberaud | I just came back from PTO so nope... nothing on my side | 15:45 |
bnemec | Sounds like we're good then. | 15:48 |
bnemec | Thanks for joining everyone! | 15:48 |
bnemec | #endmeeting | 15:48 |
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo" | 15:48 | |
hberaud | bnemec: thanks | 15:48 |
openstack | Meeting ended Mon Aug 17 15:48:39 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:48 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-08-17-15.02.html | 15:48 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-08-17-15.02.txt | 15:48 |
openstack | Log: http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-08-17-15.02.log.html | 15:48 |
*** hberaud has quit IRC | 15:52 | |
*** hberaud has joined #openstack-oslo | 15:55 | |
openstackgerrit | Merged openstack/openstack-doc-tools master: Use unittest.mock instead of mock https://review.opendev.org/734453 | 16:07 |
openstackgerrit | Merged openstack/oslo.cache master: Drop hard choice on py38 for functional tests https://review.opendev.org/746130 | 16:24 |
*** brinzhang_ has joined #openstack-oslo | 16:38 | |
*** brinzhang0 has quit IRC | 16:42 | |
*** dtantsur is now known as dtantsur|afk | 16:59 | |
openstackgerrit | Merged openstack/oslo.context master: drop mock from lower-constraints https://review.opendev.org/734488 | 17:11 |
*** brinzhang0 has joined #openstack-oslo | 17:37 | |
*** brinzhang_ has quit IRC | 17:40 | |
*** hoonetorg has joined #openstack-oslo | 18:29 | |
openstackgerrit | Merged openstack/pbr master: Increase integration test timeout https://review.opendev.org/746547 | 18:58 |
*** smcginnis has quit IRC | 19:19 | |
*** smcginnis has joined #openstack-oslo | 19:21 | |
*** cognifloyd has joined #openstack-oslo | 19:39 | |
*** kgiusti has left #openstack-oslo | 20:37 | |
*** raildo has quit IRC | 21:22 | |
openstackgerrit | Merged openstack/pbr master: Cleanup old legacy devstack-gate jobs https://review.opendev.org/745192 | 21:34 |
*** brinzhang_ has joined #openstack-oslo | 21:44 | |
*** brinzhang0 has quit IRC | 21:47 | |
*** rcernin has joined #openstack-oslo | 22:33 | |
*** tosky has quit IRC | 22:57 | |
*** tkajinam has joined #openstack-oslo | 23:07 | |
*** brinzhang0 has joined #openstack-oslo | 23:09 | |
*** brinzhang_ has quit IRC | 23:12 | |
*** Dmitrii-Sh has quit IRC | 23:37 | |
*** Dmitrii-Sh has joined #openstack-oslo | 23:39 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!