| *** elodilles is now known as elodilles_pto | 04:59 | |
| elodilles_pto | melwitt: ack, thanks for the heads up! today i'm on PTO but will look after the patches when I'll have time! | 05:00 |
|---|---|---|
| melwitt | elodilles_pto: no worry about it, enjoy your PTO :) | 05:47 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Adapt tools/test-setup to Fedora 35 https://review.opendev.org/c/openstack/nova/+/840684 | 05:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Adapt bindep ubuntu-jammy https://review.opendev.org/c/openstack/nova/+/840822 | 06:51 |
| opendevreview | Andrew Bonney proposed openstack/nova stable/yoga: Fix segment-aware scheduling permissions error https://review.opendev.org/c/openstack/nova/+/840732 | 06:53 |
| opendevreview | Andrew Bonney proposed openstack/nova stable/xena: Fix eventlet.tpool import https://review.opendev.org/c/openstack/nova/+/840733 | 06:54 |
| gibi | clarkb, frickler: now that the py310 job is moved to jammy we need bit different fix for nova to pass the job https://review.opendev.org/c/openstack/nova/+/840822 | 07:20 |
| gibi | bauzas, sean-k-mooney, elodilles_pto: ^^ | 07:20 |
| kashyap | gibi: What is "jammy"? | 07:21 |
| kashyap | Morning, BTW | 07:21 |
| kashyap | Ah, it's the next Ubuntu release | 07:21 |
| gibi | kashyap: o/ | 07:26 |
| gibi | yepp ubuntu 22.04 | 07:26 |
| * kashyap nods :) | 07:27 | |
| kashyap | gibi: Nice live-debugging on that libvirt bug! :) | 08:02 |
| kashyap | Haha, so many "quirks" we learn: "Use of --live and --persistent together is quirky in certain failure scenarios" | 08:03 |
| kashyap | s/live/async/ :D | 08:03 |
| gibi | kashyap: the doc was helpful about debugging \ | 08:03 |
| gibi | and yes I learned a lot just by openning that bug upstream | 08:03 |
| kashyap | Yeah, the debug log filters are great. The `virt-admin` tool is great | 08:04 |
| kashyap | (As it dynamically configures the filters) | 08:04 |
| gibi | yeah that virt-admin tool was totally new to me but seem pretty usefull | 08:06 |
| kashyap | gibi: Don't forget to undo the filters after your debugging; or else your disk can get fill up :) | 08:11 |
| kashyap | To turn off: | 08:11 |
| kashyap | $> virt-admin daemon-log-filters "" | 08:11 |
| bauzas | gibi: made my duty | 08:11 |
| bauzas | fun for a French folk if the new Ubuntu release name is Jammy | 08:12 |
| gibi | kashyap: yeah I will do once the discussion stops in the bug | 08:12 |
| * bauzas wonders where Fred is (/cc Uggla) | 08:12 | |
| kashyap | Yep | 08:12 |
| gibi | bauzas: it is everybodies duty to make the CI happy | 08:12 |
| gibi | *everybody's | 08:12 |
| bauzas | Jammy and Fred (for Uggla who maybe forgot :p ) | 08:13 |
| bauzas | (a French TV documentation series for explaining how the world works :) ) | 08:14 |
| bauzas | https://www.youtube.com/watch?v=1vSLW2VKEjc | 08:15 |
| Uggla | bauzas, it was cool ! I know Jammy is still doing such kind of things on Youtube. | 08:15 |
| bauzas | actually, he's named Jamy with a single 'm' :p | 08:15 |
| bauzas | and yeah I know | 08:16 |
| * bauzas wanted to be a truck driver when he was a kid | 08:16 | |
| Uggla | bauzas, I liked the truck ! | 08:16 |
| bauzas | Uggla: I guess, like any teenager :) | 08:17 |
| gibi | from jammy I associated to https://www.youtube.com/watch?v=XMNG2LtF41o | 08:18 |
| Uggla | gibi, I did not remember that. bauzas ? | 08:20 |
| bauzas | gibi: damn, never heard of this kid before | 08:20 |
| gibi | but we had a french animated series too in my childhood https://en.wikipedia.org/wiki/Once_Upon_a_Time..._Life | 08:22 |
| gibi | this was awesom | 08:23 |
| Uggla | gibi, oh yes. Was it translated in Hungarian ? | 08:25 |
| gibi | yes it was | 08:25 |
| bauzas | technically it was an european series | 08:25 |
| bauzas | one of the first | 08:25 |
| Uggla | bauzas, Roger Carel, this guy made so many cartoon voices. | 08:26 |
| bauzas | France, Italy.. and Japan | 08:26 |
| gibi | Uggla: here is one with hun dub https://www.youtube.com/watch?v=L3fTs2tz2pE | 08:26 |
| bauzas | and yes, gibi, one of the TV series I was watching as a kid (a bit older tho) | 08:27 |
| Uggla | gibi, cannot see it in France. | 08:27 |
| bauzas | fwiw, it continued to teach my own kids :) | 08:28 |
| Uggla | gibi, blocked because of rights. | 08:28 |
| Uggla | gibi, I mean the youtube video. | 08:28 |
| bauzas | the vaccine episode is worthy during 2021 :) | 08:28 |
| gibi | maybe this? https://www.youtube.com/watch?v=Kj71MJTZ1ME | 08:28 |
| bauzas | s/is/was | 08:28 |
| Uggla | bauzas, you are right, these cartoons were good. (Not done with the idea to sell toys.) | 08:30 |
| bauzas | and still useful for explaining a few things | 08:30 |
| bauzas | Uggla: just sayin' they are on Prime Video | 08:30 |
| Uggla | gibi, the latest one works. | 08:32 |
| gibi | you can start learning hungarian :) | 08:32 |
| Uggla | gibi, damned I don't get a word... :) | 08:34 |
| gibi | it is not easy | 08:34 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/yoga: Isolate PCI tracker unit tests https://review.opendev.org/c/openstack/nova/+/840830 | 08:36 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/yoga: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/840831 | 08:36 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/yoga: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/840832 | 08:36 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/yoga: Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/840833 | 08:36 |
| Uggla | gibi, I would say italian or spanish there are some words not so far from french that I can understand. But hugarian seems another story. | 08:36 |
| gibi | Uggla: https://www.theguardian.com/education/gallery/2015/jan/23/a-language-family-tree-in-pictures | 08:37 |
| gibi | this explains a lot | 08:37 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: Isolate PCI tracker unit tests https://review.opendev.org/c/openstack/nova/+/840834 | 08:40 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/840835 | 08:40 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/840836 | 08:40 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/840837 | 08:40 |
| Uggla | gibi, oh cool ! hungarian is a complete different branch ! | 08:41 |
| gibi | yeah | 08:42 |
| gibi | and even finish is faaar | 08:42 |
| gibi | the grammar has common patterns and we have couple of similar words but that is all | 08:42 |
| Uggla | gibi, so it is completely different from Russian ? | 08:44 |
| gibi | Uggla: the origins of the language is from the Ural mountains from Russia but the russian language is totally different | 08:45 |
| Uggla | gibi, and so totally different from slovak as well ? | 08:47 |
| gibi | Uggla: yes | 08:48 |
| gibi | it has no slavic roots | 08:48 |
| gibi | but also the hungarian language borrowed a lot from others after group moved from the Ural to the current place. | 08:48 |
| gibi | we had a long ottoman occupation | 08:49 |
| Uggla | gibi, and no germanic as well. | 08:51 |
| Uggla | gibi, that's really interesting. I really did not know about that. | 08:52 |
| gibi | no germanic roots but lot of germanic influence due to the Austro-Hungarian monarchy | 08:52 |
| Uggla | gibi, means you are understanding more or less germans ? | 08:53 |
| gibi | nope | 08:53 |
| gibi | we have a lot of words we took over and integrated | 08:53 |
| gibi | but the grammar is really different | 08:53 |
| Uggla | gibi, ok. | 08:54 |
| gibi | like slag = Schlauch = hose | 08:54 |
| gibi | of course we have our own word for Schlauch that is not related: tömlő ;) | 08:56 |
| gibi | as far as I see tömlő has Uralic roots so that one is an original :D | 08:59 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Isolate PCI tracker unit tests https://review.opendev.org/c/openstack/nova/+/840841 | 09:01 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/840842 | 09:01 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/840843 | 09:02 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/840844 | 09:02 |
| Uggla | gibi, thanks for the hungarian lesson ! ;) | 09:03 |
| gibi | you are welcome :) | 09:04 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Isolate PCI tracker unit tests https://review.opendev.org/c/openstack/nova/+/840845 | 09:12 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/840846 | 09:12 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/840847 | 09:12 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/840848 | 09:12 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/yoga: Retry in CellDatabases fixture when global DB state changes https://review.opendev.org/c/openstack/nova/+/840734 | 09:14 |
| *** artom__ is now known as artom | 09:17 | |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Isolate PCI tracker unit tests https://review.opendev.org/c/openstack/nova/+/840850 | 09:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/840851 | 09:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/840852 | 09:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/840853 | 09:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/train: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/839717 | 09:20 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/train: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/839718 | 09:20 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/train: Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/839719 | 09:20 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/train: Isolate PCI tracker unit tests https://review.opendev.org/c/openstack/nova/+/840854 | 09:20 |
| sean-k-mooney | gibi: +w on https://review.opendev.org/c/openstack/nova/+/840822 | 10:11 |
| gibi | sean-k-mooney: thansk | 10:11 |
| opendevreview | ribaudr proposed openstack/nova-specs master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova-specs/+/831506 | 12:15 |
| opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | 12:17 |
| opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Use network's dns_domain in metadata hostname https://review.opendev.org/c/openstack/nova/+/840899 | 13:34 |
| artom | Now to write the spec :P | 13:34 |
| sean-k-mooney | oh free -2 then :P | 13:36 |
| sean-k-mooney | the spec should be fairly short i guess | 13:36 |
| * artom hopes so | 13:36 | |
| artom | I'll need to explain why our PTG thinking of hostname per network didn't pan out | 13:37 |
| artom | tl;dr, no schema for it | 13:37 |
| artom | And cloud-init doesn't know/care, it only cares about the single hostname in the "main" metadata | 13:38 |
| sean-k-mooney | it was fqdn per port | 13:39 |
| sean-k-mooney | well we are not touching hostname at all | 13:40 |
| sean-k-mooney | just adding fqdn | 13:40 |
| * artom couldn't find fqdn per port anywhere in the existing schema | 13:40 | |
| artom | And since the point is to make cloud-init not clobber the DHCP-provided hostname, I think we have to touch the hostname | 13:41 |
| sean-k-mooney | then no | 13:41 |
| sean-k-mooney | we should not do this | 13:41 |
| sean-k-mooney | the point is not to change the value of /etc/hostname that shoudl not end up with the fqdn | 13:42 |
| sean-k-mooney | it shoudl have the hostname but the domain name shoudl be avaiabel to used for generating /etc/hosts | 13:42 |
| sean-k-mooney | or making hostname -f provide the fqdn | 13:43 |
| artom | I mean we already have code in there that uses CONF.api.dhcp_domain, which defaults to 'novalocal', and joins that with the instance hostname and returns the whole thing as the hostname | 13:44 |
| artom | So not having the FQDN in hostname has sailed | 13:44 |
| sean-k-mooney | artom: if you going donw that pacth we ar eback to needing an nova api change to pass teh instance default domain name | 13:44 |
| frickler | artom: is this related to https://review.opendev.org/c/openstack/neutron-specs/+/832658 ? | 13:44 |
| artom | frickler, yep | 13:45 |
| sean-k-mooney | well not really its relate dbut not dependent on that | 13:45 |
| artom | sean-k-mooney, not necessarily. I mean, we can, but grabbing it from the network's dns_domain is also valid | 13:45 |
| sean-k-mooney | only if doen at the per port level | 13:45 |
| sean-k-mooney | not at the vm level | 13:45 |
| artom | Yeah, so that's... not great | 13:46 |
| artom | The assumption is that all of a VM's networks will have the same domain name | 13:46 |
| sean-k-mooney | there are no port ordering garentees so theere is no such thing as first port even to say just take the first one | 13:46 |
| artom | Which is true in the case of the operator who wants this | 13:46 |
| sean-k-mooney | artom: right that is flat out not valid | 13:46 |
| sean-k-mooney | we cannot assume that | 13:46 |
| sean-k-mooney | becuae that is not how it works normally | 13:46 |
| sean-k-mooney | take a ooo deployment each neutorn network typicaly has a differnt domain | 13:47 |
| sean-k-mooney | e.g. our manament domain. internal api ectra | 13:47 |
| artom | sean-k-mooney, so, backing up a bit, the purpose is to prevent cloud-init from clobbering the DHCP-provided domain name | 13:48 |
| sean-k-mooney | no | 13:48 |
| artom | Yes! | 13:48 |
| sean-k-mooney | that is not the purpose | 13:48 |
| artom | That is literally what the use case is | 13:48 |
| sean-k-mooney | no it is not | 13:48 |
| artom | Then we're talking about different use cases :) | 13:48 |
| sean-k-mooney | the usecaue is to provide a way to pass domain info via the metadata service | 13:48 |
| sean-k-mooney | cloud-init is one implemation but not the only one | 13:49 |
| artom | We're talking about different use cases then | 13:49 |
| sean-k-mooney | sure so my joking -2 shoudl actully be a really one | 13:49 |
| artom | I mean it's WIP anyways | 13:49 |
| sean-k-mooney | i ment on the spec | 13:50 |
| artom | There is no spec yet | 13:50 |
| sean-k-mooney | so if we look at https://cloudinit.readthedocs.io/en/latest/topics/instancedata.html | 13:52 |
| artom | So there's public_hostname and local_hostname *per interface* in https://cloudinit.readthedocs.io/en/latest/topics/instancedata.html#example-output | 13:52 |
| sean-k-mooney | we had talked about updaing the info in the interface section | 13:52 |
| artom | But I couldn't find where that actually comes from | 13:52 |
| artom | I couldn't find any existing schema in either ec2 or the openstack format for per-interface hostname | 13:53 |
| sean-k-mooney | artom: this is based on aws's implemenation | 13:53 |
| sean-k-mooney | not openstacks | 13:53 |
| sean-k-mooney | so i dont think openstack genergates thsi today | 13:54 |
| artom | Yeah, there's nothing that I could find in https://docs.openstack.org/nova/latest/user/metadata.html#openstack-format-metadata | 13:54 |
| sean-k-mooney | i would have expected use to keep local host name as just the host naem and pulblic_hostname as i guess the fqdu in the interface section | 13:54 |
| artom | So I don't think we generate any of that today | 13:56 |
| sean-k-mooney | right we dont | 13:56 |
| sean-k-mooney | which is what you were ment to be implementing | 13:57 |
| artom | So what happens for the openstack formatted metadata? | 13:57 |
| sean-k-mooney | so we coudl do this on a per port basis sicne we decide ot not take domain as a server paramter | 13:57 |
| sean-k-mooney | artom: we extend it with this same info | 13:57 |
| sean-k-mooney | with a new version number | 13:57 |
| sean-k-mooney | well date sicne we use date for the version | 13:58 |
| *** dasm|off is now known as dasm | 13:59 | |
| sean-k-mooney | bauzas: do you want to wait for the next team meetign for people ot review https://review.opendev.org/c/openstack/project-config/+/837595 or will i ask infra to review | 14:06 |
| bauzas | sean-k-mooney: not really, I accepted the names | 14:13 |
| bauzas | but if you want, we can wait until the meeting | 14:13 |
| sean-k-mooney | i would be happy to proceed as is | 14:13 |
| sean-k-mooney | i told infra we would wait already so lets leave it | 14:13 |
| sean-k-mooney | we are not in a rush | 14:14 |
| bauzas | as you want | 14:15 |
| * bauzas needs to go off for my kid | 14:15 | |
| mnaser | wel | 15:09 |
| mnaser | i've made some progress with the whole "nothing spawns on a box" story | 15:09 |
| mnaser | `WeighedHost [host: (<snip>) ram: 515266MB disk: 1662976MB io_ops: 0 instances: 0, weight: -999997.0]` | 15:09 |
| sean-k-mooney | the build weigher i woudl assume | 15:09 |
| sean-k-mooney | specificly the failed build weigher | 15:10 |
| mnaser | oh crap | 15:10 |
| mnaser | that became a weigher | 15:10 |
| mnaser | instead of the host being disabled?? | 15:10 |
| mnaser | ..4 years ago... | 15:11 |
| mnaser | well im out of it lol | 15:11 |
| sean-k-mooney | apprently we have both now | 15:11 |
| sean-k-mooney | i tought we removed the old disable behivior entirly | 15:12 |
| sean-k-mooney | https://docs.openstack.org/nova/latest/configuration/config.html#filter_scheduler.build_failure_weight_multiplier | 15:12 |
| sean-k-mooney | you can set that to a smaller value but it was inteneded to force the host to be avoided unless all other wer efully | 15:12 |
| sean-k-mooney | we still have https://docs.openstack.org/nova/latest/configuration/config.html#compute.consecutive_build_service_disable_threshold in the docs | 15:13 |
| mnaser | looks like that is still in use, but wont disable sean-k-mooney | 15:14 |
| mnaser | seeing https://opendev.org/openstack/nova/commit/91e29079a0eac825c5f4fe793cf607cb1771467d | 15:14 |
| sean-k-mooney | yep | 15:15 |
| sean-k-mooney | it is used to contol if you send info to the weigher or not | 15:15 |
| mnaser | unfortuantely the by product is there is no easy way of knowing if a host is being heavily weighed down | 15:16 |
| mnaser | vs seeing it as disabled is pretty obvious to an operato | 15:16 |
| sean-k-mooney | operator feedback was why we change this | 15:20 |
| sean-k-mooney | i tought the old behavior was the correct one | 15:20 |
| sean-k-mooney | i.e. disabling the service ideally with the reason to something the operator would consume | 15:20 |
| sean-k-mooney | mnaser: by the way you shoudl be abel to force that host to be used currently | 15:23 |
| sean-k-mooney | it just wont get selected byt the schduler if there are other options | 15:24 |
| mnaser | sean-k-mooney: well this explains the whole story why when guilhermesp_ tried to spin up a vm on it | 15:24 |
| mnaser | it started accepting requests right away | 15:24 |
| sean-k-mooney | one thing i had considered in the past was having this time out in some way | 15:24 |
| mnaser | it probably reset the failure counter to 0 and its all good to go | 15:24 |
| sean-k-mooney | yep | 15:24 |
| sean-k-mooney | it would have | 15:24 |
| mnaser | this doesnt explain why i was getting the port plugging timeouts that caused this but | 15:25 |
| mnaser | at least this wont just bring a whole bunch of systems out of being scheduled for an extended period of time | 15:25 |
| sean-k-mooney | well the neutron agent on that host might have hadd issues | 15:25 |
| sean-k-mooney | and perhpase they have now been resovled? | 15:26 |
| mnaser | sean-k-mooney: yeah but it's happening across different systems so its a common thing | 15:26 |
| mnaser | and on a stuck box i saw this once | 15:26 |
| mnaser | Agent rpc_loop - iteration:1192598 - ports processed. Elapsed:6685.721 | 15:26 |
| mnaser | 1.85h for the rpc loop? :) | 15:26 |
| sean-k-mooney | nice | 15:26 |
| sean-k-mooney | must be a record | 15:26 |
| mnaser | lol yup | 15:26 |
| mnaser | so i think tehre is something in the interaction between ovs and n-ovs-agent that just started the domino effect | 15:27 |
| sean-k-mooney | what os-vif driver are you usign by the way | 15:27 |
| sean-k-mooney | the navie one or vsctl | 15:27 |
| mnaser | i believe vsctl right now | 15:28 |
| mnaser | i think we should probably flip that over to native at some point soon | 15:28 |
| sean-k-mooney | ya so the reason i brought os-vif up is the ovs python bindings have a bug which i hope is now fixed | 15:28 |
| sean-k-mooney | basically the ovs python bindings actuly made un monkeypatched poll socket calls | 15:28 |
| sean-k-mooney | that could lock up nova when there was a disconnect until it reconnected | 15:29 |
| sean-k-mooney | i think neutron have work around that but depned on the release | 15:29 |
| mnaser | yeah i managed to pull a gmr when neutron was stuck for ~2h | 15:30 |
| sean-k-mooney | mnaser: what release | 15:30 |
| sean-k-mooney | im just checkinng for the neutron workaround | 15:31 |
| mnaser | victoria, i think we're waiting for a few things before going to wallaby (but this is wallaby nova + victoria n-ovs-agent) | 15:31 |
| mnaser | this is part of the culprit https://www.irccloud.com/pastebin/nxc0Uknt/ | 15:32 |
| sean-k-mooney | https://review.opendev.org/c/openstack/neutron/+/794892 | 15:33 |
| sean-k-mooney | i wonder if they only fixed ovn | 15:33 |
| opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | 15:34 |
| sean-k-mooney | i dont see it here https://github.com/openstack/neutron/blob/57f9822a8b398e92d1efebd85a1bdd4f9b27b046/neutron/agent/ovsdb/impl_idl.py | 15:34 |
| sean-k-mooney | os-vif does it but only if you are using the native imple | 15:36 |
| sean-k-mooney | https://github.com/openstack/os-vif/blob/master/vif_plug_ovs/ovsdb/impl_idl.py#L62-L99= | 15:36 |
| mnaser | sean-k-mooney: i mean technically since neutron polls so often it might as well be a keepalived | 15:37 |
| sean-k-mooney | so this is worign aroudn a bug in the ovs python bindings | 15:38 |
| sean-k-mooney | we basically trun off the heartbeat | 15:38 |
| sean-k-mooney | and rely on tcp keepalive | 15:39 |
| sean-k-mooney | https://bugs.launchpad.net/os-vif/+bug/1929446 | 15:39 |
| sean-k-mooney | the issue has been fixed in python-ovs 2.16 | 15:41 |
| sean-k-mooney | based on https://review.opendev.org/c/openstack/ovsdbapp/+/818446 | 15:41 |
| sean-k-mooney | the os-vif and neutron fixes were mitigations | 15:41 |
| mnaser | i see | 15:41 |
| opendevreview | Merged openstack/nova master: Add oslo.limit options to generated nova.conf https://review.opendev.org/c/openstack/nova/+/831101 | 15:43 |
| opendevreview | Merged openstack/nova master: Log the exception returned from a cell during API.get() https://review.opendev.org/c/openstack/nova/+/840260 | 15:43 |
| opendevreview | Merged openstack/nova master: Enable live_migration_events in nova-ovs-hybrid-plug https://review.opendev.org/c/openstack/nova/+/840446 | 15:43 |
| opendevreview | Merged openstack/nova master: pre-commit: Sync hacking version, bump plugins https://review.opendev.org/c/openstack/nova/+/836639 | 15:43 |
| sean-k-mooney | zuul just woke up i see | 15:44 |
| mnaser | my guess is a slow job in a commit that slowed down the merge gate :p | 15:45 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/yoga: Enable live_migration_events in nova-ovs-hybrid-plug https://review.opendev.org/c/openstack/nova/+/840931 | 15:48 |
| opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | 15:48 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Add Python 3.10 functional jobs https://review.opendev.org/c/openstack/nova/+/839029 | 15:53 |
| opendevreview | ribaudr proposed openstack/python-novaclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-novaclient/+/831651 | 15:53 |
| gmann | gibi: melwitt: releasenotes for dropping py3.6|7 https://review.opendev.org/c/openstack/nova/+/840786 | 16:00 |
| gibi | gmann: thanks. +A | 16:01 |
| gmann | gibi: thanks | 16:01 |
| opendevreview | Merged openstack/nova master: Add releasenote about dropping pythin 3.6|7 support https://review.opendev.org/c/openstack/nova/+/840786 | 16:42 |
| opendevreview | Artom Lifshitz proposed openstack/nova-specs master: Domain names in metadata https://review.opendev.org/c/openstack/nova-specs/+/840974 | 18:09 |
| opendevreview | Merged openstack/nova master: Adapt bindep ubuntu-jammy https://review.opendev.org/c/openstack/nova/+/840822 | 19:21 |
| opendevreview | Mohammed Naser proposed openstack/nova master: Switch libvirt event timeout message to warning https://review.opendev.org/c/openstack/nova/+/840985 | 20:28 |
| opendevreview | Mohammed Naser proposed openstack/nova master: Switch libvirt event timeout message to warning https://review.opendev.org/c/openstack/nova/+/840985 | 20:31 |
| *** dasm is now known as dasm|off | 21:09 | |
| opendevreview | Mohammed Naser proposed openstack/nova master: Fix race condition in _get_pci_passthrough_devices https://review.opendev.org/c/openstack/nova/+/840993 | 21:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!