| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-20 | |||
| 01:45:17 | opendevreview | Merged openstack/nova stable/xena: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838007 | |
| 08:57:41 | opendevreview | Merged openstack/nova stable/xena: Cleanup old resize instances dir before resize https://review.opendev.org/c/openstack/nova/+/828407 | |
| 08:57:48 | opendevreview | Merged openstack/nova stable/wallaby: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838030 | |
| 09:54:46 | 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:47 | 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:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/838554 | |
| 10:39:40 | gibi | bauzas: I'm +2 on the whole SQLA 2.0 series on nova | |
| 10:40:05 | bauzas | gibi: ack, will take a look later (working on creating a devstack environment) | |
| 10:55:35 | opendevreview | Merged openstack/nova master: db: Narrow down deprecation warning filter https://review.opendev.org/c/openstack/nova/+/837154 | |
| 10:55:43 | opendevreview | Merged openstack/nova master: db: Trivial rewrapping of warning filters https://review.opendev.org/c/openstack/nova/+/837155 | |
| 12:02:10 | 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:11 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836015 | |
| 12:27:06 | bauzas | folks, I need to go outside for one hour, bbiab | |
| 13:36:09 | 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:41:24 | 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:44:39 | 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:48 | sean-k-mooney | erlon: so https://review.opendev.org/c/openstack/nova/+/836014 is the placse to start | |
| 13:45:21 | sean-k-mooney | erlon: looking at https://review.opendev.org/q/topic:bug%252F1944619 | |
| 13:45:39 | sean-k-mooney | i dont see the regression test backported to yoga | |
| 13:45:52 | erlon | yes, I was trying to do all of then one chainng into the other, but, yes they need to land first | |
| 13:46:42 | erlon | sean-k-mooney: the regression test landed before the release date heh | |
| 13:46:57 | sean-k-mooney | ah ok | |
| 13:47:05 | sean-k-mooney | that makes sense then | |
| 14:37:27 | elodilles | erlon: yep, the regression test landed before the yoga release but not before yoga was branched o:) | |
| 14:37:49 | elodilles | erlon: that's why it's not present on stable/yoga | |
| 14:38:12 | erlon | elodilles: hmm, got it | |
| 14:38:46 | erlon | elodilles: so I should cherry pick the 2 patches similar to what I was trying to do for the xena branch | |
| 14:39:47 | elodilles | erlon: yes, cherry pick them please, i think those should cleanly apply | |
| 14:39:51 | 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:40:36 | erlon | would it be possible to backport onlt the fix there? not the unit tests? | |
| 14:43:08 | 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:41 | elodilles | erlon: as both should be backported | |
| 14:44:42 | elodilles | erlon: but first let's merge them on yoga, and then xena, and so on :) | |
| 14:48:16 | 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 | |
| 15:27:18 | 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:30:46 | melwitt | stephenfin: yes indeed, I will take a look, thanks for the heads up | |
| 16:02:41 | opendevreview | Merged openstack/os-traits master: Remove unnecessary unicode prefixes https://review.opendev.org/c/openstack/os-traits/+/838436 | |
| 16:04:59 | opendevreview | Merged openstack/os-resource-classes master: Update python testing classifier https://review.opendev.org/c/openstack/os-resource-classes/+/822473 | |
| 16:05:01 | opendevreview | Merged openstack/os-resource-classes master: Remove unnecessary unicode prefixes https://review.opendev.org/c/openstack/os-resource-classes/+/838448 | |
| 16:21:02 | opendevreview | Merged openstack/nova master: db: Replace use of strings in join, defer operations https://review.opendev.org/c/openstack/nova/+/837156 | |
| 16:21:10 | opendevreview | Merged openstack/nova master: db: Remove use of empty 'and_()' https://review.opendev.org/c/openstack/nova/+/837157 | |
| 16:21:18 | opendevreview | Merged openstack/nova master: db: Replace use of Connection.connect() method https://review.opendev.org/c/openstack/nova/+/837158 | |
| 16:28:49 | 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:29:07 | gibi | thank you for the split, it definitely helps the review | |
| 16:42:23 | melwitt | stephenfin: +W | |
| 16:46:09 | stephenfin | ty! | |
| 16:50:12 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Add vIOMMU device to guest https://review.opendev.org/c/openstack/nova/+/830646 | |
| 20:45:01 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/yoga: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836014 | |
| 20:45:02 | 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:47:02 | erlon | elodilles: see if this is what you needed ^ | |
| #openstack-nova - 2022-04-21 | |||
| 04:05:46 | opendevreview | Merged openstack/nova stable/victoria: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838032 | |
| 07:44:19 | kashyap | gibi: Hi, will look at the review comments; if isomething is unclear, I'll ask | |
| 07:50:02 | kashyap | And thanks for the review! | |
| 07:51:44 | kashyap | Also, bonus points for reading the full commit message on baselineHypervisorCPU() :) | |
| 08:02:12 | gibi | :) | |
| 08:26:38 | gibi | Uggla: added some extra information about the notification work in https://review.opendev.org/c/openstack/nova-specs/+/833669 let me know if you have questions | |
| 08:28:19 | Uggla | gibi, thx I will have a look. | |
| 08:42:29 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (db+object) https://review.opendev.org/c/openstack/nova/+/831193 | |
| 08:42:30 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (drivers) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 08:42:30 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 08:42:31 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 08:52:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/829248 | |
| 09:43:58 | opendevreview | Merged openstack/nova master: Follow up for nova-manage image property commands https://review.opendev.org/c/openstack/nova/+/830895 | |
| 09:45:47 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Reproduce bug 1945310 https://review.opendev.org/c/openstack/nova/+/811414 | |
| 09:45:48 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Query ports with admin client to get resource_request https://review.opendev.org/c/openstack/nova/+/811416 | |
| 10:22:45 | sean-k-mooney | bauzas: if i have missed anythin or missrepresneted can you comment on http://lists.openstack.org/pipermail/openstack-discuss/2022-April/028268.html | |
| 10:23:36 | sean-k-mooney | im 99% sure that everything i said there is correct but i have not personally configured vGPUs so if you can confirm that woudl be good | |
| 10:58:07 | bauzas | sean-k-mooney: I already replied ;) | |
| 10:58:18 | bauzas | sean-k-mooney: but thanks for your reply | |
| 11:32:42 | kashyap | gibi[m]: Got a minute here? I'm going a bit mad with the unit test -- something is wrong here because the 'obj' is printing empty |
|
| 11:34:07 | opendevreview | Merged openstack/nova master: Sync rootwrap.conf from oslo.rootwrap https://review.opendev.org/c/openstack/nova/+/823229 | |
| 11:35:01 | gibi | kashyap: will look in a minute | |
| 11:37:50 | kashyap | No rush | |
| 12:09:58 | opendevreview | Wenping Song proposed openstack/nova-specs master: Usage of new trait of OWNER_NOVA https://review.opendev.org/c/openstack/nova-specs/+/819510 | |
| 12:23:48 | gibi | kashyap: now looking :) | |
| 12:24:08 | kashyap | Cool; I'm sure it's something stupid that I'm not parsing myself :D | |
| 12:27:29 | sean-k-mooney | gibi: just pushed my comment for https://review.opendev.org/c/openstack/nova-specs/+/819510 but i see the also just pushed an update... | |
| 12:29:43 | gibi | kashyap: at least I can reproduce your problem locally :) | |
| 12:30:10 | gibi | sean-k-mooney: I will look back to the spec | |
| 12:30:27 | kashyap | gibi: Yeah. I wonder where I'm messing up the parsing here | |
| 12:31:28 | sean-k-mooney | songwenping_: can you look at my comment on patchset 5 | |
| 12:32:25 | sean-k-mooney | songwenping_: if you can respin the spec with those changes i think it would me mergable, please also include my version of the upgrade impact section | |
| 12:33:01 | sean-k-mooney | its imporant to call out the depenecy on upgrading placement before nova so that the new standard traits are aviable | |
| 12:33:50 | gibi | kashyap: your test code calls https://github.com/openstack/nova/blob/dc30a7e6d19509ce34cb89ffcdb4ff2c36469d89/nova/virt/libvirt/config.py#L75 that is pretty empty | |
| 12:34:13 | gibi | wait, no | |
| 12:34:19 | gibi | hang on | |
| 12:35:18 | gibi | :D | |
| 12:35:41 | gibi | it correctly calls nova.virt.libvirt.config.LibvirtConfigCPU.parse_dom | |
| 12:37:46 | kashyap | Yea, did you put a breakpoint() to see it | |
| 12:39:52 | kashyap | All I want to assert is that the test is parsing the <cpu> element correctly. So assert for the presence of 'model' / 'vendor' etc? | |
| 12:42:22 | gibi | does nova use LibvirtConfigCPU to parse xml? I only see existing tests that verify to_xml but no parse_dom | |
| 12:45:19 | gibi | it seems that parsing code in LibvirtConfigCPU.parse_dom is either incorrect or the example XML you use is incorrect | |
| 12:45:20 | kashyap | gibi: Yeah, I didn't find any of parse_dom either | |
| 12:46:10 | gibi | the parser code assumes that everything is directly under <cpu> but in your example XML the tags are under cpu/mode | |
| 12:46:13 | kashyap | gibi: The example XML is ... trimmed down output from running `virsh domcapabilities` on my computer | |
| 12:46:56 | gibi | so the exta nesting by "mode" is makes the parse_dom to find nothing to match | |
| 12:47:07 | kashyap | gibi: Run `virsh domcapabilities` and you'll see tags under several modes | |
| 12:47:58 | gibi | then you need to handle the extra mode nesting in the parse_dom code | |
| 12:48:39 | gibi | but be aware that the to_xml code also emits the xml without the extra mode nesting right now | |
| 12:48:50 | kashyap | gibi: Hmm, I see. So, here's a test you do: the "everything under <cpu>" from `virsh capabilities` | |