| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-08-03 | |||
| 15:56:57 | gibi | yes | |
| 15:57:01 | sean-k-mooney[m] | so we wont merge it | |
| 15:57:03 | sean-k-mooney[m] | ok | |
| 15:57:19 | sean-k-mooney[m] | that a liit more context then im prepared to digest to review today | |
| 15:57:27 | sean-k-mooney[m] | but ill look back on this tomorrow | |
| 15:58:07 | gibi | ack | |
| 15:58:37 | gibi | artom and dansmith are also on it to keep me honest :) | |
| 15:58:49 | dansmith | yeah will try to look a bit later | |
| 15:59:46 | sean-k-mooney[m] | ok if dansmith and melwitt are able to review it i would prefer not to add another context switch to my current list | |
| 16:00:01 | sean-k-mooney[m] | but if this is urgent i can | |
| 16:01:55 | melwitt | sean-k-mooney[m]: sure, I can look | |
| 16:01:55 | gibi | I don't think that it is super urgent | |
| 16:03:14 | gibi | thank you all | |
| 16:37:54 | frickler | dansmith: melwitt: could you also have a look at https://review.opendev.org/c/openstack/nova/+/851909 pls? would be good to be able to have wider testing of latest sdk, currently this is blocking getting it into u-c | |
| 16:39:38 | dansmith | hmm, I guess that's needed because of when we talk to placement | |
| 16:42:15 | dansmith | frickler: +W | |
| 16:50:50 | frickler | thx | |
| 17:26:45 | opendevreview | Jan Hartkopf proposed openstack/nova master: add support for updating server's user_data https://review.opendev.org/c/openstack/nova/+/816157 | |
| 17:44:34 | opendevreview | Merged openstack/placement master: Func test for os-traits and os-resource-classes lib sync https://review.opendev.org/c/openstack/placement/+/851966 | |
| 17:44:38 | opendevreview | Merged openstack/nova master: Fix mocking SafeConnectedTestCase https://review.opendev.org/c/openstack/nova/+/851909 | |
| 17:49:01 | stephenfin | gibi: Is that PCI series ready for review? | |
| 18:59:30 | spatel | does nova support sound adaptor ? | |
| 19:19:55 | wolsen[m] | gibi: bauzas: sorry I've been slow to respond on that patch set (the mdev devices libvirt 7.x one - https://review.opendev.org/c/openstack/nova/+/838976) - I 'm reading the backscroll now and see you commented that you need something for me. I'm jugging various things at the moment but will review feedback shortly | |
| 19:21:21 | wolsen[m] | looks like an update around the FUP patch that bauzas had posted | |
| #openstack-nova - 2022-08-04 | |||
| 03:12:23 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/851337 | |
| 06:56:21 | opendevreview | Brett Milford proposed openstack/nova master: Handle "no RAM info was set" migration case https://review.opendev.org/c/openstack/nova/+/852002 | |
| 07:15:14 | gibi | stephenfin: re PCI series: I still had issues after the unittest.mock and double mocking change. But the content is ready up until https://review.opendev.org/c/openstack/nova/+/850546 . The rest top of that has TODOs in the commit message | |
| 07:15:57 | gibi | wolsen[m]: yes, I think only the FUP patch needs update the rest is OK to land | |
| 07:16:50 | gibi | stephenfin: I probably need to change the first patch in the series https://review.opendev.org/c/openstack/nova/+/844627 due to the double mocking stuff | |
| 07:26:39 | ignazio | Hello, I have very slow performances in live migration when instances using memory heavily (for example java application servers). I used converce mode in nova.conf and it does not seem to solve. If I migrate an instance with 8GB ram it takes more than 20 minutes. If I migrate again it takes few seconds. Do you think it is related to memory access ? | |
| 07:29:50 | ignazio | Any help, please ? | |
| 07:44:48 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (db) https://review.opendev.org/c/openstack/nova/+/831193 | |
| 07:44:49 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | |
| 07:44:49 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 07:44:50 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (drivers) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 07:44:50 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 07:44:51 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Bump compute version and check shares support https://review.opendev.org/c/openstack/nova/+/850499 | |
| 07:44:51 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 07:44:52 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 07:44:52 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 07:44:54 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 07:44:54 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add instance.power_on_error notification https://review.opendev.org/c/openstack/nova/+/852084 | |
| 07:44:56 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/852085 | |
| 07:44:56 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 07:44:58 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 07:44:58 | opendevreview | ribaudr proposed openstack/nova master: Change microversion to 2.93 https://review.opendev.org/c/openstack/nova/+/852088 | |
| 07:46:04 | gibi | ignazio: if the workload in the guest continuously dirtying the memory on the source host then libvirt continously needs to retransmitt the dirtied memory pages to the dest. | |
| 07:46:33 | gibi | ignazio: you can try post-copy live migration | |
| 07:46:44 | gibi | https://docs.openstack.org/nova/pike/admin/live-migration-usage.html | |
| 07:46:54 | gibi | ahh | |
| 07:46:54 | gibi | https://docs.openstack.org/nova/latest/admin/live-migration-usage.html | |
| 07:47:35 | ignazio | gibi: I have already tries post-copy instead converge with the same results | |
| 07:47:41 | gibi | with post-copy the guest will be activated on the dest earlier in the process and libvirt will copy the memory from the inactive guest from the source | |
| 07:48:09 | ignazio | Do you think hugepage can help ? | |
| 07:48:37 | ignazio | I read page size flavor deafaul is 4k. Is true ? | |
| 07:49:05 | gibi | have you checked that the network used by libvirt for transferring the memory data has enough bandwidth? | |
| 07:49:28 | gibi | I'm not sure if hugepage will make a difference. yes the default small page on x86 is 4k | |
| 07:51:04 | ignazio | So, if I migrate an istance it spend a lot of time searching memory to migrate. When I migrate back it takes few seconds . | |
| 07:52:25 | ignazio | Probably after the first migration, the destination host reorganize the memory. I do not know very much the tlb mechanism | |
| 08:10:33 | gibi | moving back is an idependent process, there is no way to remember the past memeory content | |
| 08:23:01 | ignazio | <giby> the instance is using heavily the ram also on the first destination host after the first live migration. What I mean, because if I migrate again it takes few seconds ? | |
| 08:31:51 | opendevreview | Merged openstack/placement master: Clarify trait filtering in the API doc https://review.opendev.org/c/openstack/placement/+/825501 | |
| 08:44:42 | ignazio | <gibi> I have 10Gbs bandwiths available | |
| 08:44:54 | gibi | I'm out of ideas :/ sorry | |
| 08:45:35 | ignazio | <gibi> thanks | |
| 09:22:52 | gibi | bauzas_away: o/ | |
| 09:23:20 | bauzas_away | gibi: apologies about the PCI series, totally got out of my scope btw. | |
| 09:23:38 | bauzas_away | yesterday, I thought about it and said "oh shit, forgot to review it" | |
| 09:23:38 | gibi | bauzas_away: no worries, you are on vacation :) | |
| 09:23:57 | bauzas_away | gibi: I'll do a couple of series this morning | |
| 09:24:38 | bauzas_away | my vacation trip is only next week, so this week this is just staying with the kids at home and preparing for the journey | |
| 09:50:18 | bauzas_away | gibi: thanks for your comment yesterdaty on https://review.opendev.org/c/openstack/nova/+/851924 | |
| 09:50:36 | bauzas_away | replying to it made me realize I was again writing a wrong logic | |
| 09:51:07 | bauzas_away | the FUP can't fix the recreate issue, it should just remove the need for mdev_name2uuid | |
| 10:09:27 | opendevreview | Nobuhiro MIKI proposed openstack/nova master: pci: fix checking for physical function https://review.opendev.org/c/openstack/nova/+/852135 | |
| 10:35:23 | sean-k-mooney[m] | gibi: can i assume you agree we shoudl just proceed with https://review.opendev.org/c/openstack/nova/+/845775/4 instead of ^ | |
| 10:58:25 | sean-k-mooney[m] | gibi can you rereivew https://review.opendev.org/c/openstack/nova/+/848886/18 you were previously +2 but i asked for the release note to be changed slightly | |
| 11:28:04 | gibi | sean-k-mooney[m]: I'm not sure that the two change are equivalent as after my patch we still uses is_physical_function and that queries sriov_totalvfs | |
| 11:29:16 | gibi | sean-k-mooney[m]: I'm +2+A on the evac change | |
| 11:30:23 | sean-k-mooney[m] | ack | |
| 11:30:43 | sean-k-mooney[m] | i dont think we shoudl really be basiing this off sriov_totalvfs | |
| 11:31:28 | sean-k-mooney[m] | that can be 0 if you disable sriov VFs in the bios | |
| 11:31:41 | gibi | I agree that this is shakey | |
| 11:31:47 | gibi | shaky | |
| 11:32:07 | sean-k-mooney[m] | we dont really handel type-pci well with neutron | |
| 11:32:52 | sean-k-mooney[m] | since we dont use the mac update logic for direct phyiscal when you are using a type-pci device via vnic_type=direct | |
| 11:33:52 | sean-k-mooney[m] | so what i dont want to see is use report a nic as type-pci becuase you change the bios setting to disable sriov and then prevent the port form working with direct-physical | |
| 11:34:27 | sean-k-mooney[m] | i guess we can see if there is a better way to do the detection | |
| 11:35:09 | gibi | would be the proposed phys_port_name based detection better? | |
| 11:35:36 | sean-k-mooney[m] | i would prefer if we honetly did not use these fucntions for this if we can avoid it | |
| 11:35:46 | sean-k-mooney[m] | and use the objects form the virt driver instead | |
| 11:36:15 | sean-k-mooney[m] | i have not looked recently but why are we determining if its a physical fucntion in that code path currently | |
| 11:36:52 | sean-k-mooney[m] | this does not influcne if its reported as type-pci ectra today | |
| 11:36:58 | sean-k-mooney[m] | that is done seperatly | |
| 11:39:23 | gibi | I agree that we should not use the sysfs and user the libvirt driver instead | |
| 11:39:47 | gibi | right now I cannot precisely answer why we look up the type during the whitelist parsing | |
| 11:39:54 | sean-k-mooney[m] | well rather the objects returned by the virt driver genericlly | |
| 11:40:06 | gibi | but I have many theorethical issues with that code | |
| 11:40:36 | gibi | +1 on an abstraction over the libvirt virt driver | |
| 11:40:38 | sean-k-mooney[m] | its proably used to supprot the feature where if we whitelist the pf addres with vf product id | |