| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-09-01 | |||
| 13:42:15 | sean-k-mooney | no upstream links form ath either altough i did not include the downstreeam one so either but lets use your new one for now | |
| 13:44:58 | gibi | ohh cool, I have a very bad memory | |
| 13:45:54 | sean-k-mooney | this has been on my todo list to fix evenuatlly but it keep getting pushed off. artom started fixing some of the degecases and we have resolved other incidnetally. | |
| 13:46:15 | sean-k-mooney | for example live migration did not wrok because 1 the mac was not updated and 2 detach of PF was not implmented | |
| 13:46:29 | sean-k-mooney | gibi: you fixed detach when you added sriov attach/detach supprot | |
| 13:46:37 | gibi | yepp | |
| 13:46:46 | gibi | and I see artom fixed the unshelve pci_slot case | |
| 13:46:51 | sean-k-mooney | yes | |
| 13:47:18 | sean-k-mooney | so we have been trying to fix the more commens cases first when we get time too work on bugs | |
| 13:47:19 | gibi | I made TODO now on my side to look at the MAC update at some point | |
| 13:48:03 | sean-k-mooney | ack, so it a general problem for all pf move operation including evacuate | |
| 13:48:25 | sean-k-mooney | as you noted we do update the neutron port on vm boot intially its just never done after that | |
| 13:48:35 | gibi | yepp | |
| 13:48:40 | sean-k-mooney | so we need to add the same logic in all the places to update the neutron port mac | |
| 13:48:55 | sean-k-mooney | the problem is that that is not part of multipe port bidnign which makes it annoying | |
| 13:49:46 | gibi | ohh then live migration will be a special case | |
| 13:49:48 | kashyap | gibi: Is it something recent - Zuul requires uBlockOrigin to be disabled? | |
| 13:49:59 | gibi | kashyap: didn't noticed yet | |
| 13:49:59 | kashyap | (And Privacy Badger) | |
| 13:50:16 | sean-k-mooney | yep gibi so if you work on this proably leave live migration till last | |
| 13:50:24 | gibi | sean-k-mooney: ack, good idea | |
| 13:50:42 | sean-k-mooney | we could add a api block noting its not implemented in the interim with a workaround flag to allow operators to renable it | |
| 13:51:01 | sean-k-mooney | its one of the things we disucssed downstream i just never got around to pusshing a patch | |
| 13:51:28 | gibi | it is not utterly broken as we are not leaking resources or breaking the guest, so I would not block the operation | |
| 13:51:49 | sean-k-mooney | well actully it kind of is | |
| 13:52:02 | sean-k-mooney | the vm will migrate but it can cause other vms to fail to boot | |
| 13:52:09 | gibi | why? | |
| 13:52:20 | sean-k-mooney | when we release the PF on the source | |
| 13:52:29 | kashyap | When I opened a Zuul URL to debug, it was asking me to "please try with disabling ad blocker". /me tries. It's iether that, or it's just a stable URL | |
| 13:52:34 | kashyap | s/stable/stale/ | |
| 13:52:51 | sean-k-mooney | if another vm in the same network is shcudled to the source node then it will get a mac conflict in neutron when it trys to update the mac | |
| 13:53:15 | sean-k-mooney | gibi: also the vm that was migrate will not get an adress form dhcp proeprly | |
| 13:53:17 | gibi | Ohh so neutron will reject the AMC update | |
| 13:53:24 | gibi | :/ | |
| 13:53:28 | gibi | OK then it is broken | |
| 13:53:29 | sean-k-mooney | yep | |
| 13:53:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 13:53:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 13:53:52 | sean-k-mooney | so unless you are using static ips the vm that was migrated will be broken too | |
| 13:53:58 | gibi | kashyap: could paste a link you are strugling with? | |
| 13:54:09 | kashyap | gibi: https://zuul.opendev.org/t/openstack/build/a4b92ce40be841eaaf3e97f00c39b4d4 | |
| 13:54:17 | kashyap | gibi: It's from June, granted | |
| 13:54:21 | gibi | sean-k-mooney: ahh so the dhcp agent uses the port.mac to provide the ip | |
| 13:54:24 | kashyap | Probably it just got garbage-collected | |
| 13:54:37 | gibi | kashyap: it is already gc-d | |
| 13:54:42 | gibi | kashyap: you need to recheck the patch | |
| 13:54:45 | kashyap | Yep | |
| 13:54:53 | sean-k-mooney | gibi: yes and the port will have the mac of the new PF | |
| 13:55:00 | gibi | yep I got it know | |
| 13:55:11 | gibi | then Im OK to block the operation | |
| 13:55:16 | sean-k-mooney | gibi: so either the dhcp server will not respond or it will get the wrong ip if you have moved several vms | |
| 13:56:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: [nova-manage]support extended resource request https://review.opendev.org/c/openstack/nova/+/802060 | |
| 13:57:28 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reno for qos-minimum-guaranteed-packet-rate https://review.opendev.org/c/openstack/nova/+/805046 | |
| 14:10:11 | opendevreview | Ghanshyam proposed openstack/nova master: Convert features not supported error to HTTPBadRequest https://review.opendev.org/c/openstack/nova/+/806294 | |
| 14:12:28 | gmann | stephenfin: gibi ^^ resolved merge conflict, | |
| 14:13:26 | gibi | gmann: done | |
| 14:13:50 | gmann | gibi: thanks | |
| 14:50:28 | gibi | lyarwood: fyi I found a bug in the libvirt device detach series https://bugs.launchpad.net/nova/+bug/1942345 | |
| 15:04:08 | opendevreview | Merged openstack/nova stable/victoria: Reduce mocking in test_reject_open_redirect for compat https://review.opendev.org/c/openstack/nova/+/803093 | |
| 15:38:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Parse alias from domain hostdev https://review.opendev.org/c/openstack/nova/+/806943 | |
| 15:41:06 | gibi | I marked https://bugs.launchpad.net/nova/+bug/1942345 as xena-rc-potential as we regressed SRIOV PF detach in Xena | |
| 15:42:50 | sean-k-mooney | oh it broke ya that would be rc bug then | |
| 15:43:09 | sean-k-mooney | i might see if we can get this in whitebox ot try and test it eventually | |
| 15:43:21 | sean-k-mooney | unfortunetly we dont have a way to run against upstream with sriov currently | |
| 15:45:18 | gibi | yepp, it would be nice to get automatic coverage of SRIOV ops as I'm lazy to run SRIOV tests manually in my lab env frequently | |
| 15:45:43 | gibi | I discovered this by playing with the SRIOV migration MAC issue | |
| 16:02:56 | gibi | melwitt: if you are up, do you have any opinion about https://storyboard.openstack.org/#!/story/2009159 and the related discussion happened yesterday on the weekly meeting? | |
| 16:04:10 | sean-k-mooney | gibi: i know that there is limited coverage by the melonox ci but they have not added to that as we added new feature like sriov attaach/detach | |
| 16:06:03 | melwitt | gibi: oh sorry I had missed that. need to read it | |
| 16:08:48 | melwitt | gah this traceback formatting on storyboard -_- | |
| 16:11:16 | melwitt | gibi: on the surface it seems like we have to change it back and do something manual up the stack to handle the consumer type update rollback :/ not sure what else can be done about it | |
| 16:11:49 | opendevreview | Merged openstack/nova master: workarounds: Remove disable_native_luksv1 https://review.opendev.org/c/openstack/nova/+/805647 | |
| 16:12:05 | gibi | melwitt: yepp we discussed that on the meeting yesterday that we can do project and user creation outside of the main transaction and only move the consumer type and the allocation update into the last big transaction | |
| 16:12:41 | melwitt | gibi: oh, that's a better idea | |
| 16:12:45 | melwitt | I support that | |
| 16:13:03 | gibi | I had similar idea than yours first, then on the meeting I was convinced | |
| 16:13:38 | gibi | do you have time to work on a solution? | |
| 16:14:44 | melwitt | gibi: yeah I can, especially since I caused it -_- | |
| 16:14:48 | melwitt | I'll review your repro patch (thank you for that) and then stack a fix on top of it | |
| 16:15:01 | gibi | melwitt: thank you! let me know if I can help somehow | |
| 16:34:42 | opendevreview | Merged openstack/python-novaclient master: Add pre-commit https://review.opendev.org/c/openstack/python-novaclient/+/806915 | |
| 16:38:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Avoid excessive sleep in vmware unit test https://review.opendev.org/c/openstack/nova/+/806956 | |
| 17:06:09 | opendevreview | Merged openstack/nova master: Add some missing parameters in docs of os-cells https://review.opendev.org/c/openstack/nova/+/805972 | |
| 17:06:28 | opendevreview | Merged openstack/nova master: tests: Remove unnecessary warnings filter https://review.opendev.org/c/openstack/nova/+/800901 | |
| 17:06:48 | opendevreview | Merged openstack/nova master: Add test coverage for security group checking https://review.opendev.org/c/openstack/nova/+/786044 | |
| 17:36:24 | opendevreview | Merged openstack/nova master: Convert features not supported error to HTTPBadRequest https://review.opendev.org/c/openstack/nova/+/806294 | |
| 22:02:08 | opendevreview | Merged openstack/nova master: doc: allow_resize_to_same_host needs nova-api restarted https://review.opendev.org/c/openstack/nova/+/801275 | |
| #openstack-nova - 2021-09-02 | |||
| 02:50:18 | opendevreview | melanie witt proposed openstack/placement master: Add reproducer for Project creation race bug https://review.opendev.org/c/openstack/placement/+/806730 | |
| 02:50:26 | opendevreview | melanie witt proposed openstack/placement master: Narrow scope of set allocations database transaction https://review.opendev.org/c/openstack/placement/+/807014 | |
| 06:14:18 | opendevreview | Kevin Zhao proposed openstack/nova master: Support Cpu Compararion on Aarch64 Platform https://review.opendev.org/c/openstack/nova/+/763928 | |
| 06:21:56 | opendevreview | MaAoyu proposed openstack/nova master: swap multiattach volume, report an error VolumeNotFound https://review.opendev.org/c/openstack/nova/+/807025 | |
| 07:54:55 | gibi | o/ | |
| 07:55:17 | gibi | the Asia friendly nova meeting starts in 5 minutes here in the channel | |
| 08:00:38 | opendevmeet | The meeting name has been set to 'nova_extra' | |
| 08:00:38 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 08:00:38 | opendevmeet | Meeting started Thu Sep 2 08:00:38 2021 UTC and is due to finish in 60 minutes. The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 08:00:38 | gibi | #startmeeting nova_extra | |
| 08:00:46 | gibi | who we have here to day for the meeting? | |
| 08:05:16 | gibi | nobody? | |
| 08:05:28 | gibi | not even bauzas? ;) | |