[OVN][Trunk] Sometimes subport doesn't reach status ACTIVE after live-migration

Bug #2027605 reported by Eduardo Olivares
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Confirmed
High
Rodolfo Alonso

Bug Description

Bugzilla reference: https://bugzilla.redhat.com/show_bug.cgi?id=2223997

this is similar to https://bugs.launchpad.net/neutron/+bug/2024160 and actually it is reproduced with the same tempest test, test_live_migration_with_trunk, but there are a few important differencies.

In LP2024160, the subport remained in status DOWN when the VM was created (before any migration!). And it happened always (the test always failed).

With this new bug, both subport and parent port successfully reach status=ACTIVE when the VM is created. Then, the VM is live-migrated to another compute. The test checks the VM status successfully changes to ACTIVE.
Finally, the test fails waiting for the subport status to change to ACTIVE: it never happens, it remains in status DOWN.
This failure doesn't happen always, so it seems it is due to a race condition.

Tempest logs when it fails (this job was retriggered/rechecked and the test passed):
https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_8ac/887220/4/check/nova-live-migration/8ace3a8/testr_results.html

How often does this fail?
https://zuul.opendev.org/t/openstack/builds?job_name=nova-live-migration&branch=master&skip=0
24 jobs run on 11th and 12th July
7 of them failed - all these failures are due to this bug
i.e. ~30%

neutron server logs:
https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_8ac/887220/4/check/nova-live-migration/8ace3a8/controller/logs/screen-q-svc.txt

subport: e40925a3-4e71-4fa4-80ea-cc3a464101d6
parent port: 51e121cc-4e64-411e-b16a-e605cf967332
live-migration src host: np0034645395
live-migration dst host: np0034645398

It seems that due to a race condition, the following operations are processed in the wrong order.
The first one sets the parent port status to ACTIVE and changes revision number from 9 to 10:
Jul 11 15:45:48.620648 np0034645395 neutron-server[78430]: INFO neutron.tests.unit.plugins.ml2.drivers.mechanism_logger [None req-de6d27ae-5f07-41d8-90cb-e85d1228c53f None None] update_port_postcommit called with port settings {'id': '51e121cc-4e64-411e-b16a-e605cf967332', 'name': 'tempest-parent-2113033899', 'network_id': '6d3ce4b9-e66d-4137-a356-75e402377eb7', 'tenant_id': '4e4ea23ed96240ea8108fd937f2bd12b', 'mac_address': 'fa:16:3e:e6:51:04', 'admin_state_up': True, 'status': 'ACTIVE', 'device_id': 'cb3569cf-e7e0-4b01-8d56-aaf9fbcd41a8', 'device_owner': 'compute:nova', 'standard_attr_id': 173, 'fixed_ips': [{'subnet_id': '5ed321df-19d5-4c9b-8cf2-f27f0993fe12', 'ip_address': '10.1.0.5'}], 'allowed_address_pairs': [], 'extra_dhcp_opts': [], 'security_groups': ['ad92c344-fc82-4f88-bdc7-1d7845432a2f'], 'description': '', 'binding:vnic_type': 'normal', 'binding:profile': {'migrating_to': 'np0034645398'}, 'binding:host_id': 'np0034645395', 'binding:vif_type': 'ovs', 'binding:vif_details': {'port_filter': True, 'connectivity': 'l2', 'bound_drivers': {'0': 'ovn'}}, 'port_security_enabled': True, 'trunk_details': {'trunk_id': '833832a8-c2bf-4ca9-a25a-9462f63b8926', 'sub_ports': [{'segmentation_id': 42, 'segmentation_type': 'vlan', 'port_id': 'e40925a3-4e71-4fa4-80ea-cc3a464101d6', 'mac_address': 'fa:16:3e:e5:55:14'}]}, 'tags': [], 'created_at': '2023-07-11T15:45:19Z', 'updated_at': '2023-07-11T15:45:48Z', 'revision_number': 10, 'project_id': '4e4ea23ed96240ea8108fd937f2bd12b'} (original settings {'id': '51e121cc-4e64-411e-b16a-e605cf967332', 'name': 'tempest-parent-2113033899', 'network_id': '6d3ce4b9-e66d-4137-a356-75e402377eb7', 'tenant_id': '4e4ea23ed96240ea8108fd937f2bd12b', 'mac_address': 'fa:16:3e:e6:51:04', 'admin_state_up': True, 'status': 'DOWN', 'device_id': 'cb3569cf-e7e0-4b01-8d56-aaf9fbcd41a8', 'device_owner': 'compute:nova', 'standard_attr_id': 173, 'fixed_ips': [{'subnet_id': '5ed321df-19d5-4c9b-8cf2-f27f0993fe12', 'ip_address': '10.1.0.5'}], 'allowed_address_pairs': [], 'extra_dhcp_opts': [], 'security_groups': ['ad92c344-fc82-4f88-bdc7-1d7845432a2f'], 'description': '', 'binding:vnic_type': 'normal', 'binding:profile': {'migrating_to': 'np0034645398'}, 'binding:host_id': 'np0034645395', 'binding:vif_type': 'ovs', 'binding:vif_details': {'port_filter': True, 'connectivity': 'l2', 'bound_drivers': {'0': 'ovn'}}, 'port_security_enabled': True, 'trunk_details': {'trunk_id': '833832a8-c2bf-4ca9-a25a-9462f63b8926', 'sub_ports': [{'segmentation_id': 42, 'segmentation_type': 'vlan', 'port_id': 'e40925a3-4e71-4fa4-80ea-cc3a464101d6', 'mac_address': 'fa:16:3e:e5:55:14'}]}, 'tags': [], 'created_at': '2023-07-11T15:45:19Z', 'updated_at': '2023-07-11T15:45:48Z', 'revision_number': 9, 'project_id': '4e4ea23ed96240ea8108fd937f2bd12b'}) host np0034645395 (original host np0034645395) vif type ovs (original vif type ovs) vif details {'port_filter': True, 'connectivity': 'l2'} (original vif details {'port_filter': True, 'connectivity': 'l2'}) binding levels [{'bound_driver': 'ovn', 'bound_segment': {'id': '21392775-0d30-44c3-9f06-aa8348d69be8', 'network_type': 'geneve', 'physical_network': None, 'segmentation_id': 10205, 'network_id': '6d3ce4b9-e66d-4137-a356-75e402377eb7'}}] (original binding levels [{'bound_driver': 'ovn', 'bound_segment': {'id': '21392775-0d30-44c3-9f06-aa8348d69be8', 'network_type': 'geneve', 'physical_network': None, 'segmentation_id': 10205, 'network_id': '6d3ce4b9-e66d-4137-a356-75e402377eb7'}}]) on network {'id': '6d3ce4b9-e66d-4137-a356-75e402377eb7', 'name': 'tempest-LiveAutoBlockMigrationV225Test-1684440653-network', 'tenant_id': '4e4ea23ed96240ea8108fd937f2bd12b', 'admin_state_up': True, 'mtu': 1372, 'status': 'ACTIVE', 'subnets': ['5ed321df-19d5-4c9b-8cf2-f27f0993fe12'], 'standard_attr_id': 55, 'shared': False, 'availability_zone_hints': [], 'availability_zones': [], 'ipv4_address_scope': None, 'ipv6_address_scope': None, 'router:external': False, 'vlan_transparent': None, 'description': '', 'port_security_enabled': True, 'tags': [], 'created_at': '2023-07-11T15:41:08Z', 'updated_at': '2023-07-11T15:41:10Z', 'revision_number': 2, 'project_id': '4e4ea23ed96240ea8108fd937f2bd12b', 'provider:network_type': 'geneve', 'provider:physical_network': None, 'provider:segmentation_id': 10205} with segments to bind None
Jul 11 15:45:48.621081 np0034645395 neutron-server[78430]: DEBUG neutron.tests.unit.plugins.ml2.drivers.mechanism_logger [None req-de6d27ae-5f07-41d8-90cb-e85d1228c53f None None] update_port_postcommit:
Jul 11 15:45:48.621081 np0034645395 neutron-server[78430]: key[status], DOWN -> ACTIVE
Jul 11 15:45:48.621081 np0034645395 neutron-server[78430]: key[revision_number], 9 -> 10

The second operation wrongly sets the parent port status to DOWN again, and changes revision_number from 8 to 9, so apparently it should have been processed before:
Jul 11 15:45:49.283211 np0034645395 neutron-server[78430]: INFO neutron.tests.unit.plugins.ml2.drivers.mechanism_logger [None req-de6d27ae-5f07-41d8-90cb-e85d1228c53f None None] update_port_postcommit called with port settings {'id': '51e121cc-4e64-411e-b16a-e605cf967332', 'name': 'tempest-parent-2113033899', 'network_id': '6d3ce4b9-e66d-4137-a356-75e402377eb7', 'tenant_id': '4e4ea23ed96240ea8108fd937f2bd12b', 'mac_address': 'fa:16:3e:e6:51:04', 'admin_state_up': True, 'status': 'DOWN', 'device_id': 'cb3569cf-e7e0-4b01-8d56-aaf9fbcd41a8', 'device_owner': 'compute:nova', 'standard_attr_id': 173, 'fixed_ips': [{'subnet_id': '5ed321df-19d5-4c9b-8cf2-f27f0993fe12', 'ip_address': '10.1.0.5'}], 'allowed_address_pairs': [], 'extra_dhcp_opts': [], 'security_groups': ['ad92c344-fc82-4f88-bdc7-1d7845432a2f'], 'description': '', 'binding:vnic_type': 'normal', 'binding:profile': {'migrating_to': 'np0034645398'}, 'binding:host_id': 'np0034645395', 'binding:vif_type': 'ovs', 'binding:vif_details': {'port_filter': True, 'connectivity': 'l2', 'bound_drivers': {'0': 'ovn'}}, 'port_security_enabled': True, 'trunk_details': {'trunk_id': '833832a8-c2bf-4ca9-a25a-9462f63b8926', 'sub_ports': [{'segmentation_id': 42, 'segmentation_type': 'vlan', 'port_id': 'e40925a3-4e71-4fa4-80ea-cc3a464101d6', 'mac_address': 'fa:16:3e:e5:55:14'}]}, 'tags': [], 'created_at': '2023-07-11T15:45:19Z', 'updated_at': '2023-07-11T15:45:48Z', 'revision_number': 9, 'project_id': '4e4ea23ed96240ea8108fd937f2bd12b'} (original settings {'id': '51e121cc-4e64-411e-b16a-e605cf967332', 'name': 'tempest-parent-2113033899', 'network_id': '6d3ce4b9-e66d-4137-a356-75e402377eb7', 'tenant_id': '4e4ea23ed96240ea8108fd937f2bd12b', 'mac_address': 'fa:16:3e:e6:51:04', 'admin_state_up': True, 'status': 'ACTIVE', 'device_id': 'cb3569cf-e7e0-4b01-8d56-aaf9fbcd41a8', 'device_owner': 'compute:nova', 'standard_attr_id': 173, 'fixed_ips': [{'subnet_id': '5ed321df-19d5-4c9b-8cf2-f27f0993fe12', 'ip_address': '10.1.0.5'}], 'allowed_address_pairs': [], 'extra_dhcp_opts': [], 'security_groups': ['ad92c344-fc82-4f88-bdc7-1d7845432a2f'], 'description': '', 'binding:vnic_type': 'normal', 'binding:profile': {'migrating_to': 'np0034645398'}, 'binding:host_id': 'np0034645395', 'binding:vif_type': 'ovs', 'binding:vif_details': {'port_filter': True, 'connectivity': 'l2', 'bound_drivers': {'0': 'ovn'}}, 'port_security_enabled': True, 'trunk_details': {'trunk_id': '833832a8-c2bf-4ca9-a25a-9462f63b8926', 'sub_ports': [{'segmentation_id': 42, 'segmentation_type': 'vlan', 'port_id': 'e40925a3-4e71-4fa4-80ea-cc3a464101d6', 'mac_address': 'fa:16:3e:e5:55:14'}]}, 'tags': [], 'created_at': '2023-07-11T15:45:19Z', 'updated_at': '2023-07-11T15:45:43Z', 'revision_number': 8, 'project_id': '4e4ea23ed96240ea8108fd937f2bd12b'}) host np0034645395 (original host np0034645395) vif type ovs (original vif type ovs) vif details {'port_filter': True, 'connectivity': 'l2'} (original vif details {'port_filter': True, 'connectivity': 'l2'}) binding levels [{'bound_driver': 'ovn', 'bound_segment': {'id': '21392775-0d30-44c3-9f06-aa8348d69be8', 'network_type': 'geneve', 'physical_network': None, 'segmentation_id': 10205, 'network_id': '6d3ce4b9-e66d-4137-a356-75e402377eb7'}}] (original binding levels [{'bound_driver': 'ovn', 'bound_segment': {'id': '21392775-0d30-44c3-9f06-aa8348d69be8', 'network_type': 'geneve', 'physical_network': None, 'segmentation_id': 10205, 'network_id': '6d3ce4b9-e66d-4137-a356-75e402377eb7'}}]) on network {'id': '6d3ce4b9-e66d-4137-a356-75e402377eb7', 'name': 'tempest-LiveAutoBlockMigrationV225Test-1684440653-network', 'tenant_id': '4e4ea23ed96240ea8108fd937f2bd12b', 'admin_state_up': True, 'mtu': 1372, 'status': 'ACTIVE', 'subnets': ['5ed321df-19d5-4c9b-8cf2-f27f0993fe12'], 'standard_attr_id': 55, 'shared': False, 'availability_zone_hints': [], 'availability_zones': [], 'ipv4_address_scope': None, 'ipv6_address_scope': None, 'router:external': False, 'vlan_transparent': None, 'description': '', 'port_security_enabled': True, 'tags': [], 'created_at': '2023-07-11T15:41:08Z', 'updated_at': '2023-07-11T15:41:10Z', 'revision_number': 2, 'project_id': '4e4ea23ed96240ea8108fd937f2bd12b', 'provider:network_type': 'geneve', 'provider:physical_network': None, 'provider:segmentation_id': 10205} with segments to bind None
Jul 11 15:45:49.283646 np0034645395 neutron-server[78430]: DEBUG neutron.tests.unit.plugins.ml2.drivers.mechanism_logger [None req-de6d27ae-5f07-41d8-90cb-e85d1228c53f None None] update_port_postcommit:
Jul 11 15:45:49.283646 np0034645395 neutron-server[78430]: key[status], ACTIVE -> DOWN
Jul 11 15:45:49.283646 np0034645395 neutron-server[78430]: key[updated_at], 2023-07-11T15:45:43Z -> 2023-07-11T15:45:48Z
Jul 11 15:45:49.283646 np0034645395 neutron-server[78430]: key[revision_number], 8 -> 9

Changed in neutron:
importance: Undecided → Critical
importance: Critical → High
Revision history for this message
Rodolfo Alonso (rodolfo-alonso-hernandez) wrote :
Revision history for this message
Ghanshyam Mann (ghanshyammann) wrote :
Revision history for this message
Ghanshyam Mann (ghanshyammann) wrote :

As it is blocking other changes to merge, I am marking this test unstable again, feel free to revert that once it is fixed.
thanks.

Miguel Lavalle (minsel)
Changed in neutron:
status: New → Confirmed
Revision history for this message
Rodolfo Alonso (rodolfo-alonso-hernandez) wrote :
Changed in neutron:
assignee: nobody → Rodolfo Alonso (rodolfo-alonso-hernandez)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (master)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/888769
Committed: https://opendev.org/openstack/neutron/commit/e1f887ca9ffa73dde8ab41590f2131f3297aa3cd
Submitter: "Zuul (22348)"
Branch: master

commit e1f887ca9ffa73dde8ab41590f2131f3297aa3cd
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Jul 17 23:27:58 2023 +0000

    [OVN] Skip the port status UP update during a live migration

    Skip the port status UP update during a live migration due to a
    ``PortBindingChassisUpdateEvent`` event.

    The port status will be set by the ``LogicalSwitchPortCreateUpEvent``
    and ``LogicalSwitchPortCreateDownEvent`` events, that will be issued
    when the port is deleted from the source host and created in the
    destination host. This is a planned operation and controlled by Nova,
    not a reactive event due to an unplanned host down event.

    Related-Bug: #2027605
    Change-Id: I81390af2ea2fc384423518b84de3acf7adaf9193

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.opendev.org/c/openstack/neutron/+/888776
Committed: https://opendev.org/openstack/neutron/commit/7ed79c1f7890456488a4b44769ed84fea23c9a39
Submitter: "Zuul (22348)"
Branch: master

commit 7ed79c1f7890456488a4b44769ed84fea23c9a39
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Jul 18 00:42:45 2023 +0000

    [OVN][Trunk] Set the subports correct host during live migration

    During the trunk migration, the parent port multiple port binding
    will contain the destination host. Because this update is done
    before the migration is done (in other words, the parent port
    still has two port binding registers), the method setting the
    binding profile of the subport will use the destination host
    in advance. At the end of the live migration, the subports
    host will point to the correct hostname.

    Related-Bug: #2027605
    Change-Id: I2370ea2f96e2e31dbd43bf232a63394388e6945f

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/2023.1)

Related fix proposed to branch: stable/2023.1
Review: https://review.opendev.org/c/openstack/neutron/+/892855

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Related fix proposed to branch: stable/2023.1
Review: https://review.opendev.org/c/openstack/neutron/+/892856

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/zed)

Related fix proposed to branch: stable/zed
Review: https://review.opendev.org/c/openstack/neutron/+/892883

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Related fix proposed to branch: stable/zed
Review: https://review.opendev.org/c/openstack/neutron/+/892884

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/xena)

Related fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/neutron/+/892885

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Related fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/neutron/+/892886

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/yoga)

Related fix proposed to branch: stable/yoga
Review: https://review.opendev.org/c/openstack/neutron/+/892887

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Related fix proposed to branch: stable/yoga
Review: https://review.opendev.org/c/openstack/neutron/+/892888

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/wallaby)

Related fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/neutron/+/892889

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Related fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/neutron/+/892890

description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/2023.1)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892855
Committed: https://opendev.org/openstack/neutron/commit/f0923f4b50ede1bf1be64bba9485adbcb1ef0295
Submitter: "Zuul (22348)"
Branch: stable/2023.1

commit f0923f4b50ede1bf1be64bba9485adbcb1ef0295
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Jul 17 23:27:58 2023 +0000

    [OVN] Skip the port status UP update during a live migration

    Skip the port status UP update during a live migration due to a
    ``PortBindingChassisUpdateEvent`` event.

    The port status will be set by the ``LogicalSwitchPortCreateUpEvent``
    and ``LogicalSwitchPortCreateDownEvent`` events, that will be issued
    when the port is deleted from the source host and created in the
    destination host. This is a planned operation and controlled by Nova,
    not a reactive event due to an unplanned host down event.

    Related-Bug: #2027605
    Change-Id: I81390af2ea2fc384423518b84de3acf7adaf9193
    (cherry picked from commit e1f887ca9ffa73dde8ab41590f2131f3297aa3cd)

tags: added: in-stable-zed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/zed)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892883
Committed: https://opendev.org/openstack/neutron/commit/6a9990dba1777dc5510d4fc2b249d8adc98e3e12
Submitter: "Zuul (22348)"
Branch: stable/zed

commit 6a9990dba1777dc5510d4fc2b249d8adc98e3e12
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Jul 17 23:27:58 2023 +0000

    [OVN] Skip the port status UP update during a live migration

    Skip the port status UP update during a live migration due to a
    ``PortBindingChassisUpdateEvent`` event.

    The port status will be set by the ``LogicalSwitchPortCreateUpEvent``
    and ``LogicalSwitchPortCreateDownEvent`` events, that will be issued
    when the port is deleted from the source host and created in the
    destination host. This is a planned operation and controlled by Nova,
    not a reactive event due to an unplanned host down event.

    Conflicts:
        neutron/common/ovn/utils.py

    Related-Bug: #2027605
    Change-Id: I81390af2ea2fc384423518b84de3acf7adaf9193
    (cherry picked from commit e1f887ca9ffa73dde8ab41590f2131f3297aa3cd)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/2023.1)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892856
Committed: https://opendev.org/openstack/neutron/commit/6dc93c5072f586ffbf245e4740f314d3de1e95f1
Submitter: "Zuul (22348)"
Branch: stable/2023.1

commit 6dc93c5072f586ffbf245e4740f314d3de1e95f1
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Jul 18 00:42:45 2023 +0000

    [OVN][Trunk] Set the subports correct host during live migration

    During the trunk migration, the parent port multiple port binding
    will contain the destination host. Because this update is done
    before the migration is done (in other words, the parent port
    still has two port binding registers), the method setting the
    binding profile of the subport will use the destination host
    in advance. At the end of the live migration, the subports
    host will point to the correct hostname.

    Related-Bug: #2027605
    Change-Id: I2370ea2f96e2e31dbd43bf232a63394388e6945f
    (cherry picked from commit 7ed79c1f7890456488a4b44769ed84fea23c9a39)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/zed)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892884
Committed: https://opendev.org/openstack/neutron/commit/1ce5ef7f83776512487980e3d1b0fd1364ffb0f1
Submitter: "Zuul (22348)"
Branch: stable/zed

commit 1ce5ef7f83776512487980e3d1b0fd1364ffb0f1
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Jul 18 00:42:45 2023 +0000

    [OVN][Trunk] Set the subports correct host during live migration

    During the trunk migration, the parent port multiple port binding
    will contain the destination host. Because this update is done
    before the migration is done (in other words, the parent port
    still has two port binding registers), the method setting the
    binding profile of the subport will use the destination host
    in advance. At the end of the live migration, the subports
    host will point to the correct hostname.

    Related-Bug: #2027605
    Change-Id: I2370ea2f96e2e31dbd43bf232a63394388e6945f
    (cherry picked from commit 7ed79c1f7890456488a4b44769ed84fea23c9a39)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/yoga)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892887
Committed: https://opendev.org/openstack/neutron/commit/9186d511a510bd39215581918ffb298ae6c2c4f3
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit 9186d511a510bd39215581918ffb298ae6c2c4f3
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Jul 17 23:27:58 2023 +0000

    [OVN] Skip the port status UP update during a live migration

    Skip the port status UP update during a live migration due to a
    ``PortBindingChassisUpdateEvent`` event.

    The port status will be set by the ``LogicalSwitchPortCreateUpEvent``
    and ``LogicalSwitchPortCreateDownEvent`` events, that will be issued
    when the port is deleted from the source host and created in the
    destination host. This is a planned operation and controlled by Nova,
    not a reactive event due to an unplanned host down event.

    Conflicts:
        neutron/common/ovn/utils.py

    Related-Bug: #2027605
    Change-Id: I81390af2ea2fc384423518b84de3acf7adaf9193
    (cherry picked from commit e1f887ca9ffa73dde8ab41590f2131f3297aa3cd)
    (cherry picked from commit 6a9990dba1777dc5510d4fc2b249d8adc98e3e12)

tags: added: in-stable-yoga
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892888
Committed: https://opendev.org/openstack/neutron/commit/82f06feebe190064c7a37cc30c144de71052f920
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit 82f06feebe190064c7a37cc30c144de71052f920
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Jul 18 00:42:45 2023 +0000

    [OVN][Trunk] Set the subports correct host during live migration

    During the trunk migration, the parent port multiple port binding
    will contain the destination host. Because this update is done
    before the migration is done (in other words, the parent port
    still has two port binding registers), the method setting the
    binding profile of the subport will use the destination host
    in advance. At the end of the live migration, the subports
    host will point to the correct hostname.

    Related-Bug: #2027605
    Change-Id: I2370ea2f96e2e31dbd43bf232a63394388e6945f
    (cherry picked from commit 7ed79c1f7890456488a4b44769ed84fea23c9a39)

tags: added: in-stable-xena
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/xena)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892885
Committed: https://opendev.org/openstack/neutron/commit/f2a74b8c4cc5326ee78fc1aada2e624f9a48ddfc
Submitter: "Zuul (22348)"
Branch: stable/xena

commit f2a74b8c4cc5326ee78fc1aada2e624f9a48ddfc
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Jul 17 23:27:58 2023 +0000

    [OVN] Skip the port status UP update during a live migration

    Skip the port status UP update during a live migration due to a
    ``PortBindingChassisUpdateEvent`` event.

    The port status will be set by the ``LogicalSwitchPortCreateUpEvent``
    and ``LogicalSwitchPortCreateDownEvent`` events, that will be issued
    when the port is deleted from the source host and created in the
    destination host. This is a planned operation and controlled by Nova,
    not a reactive event due to an unplanned host down event.

    Conflicts:
        neutron/common/ovn/utils.py
        neutron/common/ovn/constants.py

    Related-Bug: #2027605
    Change-Id: I81390af2ea2fc384423518b84de3acf7adaf9193
    (cherry picked from commit e1f887ca9ffa73dde8ab41590f2131f3297aa3cd)
    (cherry picked from commit 6a9990dba1777dc5510d4fc2b249d8adc98e3e12)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892886
Committed: https://opendev.org/openstack/neutron/commit/6d3c29723e5d48eefd6ba73dc152ed6c99f70ffa
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 6d3c29723e5d48eefd6ba73dc152ed6c99f70ffa
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Jul 18 00:42:45 2023 +0000

    [OVN][Trunk] Set the subports correct host during live migration

    During the trunk migration, the parent port multiple port binding
    will contain the destination host. Because this update is done
    before the migration is done (in other words, the parent port
    still has two port binding registers), the method setting the
    binding profile of the subport will use the destination host
    in advance. At the end of the live migration, the subports
    host will point to the correct hostname.

    Related-Bug: #2027605
    Change-Id: I2370ea2f96e2e31dbd43bf232a63394388e6945f
    (cherry picked from commit 7ed79c1f7890456488a4b44769ed84fea23c9a39)

tags: added: in-stable-wallaby
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/wallaby)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892889
Committed: https://opendev.org/openstack/neutron/commit/7912903b488faf16a9c914b2d1a9994e90e718bf
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 7912903b488faf16a9c914b2d1a9994e90e718bf
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Jul 17 23:27:58 2023 +0000

    [OVN] Skip the port status UP update during a live migration

    Skip the port status UP update during a live migration due to a
    ``PortBindingChassisUpdateEvent`` event.

    The port status will be set by the ``LogicalSwitchPortCreateUpEvent``
    and ``LogicalSwitchPortCreateDownEvent`` events, that will be issued
    when the port is deleted from the source host and created in the
    destination host. This is a planned operation and controlled by Nova,
    not a reactive event due to an unplanned host down event.

    Conflicts:
        neutron/common/ovn/utils.py
        neutron/common/ovn/constants.py

    Related-Bug: #2027605
    Change-Id: I81390af2ea2fc384423518b84de3acf7adaf9193
    (cherry picked from commit e1f887ca9ffa73dde8ab41590f2131f3297aa3cd)
    (cherry picked from commit 6a9990dba1777dc5510d4fc2b249d8adc98e3e12)
    (cherry picked from commit f2a74b8c4cc5326ee78fc1aada2e624f9a48ddfc)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892890
Committed: https://opendev.org/openstack/neutron/commit/9f677abd1e09bb5e9a7d25aac5f42bc49597c92c
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 9f677abd1e09bb5e9a7d25aac5f42bc49597c92c
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Jul 18 00:42:45 2023 +0000

    [OVN][Trunk] Set the subports correct host during live migration

    During the trunk migration, the parent port multiple port binding
    will contain the destination host. Because this update is done
    before the migration is done (in other words, the parent port
    still has two port binding registers), the method setting the
    binding profile of the subport will use the destination host
    in advance. At the end of the live migration, the subports
    host will point to the correct hostname.

    Related-Bug: #2027605
    Change-Id: I2370ea2f96e2e31dbd43bf232a63394388e6945f
    (cherry picked from commit 7ed79c1f7890456488a4b44769ed84fea23c9a39)
    (cherry picked from commit 6d3c29723e5d48eefd6ba73dc152ed6c99f70ffa)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.