| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-01-13 | |||
| 18:21:29 | artom | Aww, you have a sense for me | |
| 19:01:02 | efried | someone should | |
| 19:01:19 | efried | <tsssss> | |
| 20:15:37 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595 | |
| 20:15:39 | openstack | bug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms) | |
| 20:15:39 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404 | |
| 20:15:45 | artom | *finally* | |
| 20:19:48 | sean-k-mooney | oh its stacked ... | |
| 20:35:45 | efried | dustinc: You have a couple of notes in https://review.opendev.org/#/c/676522/ about needing more UT coverage. Is that still true? | |
| 20:35:54 | efried | (I haven't dug in yet to verify) | |
| 20:41:00 | openstackgerrit | Dan Smith proposed openstack/nova master: Add NovaEphemeralObject class for non-persistent objects https://review.opendev.org/702049 | |
| 20:55:54 | openstackgerrit | Merged openstack/nova-specs master: Update provider config spec for identification conflicts https://review.opendev.org/693414 | |
| 21:10:18 | dustinc | efried: I thought it was good and I just forgot to remove the note, but I checked again just to be sure and actually found another scenario I want to write a test for.. | |
| 21:10:47 | efried | dustinc: I just finished reviewing the code side and found some things that need to be fixed anyway. | |
| 21:11:32 | dustinc | I have to step away for a few minutes, but I will check it out in a few. Thanks! | |
| 21:13:09 | efried | no hurry | |
| #openstack-nova - 2020-01-14 | |||
| 02:37:46 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | |
| 02:37:47 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | |
| 02:48:19 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | |
| 02:48:19 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | |
| 06:27:35 | openstackgerrit | Brin Zhang proposed openstack/nova master: Populate 'updated_at' when aggregate is updated https://review.opendev.org/580271 | |
| 07:47:18 | openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: report storage bus traits https://review.opendev.org/666914 | |
| 08:00:25 | brinzhang | gibi: are you around? | |
| 08:20:04 | openstackgerrit | Alexandre arents proposed openstack/nova master: Avoid allocation leak when deleting instance stuck in BUILD https://review.opendev.org/702368 | |
| 08:46:43 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add support for stable device rescue https://review.opendev.org/700812 | |
| 08:46:43 | openstackgerrit | Lee Yarwood proposed openstack/nova master: virt: Provide block_device_info during rescue https://review.opendev.org/700811 | |
| 08:46:44 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP api: Introduce microverion 2.82 allowing boot from volume rescue https://review.opendev.org/701430 | |
| 08:46:44 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP compute: Report COMPUTE_BFV_RESCUE and check during rescue https://review.opendev.org/701429 | |
| 08:46:45 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Support boot from volume instance rescue https://review.opendev.org/701431 | |
| 08:47:23 | lyarwood | alex_xu: https://review.opendev.org/#/c/694063/ - I'm looking for some non-RH nova-specs core review on this if you have time today, ^ PoC code posted above. | |
| 08:47:25 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Add action event fault details https://review.opendev.org/699669 | |
| 09:06:13 | openstackgerrit | waleed mousa proposed openstack/os-vif master: [Follow Up] OVS DPDK port representors support https://review.opendev.org/665965 | |
| 09:44:16 | openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/nova master: Add ironic hypervisor doc https://review.opendev.org/701998 | |
| 09:56:13 | gibi | lyarwood, dansmith: I'm ok with https://review.opendev.org/#/c/694063/ but there are precedence in nova (support move ops with qos ports) not to add api microversions. So I want to double check with you about it before +ait | |
| 10:04:09 | brinzhang | gibi: https://review.opendev.org/#/c/699669/ I was updated this patch, add the decription of the new policy, if you have time please review again :) | |
| 10:05:48 | gibi | brinzhang: ack, will look | |
| 10:09:09 | brinzhang | gibi: thanks~ | |
| 10:25:11 | gibi | brinzhang: I'm +2 on the instance action spec | |
| 10:25:14 | gibi | thanks for the update | |
| 10:26:03 | brinzhang | gibi: cool, thanks ^^ | |
| 11:13:19 | openstackgerrit | Arnaud Morin proposed openstack/nova master: Force refresh of instance info cache on event https://review.opendev.org/702394 | |
| 11:18:33 | stephenfin | bauzas: Can you send this on its way, please? https://review.opendev.org/#/c/695733/ | |
| 11:26:29 | alex_xu | lyarwood: got it | |
| 11:29:39 | openstackgerrit | Merged openstack/nova-specs master: Spec: Ussuri: Encrypted Emulated Virtual TPM https://review.opendev.org/686804 | |
| 11:30:46 | luyao | stephenfin: Could you look at spec 'support live migration with vpmem'? https://review.opendev.org/#/c/695863 | |
| 11:31:19 | stephenfin | luyao: I can, but not today. Could you ask me again tomorrow? | |
| 11:31:56 | luyao | stephenfin: OK, thanks at advance. :) | |
| 11:42:33 | alex_xu | lyarwood: gibi https://review.opendev.org/#/c/694063/7/specs/ussuri/approved/virt-bfv-instance-rescue.rst@76 | |
| 12:14:29 | etingof | o/ do we have a JSON schema for whatever is exposed through Nova metadata service? I am particularly interested in network_data.json | |
| 12:15:34 | openstackgerrit | Luyao Zhong proposed openstack/nova-specs master: support live migration with virtual persistent memory https://review.opendev.org/695863 | |
| 12:23:54 | sean-k-mooney | dansmith: efried i deployed https://review.opendev.org/#/c/699554/2 and installed the required version fo the sdk and cyborg client | |
| 12:24:07 | sean-k-mooney | it looks like there are still issue however http://paste.openstack.org/show/788372/ | |
| 12:25:37 | sean-k-mooney | it looks like everything on the placement side is fine but its failing to boot a vm with "AttributeError: \'RequestSpec\' object has no attribute \'root_required\'\n\n'" | |
| 12:31:24 | huaqiang | hello stephenfin. I hope you enjoyed your vacation | |
| 12:32:21 | huaqiang | I also want to invite your to review https://review.opendev.org/#/c/668656/ | |
| 12:34:26 | huaqiang | we have had some disscution especially on how to create a mixed instance, and Alex have put those dicussion links to the update of the spec | |
| 12:34:34 | huaqiang | I hope to get your comments. | |
| 12:37:00 | gibi | alex_xu: responded. If you are OK with the microversion then feel free to +A, I will +A myself if lyarwood or dansmith state that the microversion is intentional | |
| 12:39:15 | lyarwood | gibi / alex_xu ; apologies just back from a long lunch, yeah it's intentional, I'll add a comment now. | |
| 12:39:25 | gibi | lyarwood: cool thanks | |
| 12:47:13 | brinzhang | This bug fix is ready to review, if you have free time, please review https://review.opendev.org/#/c/580271/ | |
| 12:48:59 | openstack | bug 1663456 in OpenStack Compute (nova) "Field 'updated_at' always 'None' when show aggregate" [Low,In progress] https://launchpad.net/bugs/1663456 - Assigned to Brin Zhang (zhangbailin) | |
| 12:48:59 | brinzhang | it's bug 1663456 | |
| 12:54:07 | openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Boot from volume instance rescue https://review.opendev.org/694063 | |
| 12:54:19 | lyarwood | ^ addressed the commit nit btw | |
| 12:55:34 | sean-k-mooney | dansmith: efried: ok so the cyborg series just need to be rebased on https://review.opendev.org/#/c/699050/ | |
| 12:56:31 | sean-k-mooney | well that is merges so rebaseing on master would be enough | |
| 13:00:24 | gibi | lyarwood: +Ad the spec | |
| 13:00:31 | gibi | lyarwood: thanks for the confirmation | |
| 13:03:07 | lyarwood | gibi: awesome thanks :) | |
| 13:19:56 | sean-k-mooney | stephenfin: can you review this when you get a chance https://review.opendev.org/#/c/701601/ | |
| 13:34:33 | openstackgerrit | Merged openstack/nova-specs master: Boot from volume instance rescue https://review.opendev.org/694063 | |
| 13:48:08 | stephenfin | huaqiang: As with luyao, if you can ask me again tomorrow I'll try get to it. Too much to do today :( | |
| 13:57:27 | lyarwood | efried: https://review.opendev.org/#/c/694033/ - The spec for this has now merged if you're able to look again today, thanks in advance. | |
| 14:01:49 | stephenfin | sean-k-mooney: done | |
| 14:04:04 | huaqiang | stephenfin: Understand. Don't worry. I will connect you later :) | |
| 14:04:18 | stephenfin | cool, thanks :) | |
| 14:16:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove unused nova-network objects https://review.opendev.org/697156 | |
| 14:16:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove now unnecessary nova-network workaround https://review.opendev.org/702440 | |
| 14:17:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove now unnecessary nova-net workaround https://review.opendev.org/702440 | |
| 14:17:39 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove unused nova-network objects https://review.opendev.org/697156 | |
| 14:18:58 | stephenfin | gibi: I addressed your comments in https://review.opendev.org/#/c/696516/ Think you could revisit at some point? | |
| 14:19:57 | stephenfin | gibi: Also, I think I addressed mriedem's concerns on https://review.opendev.org/#/c/696745/ but we can have dansmith look at it to make sure (since he was of the same opinion), if that helps | |
| 14:40:41 | openstackgerrit | sean mooney proposed openstack/os-vif master: [DNM]test composing devstack_local_conf sections https://review.opendev.org/702446 | |
| 14:41:57 | sean-k-mooney | stephenfin: if ^ works ill squash it into the previous patch but im not sure that we can split devstack_local_conf defination across the job inheritance or if we can if that extends to the post-config: section too | |
| 14:42:09 | sean-k-mooney | so i expect that might fail | |
| 14:45:25 | stephenfin | ack | |
| 14:46:06 | ganso | this spec exists https://blueprints.launchpad.net/nova/+spec/allow-disabling-cpu-flags but I don't see it handling previously existing VMs, only newly created VMs so that they get the correct flags set in the instance XML. Is there any work in-progress to minimize the impact around this? or any known workaround besides having to edit every instance XML manually? | |
| 14:46:06 | ganso | Hello folks. I have a question about the impact of security vulnerability upgrades to previously existing VMs. I have a customer that after upgrading the kernel to a newer version that supressed cpu flags ended up not being able to turn their VMs back on because qemu wouldn't allow unless the flags are specifically disabled in the XML. I believe this is something we will see happen several times in the future so it will be a recurrent impact. I see | |
| 14:47:12 | stephenfin | ganso: kashyap might be able to help you with that, if they're around ^ | |
| 14:47:17 | kashyap | Already reading :-) | |
| 14:48:11 | kashyap | ganso: Even if you edit every instance XML manually, which we don't recommend, you do know that it will go away once you reboot the guest? | |
| 14:49:13 | kashyap | ganso: I haven't gotten around to implementing that BP, got buried in other stuff. But good news, there's a valid workaround: | |
| 14:49:17 | ganso | kashyap: I wasn't aware of that, thanks. It will go away in the sense that every time it the VM is rebooted nova will overwrite the cpu flags in the XML with what it has defined? | |
| 14:49:41 | kashyap | ganso: QEMU has added newer variants of CPU models (with affected flags disabled) that you can directly specify with Nova | |
| 14:50:40 | kashyap | ganso: So, for the recent "TSX" vulnerability fiasco ... | |
| 14:51:34 | kashyap | ganso: ... QEMU / libvirt has added *-noTSX CPU models. | |
| 14:52:21 | kashyap | ganso: E.g. on my Fedora host, running qemu-system-x86-4.2.0-2.fc30.x86_64: | |
| 14:53:07 | kashyap | x86 Broadwell-noTSX (alias of Broadwell-v2) | |
| 14:53:07 | kashyap | $> qemu-system-x86_64 -cpu help | egrep *.noTSX* | |