| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-10-26 | |||
| 14:17:37 | bauzas | sean-k-mooney: but for your customer not understanding the cloud principles, please also tell him to rephrase correctly | |
| 14:17:38 | sean-k-mooney | frickler: we might port some thing by hand if we need too but if we did i would want to treat it as a bug scrub exersize | |
| 14:18:16 | sean-k-mooney | we have less then 30 stories for placment i think total | |
| 14:18:35 | sean-k-mooney | + a couple for osc-placment plugin | |
| 14:18:50 | sean-k-mooney | so thats the other reason im not too worried about tooling | |
| 14:18:56 | sean-k-mooney | other project are proably not as lucky | |
| 14:19:25 | frickler | sean-k-mooney: that sounds manageable indeed and combining with scrubbing is a good idea, too | |
| 14:21:30 | sean-k-mooney | bauzas: well i just tool them not until at least osp 20+ | |
| 14:21:49 | sean-k-mooney | for non redhattser we released 17 recently :) | |
| 14:22:08 | sean-k-mooney | in ohter words if we get to it it wont be for a few years | |
| 14:22:09 | bauzas | sean-k-mooney: that's a long ask for live-resize | |
| 14:22:27 | bauzas | my only concern is not whether we should do it, but about the term too | |
| 14:23:04 | bauzas | 'can I attach a new cpu live to a running guest" is an acceptable QEMU feature, but this is horribly not cloudy | |
| 14:23:29 | bauzas | 'can I resize my running instance with a new flavor so I could have one more CPU' is the correct way to ask | |
| 14:23:38 | sean-k-mooney | ya libvirt and qemu can do it althogh you need to know the maxium number of cpu you might have ahead of time | |
| 14:24:37 | sean-k-mooney | but agree on the not very cloudy aspect | |
| 14:24:47 | bauzas | I mean, if we nitpick on the hotplug term (that QEMU just hijacked), then I can nitpick on the customer ask saying he doesn't know what a cloud is, then :) | |
| 14:25:24 | sean-k-mooney | well hotplug did not come form the cpu side it came from pci devices orgianly as far as im aware | |
| 14:25:31 | sean-k-mooney | but more relevent topic | |
| 14:25:44 | sean-k-mooney | can you add me to the spec when you push it or ping me | |
| 14:26:00 | sean-k-mooney | ill add it to my review list | |
| 14:26:24 | bauzas | sure | |
| 14:26:32 | bauzas | I'm now working on the prototype | |
| 14:26:53 | bauzas | I was considering a new package structure in hardware | |
| 14:27:27 | sean-k-mooney | you mean creating a folder in the libvirt driver for it? | |
| 14:27:54 | sean-k-mooney | https://github.com/openstack/nova/tree/master/nova/virt/libvirt | |
| 14:27:59 | sean-k-mooney | becide storage and volume | |
| 14:28:09 | sean-k-mooney | if so that proably makes sense yes | |
| 14:28:16 | bauzas | no, turning hardware to be a package | |
| 14:28:21 | bauzas | not a single module | |
| 14:28:27 | bauzas | virt.hardware | |
| 14:28:35 | sean-k-mooney | in theory that is ment to work across drivers | |
| 14:28:36 | bauzas | so we could have virt.hardware.cpu | |
| 14:28:49 | sean-k-mooney | as in parts of it are used by not libvirt | |
| 14:28:58 | bauzas | correct, but I guess cpu online state can be independent | |
| 14:28:58 | sean-k-mooney | but ok you could | |
| 14:29:14 | sean-k-mooney | well it woudl work on any linux | |
| 14:29:15 | bauzas | that tho relies on the kernel and sysds | |
| 14:29:19 | sean-k-mooney | so maybe powervm | |
| 14:29:38 | sean-k-mooney | or zvm | |
| 14:29:38 | bauzas | true, that was my current concerns | |
| 14:30:03 | sean-k-mooney | we dont really have any other driver that woudl use it anymore | |
| 14:30:18 | bauzas | well, actually, you're right, and this would confuse people | |
| 14:30:25 | bauzas | we agreed at the PTG to have this libvirt-specifc | |
| 14:30:29 | bauzas | then, nevermind | |
| 14:30:38 | bauzas | I'll just create a package under libvirt | |
| 14:30:41 | bauzas | libvirt.cpu | |
| 14:30:51 | sean-k-mooney | ack | |
| 14:31:00 | sean-k-mooney | that or put the function into host.py | |
| 14:31:08 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/host.py | |
| 14:31:15 | sean-k-mooney | but its own thing proably is simpler | |
| 14:31:20 | sean-k-mooney | well cleaner | |
| 14:31:34 | bauzas | well, the host module is mostly for talking to the host libvirt API right? | |
| 14:31:43 | sean-k-mooney | mostly but not entirly | |
| 14:31:46 | bauzas | true | |
| 14:32:00 | bauzas | I just want to have some interface | |
| 14:32:11 | sean-k-mooney | we lookup the firmware files for uefi there and i think some vtpm stuff | |
| 14:32:13 | bauzas | but I need to think it more | |
| 14:32:26 | sean-k-mooney | honestly poc it however you feel is best | |
| 14:32:45 | sean-k-mooney | we can debttate it later but i think under the libvirt driver somewhere is the right approch | |
| 14:32:52 | bauzas | tru | |
| 14:32:54 | bauzas | true* | |
| 14:33:03 | sean-k-mooney | beyond that as long as you dont just put it in driver.py im more or less ok with it | |
| 14:33:09 | sean-k-mooney | driver.py is already too big | |
| 14:33:25 | bauzas | :) | |
| 14:34:04 | sean-k-mooney | i know we will never get around to doing this but some day i would like to split up driver.py | |
| 14:34:28 | sean-k-mooney | 10k loc is excessive | |
| 14:37:09 | bauzas | don't (git) blame me :) | |
| 14:54:12 | Uggla | agree driver.py is a bit long. ;) | |
| #openstack-nova - 2022-10-27 | |||
| 02:26:00 | opendevreview | melanie witt proposed openstack/nova master: Add mock to avoid loading guestfs in unit test https://review.opendev.org/c/openstack/nova/+/862769 | |
| 08:12:49 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: compute: enhance compute evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858383 | |
| 08:12:49 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 08:19:40 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova-specs master: spec: allowing target state for evacuate https://review.opendev.org/c/openstack/nova-specs/+/857838 | |
| 08:56:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce PCI pool filtering bug https://review.opendev.org/c/openstack/nova/+/855649 | |
| 08:56:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Strictly follow placement allocation during PCI claim https://review.opendev.org/c/openstack/nova/+/855650 | |
| 08:56:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Show candidate combinatorial explosion by dev number https://review.opendev.org/c/openstack/nova/+/855885 | |
| 09:01:52 | gibi | bauzas: a procedural question. We have the RFE bug filed for the port route metric support https://bugs.launchpad.net/nova/+bug/1994137 it is a whislist so that part is OK. Should I simply set it to Triaged or we want to discuss it in the next meeting before I set Triaged? | |
| 09:02:18 | bauzas | gibi: no, we agreed on it so Triage it directly ;) | |
| 09:02:25 | gibi | OK. | |
| 09:04:58 | auniyal_ | Hi bauzas, gibi, sean-k-mooney | |
| 09:05:23 | auniyal_ | can you please review theses - these are backports | |
| 09:05:24 | auniyal_ | https://review.opendev.org/c/openstack/nova/+/861856 | |
| 09:05:24 | auniyal_ | https://review.opendev.org/c/openstack/nova/+/861857 | |
| 09:05:24 | auniyal_ | ||
| 09:05:24 | auniyal_ | https://review.opendev.org/c/openstack/nova/+/861871 | |
| 09:05:26 | auniyal_ | https://review.opendev.org/c/openstack/nova/+/861872 | |
| 09:05:28 | auniyal_ | ||
| 09:05:30 | auniyal_ | https://review.opendev.org/c/openstack/nova/+/862604 | |
| 09:05:32 | auniyal_ | https://review.opendev.org/c/openstack/nova/+/862603 | |
| 09:05:34 | auniyal_ | ||
| 09:05:36 | auniyal_ | mostly clean cherry-pick | |
| 09:35:45 | amorin | hello nova team, when using ovs backend, nova plugs interface in ovs using vif_plug_ovs python module | |
| 09:36:02 | amorin | I was expecting to find the vif_plug_ovs option available in the config reference: | |
| 09:36:04 | amorin | https://docs.openstack.org/nova/latest/configuration/config.html | |
| 09:36:08 | amorin | but it's not | |
| 09:36:57 | amorin | https://github.com/openstack/os-vif/blob/master/vif_plug_ovs/ovs.py#L54 | |
| 09:37:57 | amorin | is that normal? | |
| 09:45:09 | sean-k-mooney | its a know issue | |
| 09:45:19 | sean-k-mooney | so technically os-vif is not part of nova | |
| 09:45:26 | sean-k-mooney | to have it incldued in novas docs | |