| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-05-11 | |||
| 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 | |
| 14:08:04 | gibi | then I think a specless bp is OK | |
| 14:08:04 | gibi | then I think a specless bp is OK | |
| 14:08:27 | gibi | except if you need RPC or DB changes | |
| 14:08:27 | gibi | except if you need RPC or DB changes | |
| 14:08:32 | ignaziocassano | Ralonsoh and Sean, so what I must to do for solving the issue on centos 7 train ? | |
| 14:08:32 | ignaziocassano | Ralonsoh and Sean, so what I must to do for solving the issue on centos 7 train ? | |
| 14:08:45 | sean-k-mooney | ack ya if i need any rpc/object/db change ill file a spec | |
| 14:08:45 | sean-k-mooney | ack ya if i need any rpc/object/db change ill file a spec | |
| 14:09:48 | ralonsoh | ignaziocassano, wait until neutron and nova code is merged in stable/train | |
| 14:09:48 | ralonsoh | ignaziocassano, wait until neutron and nova code is merged in stable/train | |
| 14:10:40 | sean-k-mooney | ignaziocassano: we need to backport the pathces. honestly though im not planning to spend to much more time workign on this. ill start some of the backport but i started trying to get this fixed in 2018 so i dont really have the energy to keep working on this after 3 years | |
| 14:10:40 | sean-k-mooney | ignaziocassano: we need to backport the pathces. honestly though im not planning to spend to much more time workign on this. ill start some of the backport but i started trying to get this fixed in 2018 so i dont really have the energy to keep working on this after 3 years | |
| 14:14:02 | ignaziocassano | ralonsoh Can I tary to patch the code with patch command or by hand meanwhile the code is merged ? | |
| 14:14:02 | ignaziocassano | ralonsoh Can I tary to patch the code with patch command or by hand meanwhile the code is merged ? | |
| 14:14:20 | ignaziocassano | sorry : tary = try | |
| 14:14:20 | ignaziocassano | sorry : tary = try | |
| 14:14:36 | ralonsoh | for sure | |
| 14:14:36 | ralonsoh | for sure | |
| 14:14:45 | ralonsoh | I'm pushing now neutron stable patches | |
| 14:14:45 | ralonsoh | I'm pushing now neutron stable patches | |
| 14:14:53 | ignaziocassano | ok | |
| 14:14:53 | ignaziocassano | ok | |
| 14:15:32 | ignaziocassano | then I must enable under [nova] section in neutron.conf live_migration_events=True ? | |
| 14:15:32 | ignaziocassano | then I must enable under [nova] section in neutron.conf live_migration_events=True ? | |
| 14:16:02 | gibi | sean-k-mooney: ack | |
| 14:16:57 | ignaziocassano | ralonsoh then I must enable under [nova] section in neutron.conf live_migration_events=True ? | |
| 14:16:57 | ignaziocassano | ralonsoh then I must enable under [nova] section in neutron.conf live_migration_events=True ? | |
| 14:17:04 | ralonsoh | yes | |