Comment 4 for bug 1952567

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

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

commit c4adec924a1f189fbe5750f1b413db0d0f23c39a
Author: LIU Yulong <email address hidden>
Date: Tue Apr 19 15:17:19 2022 +0800

    Remove useless function _add_port_tag_info

    This reverts commit: b83fedbd78a441cf34d53dba35a3ccff7d8f4ac5.

    Since port is set to dead by default after the commits of:
    7aae31c9f9ed938760ca0be3c461826b598c7004
    0ddca284542aed89df4a22607a2da03f193f083c

    And we add the local vlan tag to the port right after it is
    bound to aviod trunk port flood issue:
    c63ebef2d58e15f4388cf064066f77b503a2f841

    So that _add_port_tag_info function is not necessary anymore,
    and we will save a large OVSDB read action which is dumping
    the entire table of Port, for hosts with a huge number of
    ports this is time-comsuming. So removed it.

    Related-Bug: #1968896
    Related-Bug: #1952567
    Change-Id: Iefd765d497c7e2d4bb093052478185125b907025