| opendevreview | Merged openstack/nova stable/xena: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838007 | 01:45 |
|---|---|---|
| *** amoralej|off is now known as amoralej | 06:48 | |
| opendevreview | Merged openstack/nova stable/xena: Cleanup old resize instances dir before resize https://review.opendev.org/c/openstack/nova/+/828407 | 08:57 |
| opendevreview | Merged openstack/nova stable/wallaby: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838030 | 08:57 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/838553 | 09:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/838554 | 09:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/838555 | 09:54 |
| *** ralonsoh_ is now known as ralonsoh | 10:15 | |
| gibi | bauzas: I'm +2 on the whole SQLA 2.0 series on nova | 10:39 |
| bauzas | gibi: ack, will take a look later (working on creating a devstack environment) | 10:40 |
| opendevreview | Merged openstack/nova master: db: Narrow down deprecation warning filter https://review.opendev.org/c/openstack/nova/+/837154 | 10:55 |
| opendevreview | Merged openstack/nova master: db: Trivial rewrapping of warning filters https://review.opendev.org/c/openstack/nova/+/837155 | 10:55 |
| opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838550 | 12:02 |
| opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836015 | 12:02 |
| *** amoralej is now known as amoralej|lunch | 12:06 | |
| bauzas | folks, I need to go outside for one hour, bbiab | 12:27 |
| *** amoralej|lunch is now known as amoralej | 13:06 | |
| erlon | hey 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 |
| ricolin | stephenfin: 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/+/830646 | 13:41 |
| sean-k-mooney | erlon: 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-mooney | erlon: so https://review.opendev.org/c/openstack/nova/+/836014 is the placse to start | 13:44 |
| sean-k-mooney | erlon: looking at https://review.opendev.org/q/topic:bug%252F1944619 | 13:45 |
| sean-k-mooney | i dont see the regression test backported to yoga | 13:45 |
| erlon | yes, I was trying to do all of then one chainng into the other, but, yes they need to land first | 13:45 |
| erlon | sean-k-mooney: the regression test landed before the release date heh | 13:46 |
| sean-k-mooney | ah ok | 13:46 |
| sean-k-mooney | that makes sense then | 13:47 |
| *** dasm|off is now known as dasm | 14:03 | |
| elodilles | erlon: yep, the regression test landed before the yoga release but not before yoga was branched o:) | 14:37 |
| elodilles | erlon: that's why it's not present on stable/yoga | 14:37 |
| erlon | elodilles: hmm, got it | 14:38 |
| erlon | elodilles: so I should cherry pick the 2 patches similar to what I was trying to do for the xena branch | 14:38 |
| elodilles | erlon: yes, cherry pick them please, i think those should cleanly apply | 14:39 |
| erlon | elodilles: another thing, I was trying to backport both patches back to ussuri, but the unit tests base get very different specially in ussuri | 14:39 |
| erlon | would it be possible to backport onlt the fix there? not the unit tests? | 14:40 |
| elodilles | erlon: well, sometimes tests needs some refactor, i'll try to look at the patch later, and help with it if needed | 14:43 |
| elodilles | erlon: as both should be backported | 14:43 |
| elodilles | erlon: but first let's merge them on yoga, and then xena, and so on :) | 14:44 |
| erlon | ok, 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 there | 14:48 |
| *** artom__ is now known as artom | 15:19 | |
| stephenfin | melwitt: 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 obvious | 15:27 |
| melwitt | stephenfin: yes indeed, I will take a look, thanks for the heads up | 15:30 |
| opendevreview | Merged openstack/os-traits master: Remove unnecessary unicode prefixes https://review.opendev.org/c/openstack/os-traits/+/838436 | 16:02 |
| opendevreview | Merged openstack/os-resource-classes master: Update python testing classifier https://review.opendev.org/c/openstack/os-resource-classes/+/822473 | 16:04 |
| opendevreview | Merged openstack/os-resource-classes master: Remove unnecessary unicode prefixes https://review.opendev.org/c/openstack/os-resource-classes/+/838448 | 16:05 |
| opendevreview | Merged openstack/nova master: db: Replace use of strings in join, defer operations https://review.opendev.org/c/openstack/nova/+/837156 | 16:21 |
| opendevreview | Merged openstack/nova master: db: Remove use of empty 'and_()' https://review.opendev.org/c/openstack/nova/+/837157 | 16:21 |
| opendevreview | Merged openstack/nova master: db: Replace use of Connection.connect() method https://review.opendev.org/c/openstack/nova/+/837158 | 16:21 |
| gibi | kashyap: 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 |
| gibi | thank you for the split, it definitely helps the review | 16:29 |
| * gibi logs off for today | 16:32 | |
| *** amoralej is now known as amoralej|off | 16:35 | |
| *** ministry is now known as __ministry | 16:36 | |
| melwitt | stephenfin: +W | 16:42 |
| stephenfin | ty! | 16:46 |
| opendevreview | Rico Lin proposed openstack/nova master: libvirt: Add vIOMMU device to guest https://review.opendev.org/c/openstack/nova/+/830646 | 16:50 |
| *** artom__ is now known as artom | 19:07 | |
| opendevreview | Erlon R. Cruz proposed openstack/nova stable/yoga: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836014 | 20:45 |
| opendevreview | Erlon R. Cruz proposed openstack/nova stable/yoga: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838788 | 20:45 |
| erlon | elodilles: see if this is what you needed ^ | 20:47 |
| *** dasm is now known as dasm|off | 20:53 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!