| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-05-11 | |||
| 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 | |
| 14:17:04 | ralonsoh | yes | |
| 14:18:00 | ignaziocassano | ralonsoh Ok, so I wait your push. OK ? Many thanks | |
| 14:18:00 | ignaziocassano | ralonsoh Ok, so I wait your push. OK ? Many thanks | |
| 14:19:02 | ignaziocassano | sean-k-mooney Many thks | |
| 14:19:03 | ignaziocassano | sean-k-mooney Many thks | |
| 14:21:02 | ignaziocassano | ralonsoh: when I can try yum update to see new packages ? | |
| 14:21:02 | ignaziocassano | ralonsoh: when I can try yum update to see new packages ? | |
| 14:26:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add image metadata property reference guide https://review.opendev.org/c/openstack/nova/+/756867 | |
| 14:26:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add image metadata property reference guide https://review.opendev.org/c/openstack/nova/+/756867 | |
| 14:26:17 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add 'nova:image-meta:' directive, role https://review.opendev.org/c/openstack/nova/+/781530 | |
| 14:26:17 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add cross-references to image metadata properties https://review.opendev.org/c/openstack/nova/+/781531 | |
| 14:26:17 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add 'nova:image-meta:' directive, role https://review.opendev.org/c/openstack/nova/+/781530 | |
| 14:26:18 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add cross-references to image metadata properties https://review.opendev.org/c/openstack/nova/+/781531 | |
| 14:30:33 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Remove dead code https://review.opendev.org/c/openstack/nova/+/786291 | |
| 14:30:33 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Remove 'nova.db.sqlalchemy.utils' https://review.opendev.org/c/openstack/nova/+/786292 | |
| 14:30:33 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Remove dead code https://review.opendev.org/c/openstack/nova/+/786291 | |
| 14:30:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Remove unused DB methods https://review.opendev.org/c/openstack/nova/+/786293 | |
| 14:30:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Use module-level imports for sqlalchemy https://review.opendev.org/c/openstack/nova/+/786295 | |
| 14:30:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Remove 'nova.db.sqlalchemy.utils' https://review.opendev.org/c/openstack/nova/+/786292 | |
| 14:30:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Remove unused DB methods https://review.opendev.org/c/openstack/nova/+/786293 | |
| 14:30:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Fold in indexes https://review.opendev.org/c/openstack/nova/+/786296 | |
| 14:30:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Fold in ForeignKey constraints https://review.opendev.org/c/openstack/nova/+/786297 | |
| 14:30:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Use module-level imports for sqlalchemy https://review.opendev.org/c/openstack/nova/+/786295 | |
| 14:30:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Fold in indexes https://review.opendev.org/c/openstack/nova/+/786296 | |