| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-28 | |||
| 17:57:40 | gmann | gibi: bauzas sean-k-mooney: hi, for detach issue in nova-lvm job. tempest patch need more time it seems. I was making server sshable by default which need more work in other tests. | |
| 17:58:55 | gmann | gibi: sean-k-mooney bauzas: I am moving my house this week so might need some more time on that. For now, I think we can go with skipping the failing tests temporarily in nova-lvm | |
| 17:59:06 | bauzas | ack | |
| 17:59:09 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864007 | |
| 17:59:15 | gibi | gmann: thanks for looking into it | |
| 18:00:54 | sean-k-mooney | so i think we shoudl be ok to skip them for a few weeks to unblock things | |
| 18:01:16 | gibi | yepp I agree | |
| 18:01:25 | sean-k-mooney | the cidner code paths really shoudl not depend on the images_backend that is in use at least not lvm | |
| 18:01:26 | gibi | sady we have no better option now | |
| 18:01:29 | gmann | but thing is those failing rescue test, all use ssh-able server. there is something else might be causing the failure | |
| 18:01:45 | sean-k-mooney | gmann: yep there likely is | |
| 18:01:55 | gmann | something to do on Jammy as lvm on focal working fine | |
| 18:02:11 | sean-k-mooney | possibly | |
| 18:02:28 | sean-k-mooney | it defiently happens more on jammy im not sure it would not happen on focal | |
| 18:02:43 | sean-k-mooney | it might be related to a default config delta between the two or the qemu/libvirt verions | |
| 18:02:59 | sean-k-mooney | but we dont currently have a lot of capsity to investigate | |
| 18:02:59 | gmann | may be | |
| 18:03:40 | gmann | yeah, do we have patch up to skip the tests? | |
| 18:04:05 | sean-k-mooney | i think bauzas has one but im not sure they updated for the volume tests | |
| 18:04:19 | gmann | k | |
| 18:04:51 | gmann | this one? https://review.opendev.org/c/openstack/nova/+/865658 | |
| 18:05:13 | sean-k-mooney | that was to solve the other issue so that is not needed | |
| 18:05:14 | gmann | this does not include nova-lvm failing tests | |
| 18:05:21 | gmann | k | |
| 18:05:23 | sean-k-mooney | we shoudl pull that form the gate | |
| 18:06:22 | sean-k-mooney | gmann: i do not see one form sylvain | |
| 18:06:38 | sean-k-mooney | so i guess there is non e right now for the lvm failures | |
| 18:11:45 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864055 | |
| 23:59:23 | opendevreview | Ghanshyam proposed openstack/nova master: Temporary skip some volume detach test in nova-lvm job https://review.opendev.org/c/openstack/nova/+/865922 | |
| #openstack-nova - 2022-11-29 | |||
| 02:38:23 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/863806 | |
| 02:38:24 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864055 | |
| 02:43:59 | gmann | bauzas: sean-k-mooney[m] gibi : this is for skipping failing tests https://review.opendev.org/c/openstack/nova/+/865922 | |
| 08:38:39 | opendevreview | Amit Uniyal proposed openstack/nova master: Refactoring live_migrate function name https://review.opendev.org/c/openstack/nova/+/865954 | |
| 08:41:46 | bauzas | gibi: sean-k-mooney: your help is appreciated https://review.opendev.org/c/openstack/nova/+/865922 | |
| 09:30:33 | opendevreview | Arnaud Morin proposed openstack/nova master: Unbind port when offloading a shelved instance https://review.opendev.org/c/openstack/nova/+/853682 | |
| 09:34:08 | gibi | bauzas: on it | |
| 09:34:56 | gibi | gmann: thank you | |
| 09:45:27 | bauzas | gibi: thanks | |
| 09:45:38 | bauzas | and I updated the ML thread to notify our gerrit users | |
| 09:45:50 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds check if resized to swap zero https://review.opendev.org/c/openstack/nova/+/857339 | |
| 09:45:50 | gibi | cool | |
| 11:08:30 | opendevreview | Merged openstack/nova master: Temporary skip some volume detach test in nova-lvm job https://review.opendev.org/c/openstack/nova/+/865922 | |
| 11:08:37 | opendevreview | Merged openstack/nova stable/wallaby: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/863900 | |
| 11:13:34 | opendevreview | Manuel Bentele proposed openstack/nova-specs master: Add configuration options to set SPICE compression settings https://review.opendev.org/c/openstack/nova-specs/+/849488 | |
| 11:13:44 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/864168 | |
| 11:13:45 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/864169 | |
| 11:40:23 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/864168 | |
| 11:40:24 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/864169 | |
| 12:01:34 | opendevreview | Merged openstack/nova stable/wallaby: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/863901 | |
| 13:45:30 | sahid | morning dansmith, how do you feel regarding spec https://review.opendev.org/c/openstack/nova-specs/+/857838 I also have proposed the implementation | |
| 13:46:04 | sahid | if you have a moment i would be glad to make progress on it, eel free to let me know if there is any point that you need details thanks a lot | |
| 13:47:00 | slaweq | gibi bauzas and other nova cores: hi, please check https://review.opendev.org/c/openstack/nova/+/855664 when You will have few minutes, thx in advance | |
| 13:48:12 | pengo_ | https://www.irccloud.com/pastebin/kIDKAF54/ | |
| 13:48:12 | pengo_ | Hello I am using wallaby release openstack and having issues with cinder volumes attachments as once I try to delete, resize or unshelve the shelved vms the volume_attachement entries do not get deleted in cinder db and therefore the above mentioned operations fail every time. I have to delete these volume_attachement entries manually to make it work. I could only gather logs from nova-compute | |
| 13:49:54 | pengo_ | If I would like to unshelve the instance it wont work as it has a duplicate entry in cinder db for the attachment. So i have to delete it manually from db or via cli. This is the only choice I have if I would like to unshelve vm. But this is not a good approach for production env to delete duplicate volume attachments entries every time for every vm. Is there any way to fix this issue ? | |
| 14:11:22 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds regression functional test for 1980720 https://review.opendev.org/c/openstack/nova/+/861357 | |
| 14:11:23 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds check for VM snapshot fail while quiesce https://review.opendev.org/c/openstack/nova/+/852171 | |
| 14:16:04 | bauzas | slaweq: hah, I remember the context, we discussed this at the PTG right? | |
| 14:22:33 | pengo_ | lyes | |
| 14:35:22 | slaweq_ | bauzas: yes, we talked about it at the PTG | |
| 14:35:40 | slaweq_ | and we agreed that we can go with this approach | |
| 14:36:40 | sean-k-mooney | is this related to the mtu advertisement ? | |
| 14:36:46 | sean-k-mooney | or soemthing else | |
| 14:37:00 | sean-k-mooney | ah yes https://review.opendev.org/c/openstack/nova/+/855664 | |
| 14:37:14 | slaweq_ | sean-k-mooney: yes | |
| 14:37:29 | sean-k-mooney | ya so we agreeed that if dhcp is avialable on the subnet we can omit the mtu form the metadata | |
| 14:38:12 | sean-k-mooney | this will allow the mtu to be reduced but not increased and the vms will clamp the mtu the next time it renews its dhcp lease | |
| 14:38:19 | sean-k-mooney | will still not perfect it will help | |
| 14:38:57 | sean-k-mooney | i can take a look now before i forget about it again | |
| 14:41:23 | sean-k-mooney | slaweq_: quick question | |
| 14:42:00 | sean-k-mooney | slaweq_: is the mtu on the netowrk or on the subnet in neutron. its generally an aspect of the netwrok in a real deployment just wondering how its modeled in neutron | |
| 14:42:47 | sean-k-mooney | neutron does not supprot having diffenert mtu per network segment correct when using routed networks | |
| 14:47:44 | sean-k-mooney | i have asked that in the review https://review.opendev.org/c/openstack/nova/+/855664/3/nova/virt/netutils.py#b266 | |
| 14:48:12 | slaweq_ | sean-k-mooney: mtu is per network for sure | |
| 14:48:26 | slaweq_ | I'm not sure about segments in routed networks | |
| 14:49:29 | sean-k-mooney | "The net-mtu extension allows plug-ins to expose the MTU that is guaranteed to pass through the data path of the segments in the network." | |
| 14:49:51 | sean-k-mooney | that makes it sound like the network mtu shoudl be the maxium mtu that woudl eb supported on all segments | |
| 14:50:20 | sean-k-mooney | ok we are good | |
| 14:50:23 | sean-k-mooney | https://docs.openstack.org/api-ref/network/v2/index.html?expanded=show-segment-details-detail#show-segment-details | |
| 14:50:28 | sean-k-mooney | the segment does not have an mtu field | |
| 14:50:38 | slaweq_ | sean-k-mooney++ | |
| 14:51:10 | sean-k-mooney | nor does the subnet so ya defeintly per network ill upgade to +2 so | |
| 14:51:10 | bauzas | was trampled into internal problems, lemme look at the MTU patch | |
| 14:53:51 | slaweq_ | sean-k-mooney: thx a lot | |
| 15:02:10 | bauzas | slaweq: sent to the gate now the gate is back :) | |
| 15:02:46 | gibi | slaweq: I have a question https://review.opendev.org/c/openstack/nova/+/855664/3/nova/virt/netutils.py#274 | |
| 15:06:15 | bauzas | gibi: I had the same concern, but eventually I said yes because it's an operator question | |
| 15:07:13 | bauzas | gibi: here, that means that we won't provide the MTU in the metadata service if the subnet from the instance is using a dhcp server | |
| 15:08:18 | gibi | "subnet from the instance is using a dhcp server" <- but if the network has two subnets one with dhcp and one without dhcp then we need to check the actualy subnet the port uses, not every subnet in the network | |
| 15:08:37 | gibi | or do I miss something | |
| 15:08:38 | gibi | ? | |
| 15:09:05 | gibi | I'm OK to not set MTU if the subnet the port uses has DHCP. But the patch does not implement that | |
| 15:09:22 | gibi | that patch does not set MTU if _any_ of the subnets of the network has DHCP | |
| 15:09:38 | gibi | no just the one the port uses | |
| 15:10:30 | frickler | I need to double-check but I think for v6 the MTU is signaled by RAs, not dhcp? | |
| 15:11:10 | sean-k-mooney | gibi: good catch gibi | |
| 15:11:13 | gibi | (I assume that as the dhcp_server is defined per subnet it can be differently configured per subnet of the same network) | |
| 15:11:47 | sean-k-mooney | frickler: for ipv6 mtu is discoverd via the neibour discovery protocol | |
| 15:12:08 | sean-k-mooney | and it shoudl be automaticaly negociated regardelss of using RA or DHCP6 | |
| 15:12:26 | opendevreview | Sylvain Bauza proposed openstack/nova master: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/855664 | |