| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-05-11 | |||
| 13:43:28 | ignaziocassano | https://review.opendev.org/c/openstack/neutron/+/640258/ | |
| 13:43:28 | ignaziocassano | https://review.opendev.org/c/openstack/neutron/+/640258/ | |
| 13:43:29 | ignaziocassano | https://review.opendev.org/c/openstack/neutron/+/766277/ | |
| 13:43:29 | ignaziocassano | https://review.opendev.org/c/openstack/neutron/+/753314/ | |
| 13:43:29 | ignaziocassano | https://review.opendev.org/c/openstack/neutron/+/766277/ | |
| 13:43:29 | ignaziocassano | https://review.opendev.org/c/openstack/neutron/+/753314/ | |
| 13:43:30 | ignaziocassano | https://review.opendev.org/c/openstack/nova/+/742180/12 | |
| 13:43:30 | ignaziocassano | https://review.opendev.org/c/openstack/nova/+/742180/12 | |
| 13:43:31 | ignaziocassano | https://review.opendev.org/c/openstack/nova/+/747454/4 | |
| 13:43:31 | ignaziocassano | https://review.opendev.org/c/openstack/nova/+/747454/4 | |
| 13:44:54 | sean-k-mooney | so for ovs with iptables we were already pluging the port into ovs in prelive migrate | |
| 13:44:54 | sean-k-mooney | so for ovs with iptables we were already pluging the port into ovs in prelive migrate | |
| 13:45:08 | sean-k-mooney | and by that i mean creating the ovs port in the ovs db | |
| 13:45:08 | sean-k-mooney | and by that i mean creating the ovs port in the ovs db | |
| 13:45:50 | sean-k-mooney | so with https://review.opendev.org/c/openstack/neutron/+/640258/ and https://review.opendev.org/c/openstack/neutron/+/753314/ | |
| 13:45:50 | sean-k-mooney | so with https://review.opendev.org/c/openstack/neutron/+/640258/ and https://review.opendev.org/c/openstack/neutron/+/753314/ | |
| 13:46:14 | sean-k-mooney | actully you dont need either of those pathces | |
| 13:46:14 | sean-k-mooney | actully you dont need either of those pathces | |
| 13:46:40 | sean-k-mooney | the l2 agent woudl have wired it up because with iptabels we are createing a linux bridge and a veth pair | |
| 13:46:40 | sean-k-mooney | the l2 agent woudl have wired it up because with iptabels we are createing a linux bridge and a veth pair | |
| 13:46:57 | ignaziocassano | I am facing same problems I got with stein | |
| 13:46:57 | ignaziocassano | I am facing same problems I got with stein | |
| 13:47:12 | sean-k-mooney | so the only neutorn patch you needed was https://review.opendev.org/c/openstack/neutron/+/766277/ to avoid the dhcp server race | |
| 13:47:12 | sean-k-mooney | so the only neutorn patch you needed was https://review.opendev.org/c/openstack/neutron/+/766277/ to avoid the dhcp server race | |
| 13:47:36 | ignaziocassano | On stein I solved with your workaround force legacy port binding | |
| 13:47:36 | ignaziocassano | On stein I solved with your workaround force legacy port binding | |
| 13:48:00 | sean-k-mooney | well that wont actuly solve it | |
| 13:48:00 | sean-k-mooney | well that wont actuly solve it | |
| 13:48:54 | sean-k-mooney | when you use legacy port bindign neutron has the port boudn to the souce node until much much later in the live migrtion | |
| 13:48:54 | sean-k-mooney | when you use legacy port bindign neutron has the port boudn to the souce node until much much later in the live migrtion | |
| 13:49:11 | sean-k-mooney | we still create the port on the destitaiton at the same time in both flows | |
| 13:49:11 | sean-k-mooney | we still create the port on the destitaiton at the same time in both flows | |
| 13:49:23 | sean-k-mooney | but we use stale info form nuetorn to do it | |
| 13:49:23 | sean-k-mooney | but we use stale info form nuetorn to do it | |
| 13:50:01 | sean-k-mooney | ignaziocassano: are you still forcing legacy mode in train | |
| 13:50:01 | sean-k-mooney | ignaziocassano: are you still forcing legacy mode in train | |
| 13:51:28 | sean-k-mooney | if you backport https://review.opendev.org/c/openstack/neutron/+/766277 it will only help if you are using multiple port bindings | |
| 13:51:28 | sean-k-mooney | if you backport https://review.opendev.org/c/openstack/neutron/+/766277 it will only help if you are using multiple port bindings | |
| 13:51:37 | ignaziocassano | Sean, I desabled the workaround for using legacy port binding. I think It is automatically disabled when I upgraded from stein to train because workaround.py is covered be new version. Right ? | |
| 13:51:37 | ignaziocassano | Sean, I desabled the workaround for using legacy port binding. I think It is automatically disabled when I upgraded from stein to train because workaround.py is covered be new version. Right ? | |
| 13:52:26 | sean-k-mooney | well you are expecting the new install to overright the exsiting files | |
| 13:52:26 | sean-k-mooney | well you are expecting the new install to overright the exsiting files | |
| 13:52:39 | sean-k-mooney | it should but that depens on how you isntalled | |
| 13:52:39 | sean-k-mooney | it should but that depens on how you isntalled | |
| 13:52:55 | ignaziocassano | yum update | |
| 13:52:55 | ignaziocassano | yum update | |
| 13:53:17 | sean-k-mooney | i think that will yes the rpm should unpack over the modifed one | |
| 13:53:17 | sean-k-mooney | i think that will yes the rpm should unpack over the modifed one | |
| 13:53:52 | sean-k-mooney | so on the neutron side you have backported https://review.opendev.org/c/openstack/neutron/+/766277 to train | |
| 13:53:52 | sean-k-mooney | so on the neutron side you have backported https://review.opendev.org/c/openstack/neutron/+/766277 to train | |
| 13:54:30 | sean-k-mooney | and enabled it by seeing [nova]/live_migration_events=true | |
| 13:54:30 | sean-k-mooney | and enabled it by seeing [nova]/live_migration_events=true | |
| 13:54:32 | sean-k-mooney | https://review.opendev.org/c/openstack/neutron/+/766277/10/neutron/conf/common.py#177 | |
| 13:54:32 | sean-k-mooney | https://review.opendev.org/c/openstack/neutron/+/766277/10/neutron/conf/common.py#177 | |
| 13:54:41 | ignaziocassano | So | |
| 13:54:41 | ignaziocassano | So | |
| 13:54:57 | ignaziocassano | let me to verify if I understodd well | |
| 13:54:57 | ignaziocassano | let me to verify if I understodd well | |
| 13:55:52 | ignaziocassano | 766277 patch is alrady in centos 7 packages | |
| 13:55:53 | ignaziocassano | 766277 patch is alrady in centos 7 packages | |
| 13:57:06 | ignaziocassano | and I must enable live_migration_events=true | |
| 13:57:06 | ignaziocassano | and I must enable live_migration_events=true | |
| 13:58:24 | sean-k-mooney | am if 766277 patch is alrady in centos 7 packages its because we have backported that downstream and it got pulled into rdo that may be the case but i dont know why ralonsoh woudl not have also done an upstream backport | |
| 13:58:24 | sean-k-mooney | am if 766277 patch is alrady in centos 7 packages its because we have backported that downstream and it got pulled into rdo that may be the case but i dont know why ralonsoh woudl not have also done an upstream backport | |
| 13:58:53 | sean-k-mooney | but yes enabling live_migration_events=true will prevent the race between the dhcp agent and the l2 agent | |
| 13:58:53 | sean-k-mooney | but yes enabling live_migration_events=true will prevent the race between the dhcp agent and the l2 agent | |
| 13:59:12 | sean-k-mooney | that race exists for legacy and mutltiple prot binding workflows | |
| 13:59:12 | sean-k-mooney | that race exists for legacy and mutltiple prot binding workflows | |
| 13:59:51 | ignaziocassano | So I must check if 766277 is in centos7 rpm packages ? | |
| 13:59:51 | ignaziocassano | So I must check if 766277 is in centos7 rpm packages ? | |
| 14:00:08 | ignaziocassano | Sorry for my english | |
| 14:00:08 | ignaziocassano | Sorry for my english | |
| 14:00:45 | sean-k-mooney | yes i am not sure if it will be | |
| 14:00:45 | sean-k-mooney | yes i am not sure if it will be | |
| 14:01:46 | ralonsoh | sean-k-mooney, what patch? | |
| 14:01:46 | ralonsoh | sean-k-mooney, what patch? | |
| 14:01:49 | ignaziocassano | OK. Let me to recap: 1) check if patch is includes. If not I patch code with patch command. 2) enable live_migration_events=true | |
| 14:01:49 | ignaziocassano | OK. Let me to recap: 1) check if patch is includes. If not I patch code with patch command. 2) enable live_migration_events=true | |
| 14:02:51 | ignaziocassano | the patch is https://review.opendev.org/c/openstack/neutron/+/766277/10/neutron/conf/common.py#177 | |
| 14:02:51 | ignaziocassano | the patch is https://review.opendev.org/c/openstack/neutron/+/766277/10/neutron/conf/common.py#177 | |
| 14:03:32 | ignaziocassano | or all 766277 patches ? | |
| 14:03:32 | ignaziocassano | or all 766277 patches ? | |
| 14:03:46 | ralonsoh | I didn't backported this patch because was not required and not supported in older versions | |
| 14:03:46 | ralonsoh | I didn't backported this patch because was not required and not supported in older versions | |
| 14:04:45 | sean-k-mooney | ralonsoh: well the dhcp race has always exsited so that technially affecte every neutron release | |
| 14:04:45 | sean-k-mooney | ralonsoh: well the dhcp race has always exsited so that technially affecte every neutron release | |
| 14:05:09 | ralonsoh | only for hybrid plugin | |
| 14:05:09 | ralonsoh | only for hybrid plugin | |
| 14:05:35 | ralonsoh | now nova has this code, I can backport it in U/S | |
| 14:05:35 | ralonsoh | now nova has this code, I can backport it in U/S | |
| 14:05:46 | ralonsoh | I'll do it today | |
| 14:05:46 | ralonsoh | I'll do it today | |
| 14:06:14 | sean-k-mooney | well hybrid plug was the default deployment mode until very very recently | |
| 14:06:14 | sean-k-mooney | well hybrid plug was the default deployment mode until very very recently | |
| 14:07:09 | gibi | sean-k-mooney: does the vdpa lifecycle operations handled as bugfix from API perspective or we bump a microversion for it? | |
| 14:07:09 | gibi | sean-k-mooney: does the vdpa lifecycle operations handled as bugfix from API perspective or we bump a microversion for it? | |
| 14:07:32 | sean-k-mooney | we do not bump the microversion no | |
| 14:07:32 | sean-k-mooney | we do not bump the microversion no | |
| 14:08:01 | sean-k-mooney | we are removing the 400 error and allowign the operations to proceed | |
| 14:08:01 | sean-k-mooney | we are removing the 400 error and allowign the operations to proceed | |