Wednesday, 2022-04-20

opendevreviewMerged openstack/nova stable/xena: [stable-only] Drop lower-constraints job  https://review.opendev.org/c/openstack/nova/+/83800701:45
*** amoralej|off is now known as amoralej06:48
opendevreviewMerged openstack/nova stable/xena: Cleanup old resize instances dir before resize  https://review.opendev.org/c/openstack/nova/+/82840708:57
opendevreviewMerged openstack/nova stable/wallaby: [stable-only] Drop lower-constraints job  https://review.opendev.org/c/openstack/nova/+/83803008:57
opendevreviewBalazs Gibizer proposed openstack/nova master: Remove unavailable but not reported PCI devices at startup  https://review.opendev.org/c/openstack/nova/+/83855309:54
opendevreviewBalazs Gibizer proposed openstack/nova master: Simulate bug 1969496  https://review.opendev.org/c/openstack/nova/+/83855409:54
opendevreviewBalazs Gibizer proposed openstack/nova master: DNM:Allow claiming PCI PF if child VF is unavailable  https://review.opendev.org/c/openstack/nova/+/83855509:54
*** ralonsoh_ is now known as ralonsoh10:15
gibibauzas: I'm +2 on the whole SQLA 2.0 series on nova10:39
bauzasgibi: ack, will take a look later (working on creating a devstack environment)10:40
opendevreviewMerged openstack/nova master: db: Narrow down deprecation warning filter  https://review.opendev.org/c/openstack/nova/+/83715410:55
opendevreviewMerged openstack/nova master: db: Trivial rewrapping of warning filters  https://review.opendev.org/c/openstack/nova/+/83715510:55
opendevreviewErlon R. Cruz proposed openstack/nova stable/xena: Adds regression test for bug LP#1944619  https://review.opendev.org/c/openstack/nova/+/83855012:02
opendevreviewErlon R. Cruz proposed openstack/nova stable/xena: Fix pre_live_migration rollback  https://review.opendev.org/c/openstack/nova/+/83601512:02
*** amoralej is now known as amoralej|lunch12:06
bauzasfolks, I need to go outside for one hour, bbiab12:27
*** amoralej|lunch is now known as amoralej13:06
erlonhey folks, could I get some reviews on: https://review.opendev.org/q/topic:bug/1944619+AND+-status:abandoned+AND+(branch:stable/xena+OR+branch:stable/yoga)13:36
ricolinstephenfin: Hey, I update the patch set for IOMMU, and it will be great if you can take a look together :) https://review.opendev.org/c/openstack/nova/+/83064613:41
sean-k-mooneyerlon: ok so this will have ot be backported in order so the yoga backports will have to land before the xena ectra 13:44
sean-k-mooneyerlon: so https://review.opendev.org/c/openstack/nova/+/836014 is the placse to start 13:44
sean-k-mooneyerlon: looking at https://review.opendev.org/q/topic:bug%252F194461913:45
sean-k-mooneyi dont see the regression test backported to yoga13:45
erlonyes, I was trying to do all of then one chainng into the other, but, yes they need to land first13:45
erlonsean-k-mooney: the regression test landed before the release date heh13:46
sean-k-mooneyah ok13:46
sean-k-mooneythat makes sense then13:47
*** dasm|off is now known as dasm14:03
elodilleserlon: yep, the regression test landed before the yoga release but not before yoga was branched o:)14:37
elodilleserlon: that's why it's not present on stable/yoga14:37
erlonelodilles: hmm, got it14:38
erlonelodilles: so I should cherry pick the 2 patches similar to what I was trying to do for the xena branch14:38
elodilleserlon: yes, cherry pick them please, i think those should cleanly apply14:39
erlonelodilles: another thing, I was trying to backport both patches back to ussuri, but the unit tests base get very different specially in ussuri14:39
erlonwould it be possible to backport onlt the fix there? not the unit tests?14:40
elodilleserlon: well, sometimes tests needs some refactor, i'll try to look at the patch later, and help with it if needed14:43
elodilleserlon: as both should be backported14:43
elodilleserlon: but first let's merge them on yoga, and then xena, and so on :)14:44
erlonok, yes, on wallaby I needed to do some refactor, but for ussuri it was a bit too much, but we can try when we get there14:48
*** artom__ is now known as artom15:19
stephenfinmelwitt: You'd probably be interested in https://review.opendev.org/c/openstack/nova/+/837162 as I think the issue I'm addressing is your code. Would like to confirm I'm not missing anything obvious15:27
melwittstephenfin: yes indeed, I will take a look, thanks for the heads up15:30
opendevreviewMerged openstack/os-traits master: Remove unnecessary unicode prefixes  https://review.opendev.org/c/openstack/os-traits/+/83843616:02
opendevreviewMerged openstack/os-resource-classes master: Update python testing classifier  https://review.opendev.org/c/openstack/os-resource-classes/+/82247316:04
opendevreviewMerged openstack/os-resource-classes master: Remove unnecessary unicode prefixes  https://review.opendev.org/c/openstack/os-resource-classes/+/83844816:05
opendevreviewMerged openstack/nova master: db: Replace use of strings in join, defer operations  https://review.opendev.org/c/openstack/nova/+/83715616:21
opendevreviewMerged openstack/nova master: db: Remove use of empty 'and_()'  https://review.opendev.org/c/openstack/nova/+/83715716:21
opendevreviewMerged openstack/nova master: db: Replace use of Connection.connect() method  https://review.opendev.org/c/openstack/nova/+/83715816:21
gibikashyap: I reviewed https://review.opendev.org/q/topic:bp%252Fcpu-selection-with-hypervisor-consideration and left comments inline. Let me know if I should expand on some of some of my comments.16:28
gibithank you for the split, it definitely helps the review16:29
* gibi logs off for today16:32
*** amoralej is now known as amoralej|off16:35
*** ministry is now known as __ministry16:36
melwittstephenfin: +W16:42
stephenfinty!16:46
opendevreviewRico Lin proposed openstack/nova master: libvirt: Add vIOMMU device to guest  https://review.opendev.org/c/openstack/nova/+/83064616:50
*** artom__ is now known as artom19:07
opendevreviewErlon R. Cruz proposed openstack/nova stable/yoga: Fix pre_live_migration rollback  https://review.opendev.org/c/openstack/nova/+/83601420:45
opendevreviewErlon R. Cruz proposed openstack/nova stable/yoga: Adds regression test for bug LP#1944619  https://review.opendev.org/c/openstack/nova/+/83878820:45
erlonelodilles: see if this is what you needed ^20:47
*** dasm is now known as dasm|off20:53

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