| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-28 | |||
| 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 | |
| 15:12:56 | sean-k-mooney | RA and DHCPv6 could provide an inital value. | |
| 15:12:57 | bauzas | slaweq: gibi: sean-k-mooney: in order to stop the check pipeline, I created a new revision ^ | |
| 15:13:08 | sean-k-mooney | gibi: yes the dhcp option is per subnet not per network | |
| 15:13:17 | sean-k-mooney | bauzas: ack | |
| 15:13:28 | sean-k-mooney | well you could have just removed the +w | |
| 15:14:48 | sean-k-mooney | gibi: actully | |
| 15:15:45 | sean-k-mooney | gibi: is the set of subnets that it is lopping over the subnets the port is attach to via the fixed ipes it has or just the one on the network | |
| 15:15:54 | sean-k-mooney | i woudl assume we have not prefiltered them | |
| 15:16:13 | gibi | I don't think we prefilter them | |
| 15:16:18 | gibi | but I haven't checked explicitly | |
| 15:16:35 | gibi | I assumed it is all the subnets of the network as it is under [networ][subnets] | |
| 15:16:41 | sean-k-mooney | so we need to do an intersection between the subnets of the fixed ips and the subnets of the network and then check only those | |
| 15:16:54 | sean-k-mooney | gibi: ya that is what i woudl assume too | |
| 15:18:47 | bauzas | sean-k-mooney: no, you can't just remove the +W if it was running to the gate | |
| 15:19:17 | bauzas | even in check pipeline | |
| 15:20:26 | bauzas | gibi: we prefilter only if we opt in for routed networks | |
| 15:20:40 | sean-k-mooney | bauzas: if it was in check you can if it was in gate then no | |
| 15:21:01 | bauzas | sean-k-mooney: anyway, this is done but I'm pretty sure of the other way | |
| 15:21:06 | bauzas | meh | |
| 15:21:23 | sean-k-mooney | so this is not related to routed networks | |
| 15:21:32 | bauzas | so, here, we have a list of subnets that's given from a network | |
| 15:21:38 | sean-k-mooney | you can have as may subnetes on a network as you liek to add more ips to the network | |