*** TxGirlGeek has quit IRC | 00:04 | |
*** vishalmanchanda has joined #openstack-nova | 00:06 | |
*** TxGirlGeek has joined #openstack-nova | 00:07 | |
*** macz has quit IRC | 00:17 | |
*** jmlowe has quit IRC | 00:25 | |
*** jmlowe has joined #openstack-nova | 00:26 | |
*** jmlowe has quit IRC | 00:26 | |
*** brault has quit IRC | 00:33 | |
*** brault has joined #openstack-nova | 00:33 | |
*** gentoorax has quit IRC | 00:46 | |
*** gentoorax has joined #openstack-nova | 00:50 | |
*** igordc has joined #openstack-nova | 00:51 | |
*** tkajinam has quit IRC | 00:56 | |
*** tkajinam has joined #openstack-nova | 00:59 | |
*** Liang__ has joined #openstack-nova | 01:10 | |
*** brinzhang has joined #openstack-nova | 01:13 | |
*** brinzhang_ has quit IRC | 01:17 | |
*** nicolasbock has quit IRC | 01:18 | |
*** zhanglong has joined #openstack-nova | 01:23 | |
*** TxGirlGeek has quit IRC | 01:28 | |
*** gyee has quit IRC | 01:34 | |
*** yedongcan has joined #openstack-nova | 01:36 | |
*** igordc has quit IRC | 01:45 | |
*** dviroel has quit IRC | 01:57 | |
*** Liang__ is now known as LiangFang | 02:26 | |
*** LiangFang has quit IRC | 02:32 | |
*** Liang__ has joined #openstack-nova | 02:32 | |
*** tetsuro has joined #openstack-nova | 02:33 | |
*** Liang__ has quit IRC | 02:34 | |
*** Liang__ has joined #openstack-nova | 02:34 | |
*** tkajinam has quit IRC | 02:36 | |
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 |
---|---|---|
openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | 02:37 |
*** davee_ has quit IRC | 02:38 | |
*** davee__ has quit IRC | 02:38 | |
openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | 02:48 |
openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | 02:48 |
*** davee_ has joined #openstack-nova | 02:55 | |
*** davee__ has joined #openstack-nova | 02:55 | |
*** tkajinam has joined #openstack-nova | 03:02 | |
*** awalende has joined #openstack-nova | 03:24 | |
*** awalende has quit IRC | 03:29 | |
*** psachin has joined #openstack-nova | 03:36 | |
*** hongbin has joined #openstack-nova | 03:50 | |
*** macz has joined #openstack-nova | 03:56 | |
*** brault has quit IRC | 03:58 | |
*** psachin has quit IRC | 04:09 | |
*** bhagyashris has joined #openstack-nova | 04:18 | |
*** bhagyashris_ has joined #openstack-nova | 04:20 | |
*** udesale has joined #openstack-nova | 04:21 | |
*** jhesketh has quit IRC | 04:23 | |
*** bhagyashris has quit IRC | 04:23 | |
*** jhesketh has joined #openstack-nova | 04:24 | |
*** sapd1_x has joined #openstack-nova | 04:25 | |
*** rchurch_ has quit IRC | 04:34 | |
*** rchurch has joined #openstack-nova | 04:35 | |
*** TxGirlGeek has joined #openstack-nova | 04:52 | |
*** hongbin has quit IRC | 05:05 | |
*** macz has quit IRC | 05:06 | |
*** macz has joined #openstack-nova | 05:07 | |
*** hongbin has joined #openstack-nova | 05:14 | |
*** hongbin has quit IRC | 05:19 | |
*** gentoorax has quit IRC | 05:27 | |
*** gentoorax has joined #openstack-nova | 05:27 | |
*** evrardjp has quit IRC | 05:33 | |
*** evrardjp has joined #openstack-nova | 05:34 | |
*** ratailor has joined #openstack-nova | 05:38 | |
*** ratailor_ has joined #openstack-nova | 05:42 | |
*** ratailor has quit IRC | 05:45 | |
*** lvbin02 has joined #openstack-nova | 05:50 | |
*** sapd1_x has quit IRC | 05:53 | |
*** lvbin01 has quit IRC | 05:53 | |
*** lvbin01 has joined #openstack-nova | 05:55 | |
*** lvbin02 has quit IRC | 05:55 | |
*** ociuhandu has joined #openstack-nova | 05:56 | |
*** ociuhandu has quit IRC | 06:00 | |
*** macz has quit IRC | 06:06 | |
*** pcaruana has quit IRC | 06:17 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Populate 'updated_at' when aggregate is updated https://review.opendev.org/580271 | 06:27 |
*** yedongcan has quit IRC | 06:34 | |
*** bhagyashris_ has quit IRC | 06:38 | |
*** bhagyashris_ has joined #openstack-nova | 06:42 | |
*** hamzy has quit IRC | 06:49 | |
*** hamzy has joined #openstack-nova | 06:50 | |
*** tetsuro has quit IRC | 06:53 | |
*** tetsuro has joined #openstack-nova | 06:54 | |
*** ccamacho has quit IRC | 07:00 | |
*** TxGirlGeek has quit IRC | 07:05 | |
*** ralonsoh has joined #openstack-nova | 07:12 | |
*** tesseract has joined #openstack-nova | 07:41 | |
*** bhagyashris_ is now known as bhagyashris | 07:45 | |
openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: report storage bus traits https://review.opendev.org/666914 | 07:47 |
*** ociuhandu has joined #openstack-nova | 07:52 | |
*** ociuhandu has quit IRC | 07:52 | |
*** rcernin has quit IRC | 07:52 | |
*** ociuhandu has joined #openstack-nova | 07:55 | |
*** damien_r has joined #openstack-nova | 07:59 | |
*** damien_r has quit IRC | 07:59 | |
brinzhang | gibi: are you around? | 08:00 |
*** damien_r has joined #openstack-nova | 08:00 | |
*** slaweq has joined #openstack-nova | 08:02 | |
*** maciejjozefczyk has joined #openstack-nova | 08:06 | |
*** awalende has joined #openstack-nova | 08:06 | |
*** ircuser-1 has joined #openstack-nova | 08:07 | |
*** tosky has joined #openstack-nova | 08:08 | |
*** bhagyashris has quit IRC | 08:11 | |
*** zbr|rover has quit IRC | 08:14 | |
*** zbr has joined #openstack-nova | 08:15 | |
*** yedongcan has joined #openstack-nova | 08:16 | |
openstackgerrit | Alexandre arents proposed openstack/nova master: Avoid allocation leak when deleting instance stuck in BUILD https://review.opendev.org/702368 | 08:20 |
*** ccamacho has joined #openstack-nova | 08:20 | |
*** dpawlik has quit IRC | 08:22 | |
*** tkajinam has quit IRC | 08:24 | |
*** bhagyashris has joined #openstack-nova | 08:26 | |
*** dpawlik has joined #openstack-nova | 08:27 | |
*** iurygregory has joined #openstack-nova | 08:37 | |
*** Liang__ is now known as LiangFang | 08:42 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: virt: Provide block_device_info during rescue https://review.opendev.org/700811 | 08:46 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add support for stable device rescue https://review.opendev.org/700812 | 08:46 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP compute: Report COMPUTE_BFV_RESCUE and check during rescue https://review.opendev.org/701429 | 08:46 |
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 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Support boot from volume instance rescue https://review.opendev.org/701431 | 08:46 |
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 |
openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Add action event fault details https://review.opendev.org/699669 | 08:47 |
*** martinkennelly has joined #openstack-nova | 08:56 | |
*** bhagyashris has quit IRC | 08:56 | |
*** pcaruana has joined #openstack-nova | 08:56 | |
*** ccamacho has quit IRC | 09:00 | |
openstackgerrit | waleed mousa proposed openstack/os-vif master: [Follow Up] OVS DPDK port representors support https://review.opendev.org/665965 | 09:06 |
*** xek has joined #openstack-nova | 09:08 | |
*** avolkov has joined #openstack-nova | 09:12 | |
*** rpittau|afk is now known as rpittau | 09:16 | |
*** bhagyashris has joined #openstack-nova | 09:21 | |
*** zhanglong has quit IRC | 09:37 | |
*** derekh has joined #openstack-nova | 09:39 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/nova master: Add ironic hypervisor doc https://review.opendev.org/701998 | 09:44 |
*** Liang__ has joined #openstack-nova | 09:48 | |
*** LiangFang has quit IRC | 09:51 | |
*** ccamacho has joined #openstack-nova | 09:52 | |
*** Liang__ has quit IRC | 09:55 | |
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 | 09:56 |
*** ociuhandu has quit IRC | 09:57 | |
*** ociuhandu has joined #openstack-nova | 10:02 | |
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:04 |
gibi | brinzhang: ack, will look | 10:05 |
brinzhang | gibi: thanks~ | 10:09 |
*** ociuhandu has quit IRC | 10:22 | |
*** pcaruana has quit IRC | 10:23 | |
gibi | brinzhang: I'm +2 on the instance action spec | 10:25 |
gibi | thanks for the update | 10:25 |
brinzhang | gibi: cool, thanks ^^ | 10:26 |
*** CeeMac has joined #openstack-nova | 10:30 | |
*** ociuhandu has joined #openstack-nova | 10:35 | |
*** pcaruana has joined #openstack-nova | 11:02 | |
*** damien_r has quit IRC | 11:04 | |
*** rpittau is now known as rpittau|bbl | 11:11 | |
*** ociuhandu has quit IRC | 11:13 | |
openstackgerrit | Arnaud Morin proposed openstack/nova master: Force refresh of instance info cache on event https://review.opendev.org/702394 | 11:13 |
*** yedongcan has left #openstack-nova | 11:15 | |
*** dviroel has joined #openstack-nova | 11:16 | |
stephenfin | bauzas: Can you send this on its way, please? https://review.opendev.org/#/c/695733/ | 11:18 |
*** ociuhandu has joined #openstack-nova | 11:21 | |
alex_xu | lyarwood: got it | 11:26 |
*** udesale has quit IRC | 11:28 | |
*** lvbin01 has quit IRC | 11:28 | |
openstackgerrit | Merged openstack/nova-specs master: Spec: Ussuri: Encrypted Emulated Virtual TPM https://review.opendev.org/686804 | 11:29 |
luyao | stephenfin: Could you look at spec 'support live migration with vpmem'? https://review.opendev.org/#/c/695863 | 11:30 |
stephenfin | luyao: I can, but not today. Could you ask me again tomorrow? | 11:31 |
* stephenfin is just back from PTO | 11:31 | |
luyao | stephenfin: OK, thanks at advance. :) | 11:31 |
alex_xu | lyarwood: gibi https://review.opendev.org/#/c/694063/7/specs/ussuri/approved/virt-bfv-instance-rescue.rst@76 | 11:42 |
*** tbachman has quit IRC | 11:43 | |
*** ratailor_ has quit IRC | 11:51 | |
*** dtantsur|afk is now known as dtantsur | 12:03 | |
*** ociuhandu has quit IRC | 12:03 | |
*** etingof has joined #openstack-nova | 12:12 | |
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:14 |
openstackgerrit | Luyao Zhong proposed openstack/nova-specs master: support live migration with virtual persistent memory https://review.opendev.org/695863 | 12:15 |
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:23 |
sean-k-mooney | it looks like there are still issue however http://paste.openstack.org/show/788372/ | 12:24 |
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:25 |
*** Luzi has joined #openstack-nova | 12:26 | |
*** dpawlik has quit IRC | 12:26 | |
*** dpawlik has joined #openstack-nova | 12:28 | |
huaqiang | hello stephenfin. I hope you enjoyed your vacation | 12:31 |
huaqiang | I also want to invite your to review https://review.opendev.org/#/c/668656/ | 12:32 |
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 |
huaqiang | I hope to get your comments. | 12:34 |
*** damien_r has joined #openstack-nova | 12:35 | |
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:37 |
*** jangutter has quit IRC | 12:37 | |
*** ociuhandu has joined #openstack-nova | 12:39 | |
lyarwood | gibi / alex_xu ; apologies just back from a long lunch, yeah it's intentional, I'll add a comment now. | 12:39 |
gibi | lyarwood: cool thanks | 12:39 |
*** ociuhandu has quit IRC | 12:43 | |
brinzhang | This bug fix is ready to review, if you have free time, please review https://review.opendev.org/#/c/580271/ | 12:47 |
brinzhang | it's bug 1663456 | 12:48 |
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 |
openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Boot from volume instance rescue https://review.opendev.org/694063 | 12:54 |
lyarwood | ^ addressed the commit nit btw | 12:54 |
sean-k-mooney | dansmith: efried: ok so the cyborg series just need to be rebased on https://review.opendev.org/#/c/699050/ | 12:55 |
sean-k-mooney | well that is merges so rebaseing on master would be enough | 12:56 |
gibi | lyarwood: +Ad the spec | 13:00 |
gibi | lyarwood: thanks for the confirmation | 13:00 |
lyarwood | gibi: awesome thanks :) | 13:03 |
*** ociuhandu has joined #openstack-nova | 13:03 | |
*** tbachman has joined #openstack-nova | 13:05 | |
*** tbachman_ has joined #openstack-nova | 13:07 | |
*** tbachman has quit IRC | 13:10 | |
*** tbachman_ is now known as tbachman | 13:10 | |
*** bbowen has joined #openstack-nova | 13:12 | |
sean-k-mooney | stephenfin: can you review this when you get a chance https://review.opendev.org/#/c/701601/ | 13:19 |
*** iurygregory has quit IRC | 13:23 | |
*** iurygregory has joined #openstack-nova | 13:23 | |
*** rpittau|bbl is now known as rpittau | 13:26 | |
openstackgerrit | Merged openstack/nova-specs master: Boot from volume instance rescue https://review.opendev.org/694063 | 13:34 |
*** bhagyashris has quit IRC | 13:41 | |
stephenfin | huaqiang: As with luyao, if you can ask me again tomorrow I'll try get to it. Too much to do today :( | 13:48 |
*** nweinber__ has joined #openstack-nova | 13:48 | |
*** liuyulong has joined #openstack-nova | 13:53 | |
*** tkajinam has joined #openstack-nova | 13:56 | |
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. | 13:57 |
*** mlavalle has joined #openstack-nova | 14:01 | |
stephenfin | sean-k-mooney: done | 14:01 |
*** ociuhandu has quit IRC | 14:03 | |
huaqiang | stephenfin: Understand. Don't worry. I will connect you later :) | 14:04 |
stephenfin | cool, thanks :) | 14:04 |
*** ociuhandu has joined #openstack-nova | 14:05 | |
*** eharney has joined #openstack-nova | 14:16 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove unused nova-network objects https://review.opendev.org/697156 | 14:16 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove now unnecessary nova-network workaround https://review.opendev.org/702440 | 14:16 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove now unnecessary nova-net workaround https://review.opendev.org/702440 | 14:17 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove unused nova-network objects https://review.opendev.org/697156 | 14:17 |
*** iurygregory has quit IRC | 14:18 | |
stephenfin | gibi: I addressed your comments in https://review.opendev.org/#/c/696516/ Think you could revisit at some point? | 14:18 |
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:19 |
*** ociuhandu has quit IRC | 14:29 | |
*** udesale has joined #openstack-nova | 14:30 | |
*** iurygregory has joined #openstack-nova | 14:33 | |
*** amodi has joined #openstack-nova | 14:35 | |
*** awalende has quit IRC | 14:36 | |
openstackgerrit | sean mooney proposed openstack/os-vif master: [DNM]test composing devstack_local_conf sections https://review.opendev.org/702446 | 14:40 |
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:41 |
sean-k-mooney | so i expect that might fail | 14:42 |
*** liuyulong has quit IRC | 14:45 | |
stephenfin | ack | 14:45 |
*** READ10 has joined #openstack-nova | 14:45 | |
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:46 |
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 |
stephenfin | ganso: kashyap might be able to help you with that, if they're around ^ | 14:47 |
kashyap | Already reading :-) | 14:47 |
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:48 |
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 |
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 |
kashyap | ganso: QEMU has added newer variants of CPU models (with affected flags disabled) that you can directly specify with Nova | 14:49 |
kashyap | ganso: So, for the recent "TSX" vulnerability fiasco ... | 14:50 |
kashyap | ganso: ... QEMU / libvirt has added *-noTSX CPU models. | 14:51 |
kashyap | ganso: E.g. on my Fedora host, running qemu-system-x86-4.2.0-2.fc30.x86_64: | 14:52 |
*** vesper has quit IRC | 14:53 | |
kashyap | $> qemu-system-x86_64 -cpu help | egrep *.noTSX* | 14:53 |
kashyap | x86 Broadwell-noTSX (alias of Broadwell-v2) | 14:53 |
kashyap | x86 Broadwell-noTSX-IBRS (alias of Broadwell-v4) | 14:53 |
kashyap | x86 Cascadelake-Server-noTSX (alias of Cascadelake-Server-v3) | 14:53 |
*** vesper11 has joined #openstack-nova | 14:53 | |
kashyap | x86 Haswell-noTSX (alias of Haswell-v2) | 14:53 |
kashyap | x86 Haswell-noTSX-IBRS (alias of Haswell-v4) | 14:53 |
kashyap | x86 Icelake-Client-noTSX (alias of Icelake-Client-v2) | 14:53 |
kashyap | x86 Icelake-Server-noTSX (alias of Icelake-Server-v2) | 14:53 |
kashyap | x86 Skylake-Client-noTSX-IBRS (alias of Skylake-Client-v3) | 14:53 |
kashyap | x86 Skylake-Server-noTSX-IBRS (alias of Skylake-Server-v3) | 14:53 |
kashyap | --- | 14:53 |
ganso | kashyap: cool, so the new name needs to get to the instance XML | 14:53 |
kashyap | ganso: So if you've got libvirt/QEMU versions with noTSX stuff (confirm by running the above version), then you can use those model names in your nova.conf [libvirt] section | 14:53 |
kashyap | ganso: Yes, indeed. | 14:54 |
ganso | kashyap: ok, replacing them in nova.conf [libvirt] section will only affect new VMs or previously existing ones as well? | 14:54 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: nova-net: Remove unused nova-network objects https://review.opendev.org/697156 | 14:54 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module https://review.opendev.org/702451 | 14:54 |
kashyap | ganso: Yes, it will affect — once you reboot the Compute node — _all_ the VMs re-started on that node | 14:55 |
kashyap | You'd want something like: | 14:55 |
kashyap | [libvirt] | 14:55 |
kashyap | cpu_mode = custom | 14:55 |
kashyap | cpu_models = Haswell-noTSX-IBRS | 14:55 |
kashyap | cpu_model_extra_flags = pcid,spec-ctrl,ssbd,md-clear | 14:55 |
kashyap | ganso: And also, related reference: https://docs.openstack.org/nova/latest/admin/mitigation-for-Intel-MDS-security-flaws.html | 14:56 |
kashyap | Speaking of which ... /me should probably one for the not-so-fresh-off-the-oven "TAA" (TSX Asynchronous Abort) CVE | 14:57 |
ganso | kashyap: thanks for the clarification! :D | 14:57 |
ganso | kashyap: I will talk to the customer about this approach, should get the impact sorted out | 14:58 |
kashyap | No problem; this whole space is a barrel of cockroaches | 14:58 |
dansmith | sean-k-mooney: yep (re: the rebase) | 14:59 |
*** Luzi has quit IRC | 14:59 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: pre-commit: Use Python 3 to run checks https://review.opendev.org/702453 | 15:00 |
sean-k-mooney | it look like some fo the cyborg code is still pending by the way | 15:01 |
*** artom has joined #openstack-nova | 15:02 | |
sean-k-mooney | specificly https://review.opendev.org/#/c/698190/ for the deploables v2 api although i dont know if that is needed for nova integration | 15:03 |
sean-k-mooney | the devices v2 patches merged today https://review.opendev.org/#/c/695648/ | 15:04 |
stephenfin | efried, bauzas: Since you reviewed the original, could you blast this through? It's hurting me when working on some of the newly Python 3-only files we now have https://review.opendev.org/702453 | 15:05 |
*** awalende has joined #openstack-nova | 15:06 | |
*** ociuhandu has joined #openstack-nova | 15:06 | |
efried | sean-k-mooney: would you +1 that ^ and I'll fast approve? | 15:08 |
efried | lyarwood: can you please tag the commit message for https://review.opendev.org/#/c/694033/ with the `blueprint xxx-xxx-xxx` magic? | 15:09 |
sean-k-mooney | the precommit change yes runnign it with python3 makes sesne give me a sec and ill add a link in a comment to the relevent docs | 15:10 |
*** tkajinam has quit IRC | 15:10 | |
lyarwood | efried: ack, I'll do that now. | 15:10 |
efried | lyarwood: when was the blueprint set to Definition:Approved and by whom? | 15:10 |
*** awalende has quit IRC | 15:11 | |
efried | (I thought there used to be a History button, but I must be thinking of something else) | 15:11 |
*** ociuhandu has quit IRC | 15:11 | |
efried | sean-k-mooney: yes, cyborg series should be rebased to use root_required | 15:11 |
*** maciejjozefczyk has quit IRC | 15:12 | |
efried | okay, I think I'm caught up | 15:12 |
lyarwood | efried: I'm not sure about https://blueprints.launchpad.net/nova/+spec/virt-rescue-stable-disk-devices but https://blueprints.launchpad.net/nova/+spec/virt-bfv-instance-rescue for this change and spec still needs approval | 15:12 |
lyarwood | efried: ah found the email, MattR approved virt-rescue-stable-disk-devices | 15:13 |
efried | FYI nova, we've merged merged https://review.opendev.org/#/c/701792/ which should get rid of the "multiple possible networks" tempest errors we've been seeing a lot of lately. If you see more of them, let me know, cause the fix should be simple. | 15:13 |
efried | stephenfin: ^ | 15:13 |
efried | "we've merged merged"? #uncaffeinated | 15:14 |
artom | Question about bug triage | 15:14 |
artom | " Close as "invalid" if it is a support request or feature request." from https://wiki.openstack.org/wiki/Nova/BugTriage | 15:14 |
artom | Is that a thing we do? | 15:14 |
efried | lyarwood: would you mind adding a note for that. But... why are there two blueprints? (I haven't actually *read* anything, so maybe it's obvious, but lead me by the nose here) | 15:14 |
artom | Specifically about https://bugs.launchpad.net/nova/+bug/1859403 | 15:14 |
openstack | Launchpad bug 1859403 in OpenStack Compute (nova) "The instance needs to supports dongle devices" [Wishlist,New] | 15:14 |
efried | artom: this actually came up last week and I didn't consult that wiki page. | 15:14 |
openstackgerrit | Lee Yarwood proposed openstack/os-traits master: Add COMPUTE_BFV_RESCUE trait https://review.opendev.org/694033 | 15:15 |
artom | I set it to wishlist, is that it? Or since we know what it is (wishlist), we can set it to Triaged as well to remove it from filters/lists | 15:15 |
efried | dansmith opened a pro forma bug for something, trying to remember... | 15:15 |
dansmith | "opened a pro forma bug....against his will" | 15:16 |
efried | yeah yeah | 15:18 |
efried | dansmith: this one rite? https://bugs.launchpad.net/nova/+bug/1858877 | 15:18 |
openstack | Launchpad bug 1858877 in OpenStack Compute (nova) "Silent wasted storage with multiple RBD backends" [Wishlist,Confirmed] | 15:18 |
dansmith | yar | 15:18 |
efried | Since artom points to an actual document saying we should close as Invalid, Ima do that. | 15:18 |
efried | and artom, yeah, do same with yours. | 15:18 |
dansmith | lol | 15:18 |
artom | efried, well, wikis are fickle things | 15:18 |
efried | I was looking for any excuse | 15:18 |
artom | I could edit that to say every time a wishlist is filed, we need to call POTUS and get his approval | 15:18 |
efried | I think if we get to 100 open bugs, the stay-puft marshmallow man shows up or something. | 15:19 |
efried | Then we have to cross the streams, marshmallow everywhere, big mess. | 15:19 |
*** maciejjozefczyk has joined #openstack-nova | 15:19 | |
gibi | stephenfin: I'm +2 on https://review.opendev.org/#/c/696516/ based on your answers | 15:19 |
stephenfin | gibi: thanks | 15:19 |
artom | efried, follow-up question then - if we close wishlist items as invalid, how do we expect folks to file them? | 15:19 |
efried | blueprints, right? | 15:20 |
artom | You're asking me? | 15:20 |
gibi | stephenfin: regarding the rename in https://review.opendev.org/#/c/696745/ I can be convinced both ways so I'm not voting now, sorry | 15:20 |
dansmith | yes, my point about my bug was that "nova does not support $thing" is not a bug, it's a feature request, hence blueprint at some point | 15:20 |
dansmith | however, we don't file blueprints for things we may or may not do in the future | 15:20 |
stephenfin | gibi: All good. Comes down to dansmith in that case | 15:21 |
dansmith | (nor should we) | 15:21 |
efried | Right, there seems to be a process gap there. | 15:21 |
dansmith | why? | 15:21 |
artom | IOW, if you have a feature request but aren't prepared to work on it yourself or pay someone to do it, your sool? | 15:21 |
artom | *you're | 15:21 |
efried | if there's a closed/wishlist bug, how do we ever "find" it and make a blueprint out of it. | 15:21 |
dansmith | we don't need to enumerate everything nova doesn't do, right? | 15:21 |
efried | of course, but a way to track "something nova doesn't do, but probably should, but nobody's going to work on it right now, but we don't really want to forget about it and *never* do it" | 15:22 |
efried | dansmith: I think your bug is a good example. You said it's something we probably want to do soonish. But who's gonna remember? | 15:22 |
artom | I mean, given the project/community dynamics, it'd be reasonable to say "sorry, if you want a thing but can't commit resources, it'll realistically never get done." | 15:22 |
artom | Not very welcoming, but reasonable | 15:22 |
dansmith | efried: if putting together that list is actually going to form a backlog that we chew through, then sure, but history tells me it will just become a wasteland of every crazy thing anyone ever thought of once | 15:23 |
dansmith | if it was a cultivated list that was pruned to just the things we do actually want to do then sure, but.. closed wishlist bugs are not that :) | 15:24 |
efried | okay, well, I closed https://bugs.launchpad.net/nova/+bug/1858877 as invalid, so artom you have a recent precedent as well as instructions from the wiki. Go forth and close. | 15:26 |
openstack | Launchpad bug 1858877 in OpenStack Compute (nova) "Silent wasted storage with multiple RBD backends" [Wishlist,Invalid] | 15:26 |
dansmith | stephenfin: I saw lots of mentions as my backscroll flowed in this morning.. if you want me to look at something specific, please relink me | 15:26 |
stephenfin | dansmith: https://review.opendev.org/#/c/696745/ (see last comment) | 15:30 |
dansmith | oh jesus | 15:31 |
dansmith | I still would rather it not change, mostly for selfish reasons, but I'm sure I'm the only one (left) that feels that way | 15:32 |
dansmith | so please just don't make me review it | 15:32 |
*** udesale_ has joined #openstack-nova | 15:34 | |
*** udesale has quit IRC | 15:37 | |
stephenfin | :D | 15:38 |
stephenfin | fair | 15:38 |
stephenfin | I know the refactoring impact is still present, but does the use of a 'nova.network.neutron' module (vs. 'nova.network.api') resolve your other concerns at least? | 15:40 |
stephenfin | iiuc that was mriedem's primary concern (that people would see nova.network.api and think we were talking abut nova-network) | 15:40 |
*** ociuhandu has joined #openstack-nova | 15:42 | |
dansmith | it's better in terms of the confusion, yeah | 15:46 |
efried | lyarwood: removed -2, but left -1 for discrepancy with the spec. Honestly don't know which would be better tho. | 15:48 |
lyarwood | efried: ack yeah apologies I totally forgot to update this in the spec after switching here. I really don't mind tbh so I'll just change this back to COMPUTE_RESCUE_BFV to avoid editing the spec. | 15:56 |
openstackgerrit | Lee Yarwood proposed openstack/os-traits master: Add COMPUTE_RESCUE_BFV trait https://review.opendev.org/694033 | 15:58 |
*** maciejjozefczyk has quit IRC | 16:02 | |
*** ccamacho has quit IRC | 16:02 | |
*** TxGirlGeek has joined #openstack-nova | 16:02 | |
efried | lyarwood: I honestly want thought put into which is better, actually. | 16:09 |
efried | We don't need to belabor it, but avoiding editing one patch or the other shouldn't be the reason. | 16:10 |
*** Sundar has joined #openstack-nova | 16:10 | |
lyarwood | efried: yeah that's fair, well an actual reason to switch back to COMPUTE_RESCUE_BFV would be that it would allow additional traits to be added in the future | 16:10 |
lyarwood | sorry I mean additional rescue traits, COMPUTE_RESCUE_FOO etc | 16:11 |
efried | that's my point: do we want to add more COMPUTE_RESCUE_* traits, or do we want to add more COMPUTE_BFV_* traits? | 16:11 |
efried | okay, so COMPUTE_RESCUE_BFV is actually preferred. I can dig that. | 16:12 |
*** arxcruz is now known as arxcruz|off | 16:13 | |
efried | lyarwood: +2 | 16:14 |
lyarwood | efried: thanks | 16:14 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add NovaEphemeralObject class for non-persistent objects https://review.opendev.org/702049 | 16:14 |
efried | stephenfin: +A on the precommit gizmo, sorry for the delay. And thanks for the vTPM spec approval \o/ | 16:15 |
*** dosaboy has quit IRC | 16:28 | |
*** ganso has quit IRC | 16:28 | |
sean-k-mooney | efried: is the blueprint/spec approval deadline m2? | 16:30 |
sean-k-mooney | im just looking at my downstream feature backlog and i might file one more blueprint for this cycle to add viommu support to th elibvirt driver. it will just be a small change like the vpmu change but just tring to figure out when i need to decided if im going to file it | 16:32 |
*** hamzy has quit IRC | 16:33 | |
sean-k-mooney | i want to finish the image metadata prefiltering work first but the viommu feature is tecnically more useful to end customers | 16:34 |
*** awalende has joined #openstack-nova | 16:37 | |
*** dave-mccowan has joined #openstack-nova | 16:38 | |
*** awalende has quit IRC | 16:42 | |
*** gyee has joined #openstack-nova | 16:42 | |
*** udesale_ has quit IRC | 16:43 | |
efried | sean-k-mooney: spec freeze 2/13 https://wiki.openstack.org/wiki/Nova/Ussuri_Release_Schedule yes. At that point I'll probably ask us to do a scrub to prioritize whatever's been Definition:Approved and drop some things off the bottom. | 16:49 |
sean-k-mooney | ya i was execpting there to be a scrub | 16:50 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove extra instance.save() calls related to qos SRIOV ports https://review.opendev.org/702261 | 16:51 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use common server create function for qos func tests https://review.opendev.org/701353 | 16:51 |
sean-k-mooney | my main two feature that were important for this cycle have already merged so the rest are less imporant. i want to finish stuff i have started before doing new stuff too so im lean to not filing another blueprint but i also know its will be trivail to do | 16:52 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enable live migration with qos ports https://review.opendev.org/699066 | 16:53 |
*** macz has joined #openstack-nova | 16:54 | |
*** maciejjozefczyk has joined #openstack-nova | 16:55 | |
*** rpittau is now known as rpittau|afk | 17:01 | |
*** mriedem has joined #openstack-nova | 17:01 | |
*** ganso has joined #openstack-nova | 17:04 | |
*** dpawlik has quit IRC | 17:14 | |
*** ociuhandu_ has joined #openstack-nova | 17:20 | |
*** damien_r has quit IRC | 17:23 | |
*** ociuhandu has quit IRC | 17:24 | |
*** ociuhandu_ has quit IRC | 17:24 | |
*** ociuhandu has joined #openstack-nova | 17:26 | |
*** ociuhandu has quit IRC | 17:28 | |
*** hamzy has joined #openstack-nova | 17:33 | |
*** evrardjp has quit IRC | 17:33 | |
*** evrardjp has joined #openstack-nova | 17:34 | |
*** martinkennelly has quit IRC | 17:36 | |
*** brinzhang has quit IRC | 17:38 | |
*** brinzhang has joined #openstack-nova | 17:39 | |
*** born2bake has joined #openstack-nova | 17:52 | |
*** dosaboy has joined #openstack-nova | 17:59 | |
*** derekh has quit IRC | 18:00 | |
*** maciejjozefczyk has quit IRC | 18:02 | |
*** tbachman has quit IRC | 18:09 | |
*** tbachman has joined #openstack-nova | 18:10 | |
*** maciejjozefczyk has joined #openstack-nova | 18:17 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Use COMPUTE_SAME_HOST_COLD_MIGRATE trait during migrate https://review.opendev.org/695220 | 18:18 |
*** dtantsur is now known as dtantsur|afk | 18:32 | |
*** maciejjozefczyk has quit IRC | 18:56 | |
*** maciejjozefczyk has joined #openstack-nova | 18:59 | |
*** brinzhang_ has joined #openstack-nova | 19:01 | |
*** brinzhang has quit IRC | 19:04 | |
*** brinzhang has joined #openstack-nova | 19:10 | |
*** brinzhang_ has quit IRC | 19:14 | |
*** pcaruana has quit IRC | 19:16 | |
*** factor has joined #openstack-nova | 19:18 | |
*** maciejjozefczyk has quit IRC | 19:26 | |
*** nweinber__ has quit IRC | 19:35 | |
*** nweinber__ has joined #openstack-nova | 19:35 | |
*** gmann is now known as gmann_afk | 19:36 | |
*** macz has quit IRC | 19:50 | |
*** macz has joined #openstack-nova | 19:51 | |
*** slaweq has quit IRC | 20:02 | |
*** ralonsoh has quit IRC | 20:06 | |
*** hamzy_ has joined #openstack-nova | 20:08 | |
*** hamzy has quit IRC | 20:11 | |
efried | dansmith: old, simple, should be an easy +A: https://review.opendev.org/#/c/694806/ | 20:12 |
*** maciejjozefczyk has joined #openstack-nova | 20:13 | |
*** born2bake has quit IRC | 20:15 | |
*** born2bake has joined #openstack-nova | 20:15 | |
*** READ10 has quit IRC | 20:17 | |
*** born2bake has quit IRC | 20:18 | |
*** born2bake has joined #openstack-nova | 20:18 | |
dustinc | efried: I was working on moving checks for no-op providers from the merge step to the validation step as you suggested here https://review.opendev.org/#/c/676029/31/nova/compute/provider_config.py@199 | 20:23 |
dustinc | I realized that there was an unintended change in functionality and am not sure if it is worth worrying about or not | 20:23 |
dustinc | old behavior: if using uuid=$COMPUTE_NODE you could exempt individual providers from the $COMPUTE_NODE entry during merge step by adding specific no-op entries for them | 20:23 |
dustinc | new behavior: since the no-op entry gets removed during validation, the $COMPUTE_NODE value gets used when merging | 20:23 |
*** brinzhang_ has joined #openstack-nova | 20:24 | |
dustinc | the old behavior was not specifically intended or documented, but might actually be useful to someone | 20:24 |
*** maciejjozefczyk has quit IRC | 20:25 | |
efried | dustinc: interesting, glad we caught it before we landed this rather than later when somebody *was* relying on it and we yanked the rug. We should noodle whether we want to allow that or not. | 20:25 |
efried | I'm going to say: for now, we should explicitly document (wherever we're going to document this thing) that no-op stanzas will be ignored completely and have no effect. | 20:26 |
efried | which means the new way you're doing the check will be the right way. | 20:26 |
efried | "ignored completely" with a warning (which I guess isn't ignored completely, but you get the idea) | 20:26 |
*** brinzhang has quit IRC | 20:27 | |
*** maciejjozefczyk has joined #openstack-nova | 20:29 | |
*** N3l1x has joined #openstack-nova | 20:31 | |
*** Sundar has quit IRC | 20:33 | |
*** artom has quit IRC | 20:33 | |
*** slaweq has joined #openstack-nova | 20:33 | |
*** N3l1x has quit IRC | 20:34 | |
*** N3l1x has joined #openstack-nova | 20:34 | |
*** N3l1x has quit IRC | 20:35 | |
*** N3l1x has joined #openstack-nova | 20:35 | |
*** damien_r has joined #openstack-nova | 20:37 | |
*** awalende has joined #openstack-nova | 20:38 | |
dustinc | Sounds good to me, I want to push what I have so will go that route for now | 20:39 |
*** rchurch has quit IRC | 20:39 | |
*** rchurch has joined #openstack-nova | 20:40 | |
*** damien_r has quit IRC | 20:41 | |
*** awalende has quit IRC | 20:42 | |
*** brinzhang has joined #openstack-nova | 20:43 | |
*** hamzy__ has joined #openstack-nova | 20:43 | |
*** bbowen has quit IRC | 20:45 | |
*** hamzy_ has quit IRC | 20:46 | |
*** brinzhang_ has quit IRC | 20:46 | |
*** tesseract has quit IRC | 20:50 | |
*** maciejjozefczyk has quit IRC | 20:50 | |
*** hamzy_ has joined #openstack-nova | 20:51 | |
*** hamzy__ has quit IRC | 20:53 | |
*** hamzy__ has joined #openstack-nova | 21:02 | |
*** hamzy_ has quit IRC | 21:04 | |
*** hamzy_ has joined #openstack-nova | 21:10 | |
*** avolkov has quit IRC | 21:12 | |
*** hamzy__ has quit IRC | 21:13 | |
*** nweinber__ has quit IRC | 21:17 | |
*** brinzhang_ has joined #openstack-nova | 21:21 | |
*** hamzy has joined #openstack-nova | 21:22 | |
*** hamzy_ has quit IRC | 21:23 | |
*** brinzhang has quit IRC | 21:24 | |
openstackgerrit | Eric Fried proposed openstack/nova master: nova-net: Kill it https://review.opendev.org/696518 | 21:25 |
openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: YAML file loading and schema validation https://review.opendev.org/673341 | 21:49 |
openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | 21:49 |
openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | 21:49 |
openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | 21:49 |
*** hamzy_ has joined #openstack-nova | 21:50 | |
*** tbachman has quit IRC | 21:53 | |
*** hamzy has quit IRC | 21:53 | |
*** bbowen has joined #openstack-nova | 21:54 | |
*** hamzy_ has quit IRC | 21:58 | |
*** hamzy has joined #openstack-nova | 21:59 | |
*** Sundar has joined #openstack-nova | 22:05 | |
Sundar | Hello all, in release notes like https://github.com/openstack/nova/blame/master/releasenotes/notes/image-precaching-d46506568fefa1ea.yaml, what is the id that gets added at the end? | 22:06 |
*** hamzy_ has joined #openstack-nova | 22:06 | |
*** rcernin has joined #openstack-nova | 22:08 | |
*** hamzy has quit IRC | 22:09 | |
*** tbachman has joined #openstack-nova | 22:12 | |
*** brinzhang has joined #openstack-nova | 22:20 | |
*** mvkr has quit IRC | 22:21 | |
dansmith | Sundar: don't hand-create them | 22:23 |
dansmith | Sundar: tox -evenv reno new some-short-name-of-thing | 22:23 |
*** brinzhang_ has quit IRC | 22:23 | |
Sundar | dansmith: do I use that to create a file which I then 'git add'? | 22:24 |
dansmith | Sundar: yes | 22:25 |
Sundar | Thanks | 22:25 |
*** tbachman has quit IRC | 22:30 | |
openstackgerrit | Merged openstack/nova master: Avoid spurious error logging in _get_compute_nodes_in_db https://review.opendev.org/695453 | 22:31 |
*** xek has quit IRC | 22:33 | |
*** slaweq has quit IRC | 22:34 | |
*** N3l1x has quit IRC | 22:40 | |
*** gmann_afk is now known as gmann | 22:53 | |
*** tkajinam has joined #openstack-nova | 22:55 | |
*** igordc has joined #openstack-nova | 22:58 | |
*** igordc has quit IRC | 23:01 | |
*** igordc has joined #openstack-nova | 23:01 | |
*** igordc has quit IRC | 23:02 | |
*** igordc has joined #openstack-nova | 23:03 | |
*** igordc has quit IRC | 23:06 | |
*** igordc has joined #openstack-nova | 23:07 | |
*** igordc has quit IRC | 23:09 | |
*** igordc has joined #openstack-nova | 23:10 | |
efried | dustinc: finished reviewing the provider config series. Getting close, nice work. | 23:10 |
*** igordc has quit IRC | 23:11 | |
*** igordc has joined #openstack-nova | 23:12 | |
dustinc | efried: thanks, checking it out now.. | 23:12 |
*** igordc has quit IRC | 23:14 | |
*** igordc has joined #openstack-nova | 23:15 | |
*** tosky has quit IRC | 23:15 | |
efried | aaand I'm out | 23:16 |
*** igordc has quit IRC | 23:19 | |
*** awalende has joined #openstack-nova | 23:21 | |
*** awalende has quit IRC | 23:25 | |
*** mvkr has joined #openstack-nova | 23:27 | |
*** ociuhandu has joined #openstack-nova | 23:30 | |
*** ociuhandu has quit IRC | 23:35 | |
*** dviroel has quit IRC | 23:37 | |
*** hamzy_ has quit IRC | 23:38 | |
*** tbachman has joined #openstack-nova | 23:38 | |
*** tbachman_ has joined #openstack-nova | 23:44 | |
*** mriedem has left #openstack-nova | 23:46 | |
*** tbachman has quit IRC | 23:46 | |
*** tbachman_ is now known as tbachman | 23:46 | |
*** born2bake has quit IRC | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!