| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-08-03 | |||
| 15:52:52 | gibi | the decorator will catch all the problematic cases | |
| 15:53:16 | gibi | and we have two of those one that is fixed by the bugfix and the other was as simple one with rebuild | |
| 15:53:20 | sean-k-mooney[m] | ill review tomorrow but i assume you are just logging the issue for now and not saving | |
| 15:53:29 | gibi | wait | |
| 15:53:38 | gibi | so for the rollback live migration | |
| 15:53:49 | gibi | I was able to remove the mutated migration context and keep the save | |
| 15:54:05 | sean-k-mooney[m] | ok | |
| 15:54:15 | gibi | the vpmem cleanup that needed the mutation contex can be done based on the domain and not based on the instance.resources storage | |
| 15:54:30 | gibi | this solves our current bug I believe | |
| 15:54:51 | gibi | the decorator is added in a followup to catch new cases by raising at save() if it is called under a mutated context | |
| 15:55:10 | gibi | and that revealed a second place where we do this, at rebuild | |
| 15:55:17 | gibi | but that was fairly simple to fix | |
| 15:56:49 | sean-k-mooney[m] | oh ok so you have resolved the current incorrect uses | |
| 15:56:55 | gibi | yes | |
| 15:56:55 | sean-k-mooney[m] | and new incorrect uses will raise | |
| 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 | gibi | I don't think that it is super urgent | |
| 16:01:55 | melwitt | sean-k-mooney[m]: sure, I can look | |
| 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 (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 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:50 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 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:51 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 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:52 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 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:54 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add instance.power_on_error notification https://review.opendev.org/c/openstack/nova/+/852084 | |
| 07:44:54 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 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: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:58 | opendevreview | ribaudr proposed openstack/nova master: Change microversion to 2.93 https://review.opendev.org/c/openstack/nova/+/852088 | |
| 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: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 | https://docs.openstack.org/nova/latest/admin/live-migration-usage.html | |
| 07:46:54 | gibi | ahh | |
| 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 | gibi | bauzas_away: no worries, you are on vacation :) | |
| 09:23:38 | bauzas_away | yesterday, I thought about it and said "oh shit, forgot to review it" | |
| 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 | |