openstack
/
openstack
Code Issues Proposed changes

Update git submodules 

* Update neutron from branch 'master'
  to 1374b01cfb37c4f1ff745a1614403e6d913bd13c
  - Merge "Discard port with ofport -1 in _get_ofport_moves"
  - Discard port with ofport -1 in _get_ofport_moves
    
    When libvirt (nova) detach a port on OVS bridge, two events are sent:
    * one event with 2 actions "old" and "new": a change on ofport (from a
      regular value to -1)
    * a second event with action "delete"
    
    If, for some reason, the second event is delayed, the rpc_loop iteration
    will consider this port as "updated" instead of "deleted".
    But, because ofport == -1, the port update will be discarded, and
    finally removed from port_info["current"].
    
    As a result, on next iteration, the deletion wont be performed.
    
    Most of the time, we endup with some leftovers (like openflow rules,
    etc.)
    
    The purpose of this patch is very simple, when looping over ports in
    _get_ofport_moves, we will discards the ports that have ofport == -1, so
    the port will not be considered as updated and next iteration will be
    able to delete it correctly.
    
    Closes-Bug: #1992109
    
    Change-Id: Ib4a7183867e1b21810b6915a475a234278bf884c
    Signed-off-by: Arnaud Morin <arnaud.morin@ovhcloud.com>
This commit is contained in:
Zuul 2023-01-09 06:33:30 +00:00 committed by Gerrit Code Review
parent 9ea1bbc9e9
commit 1944ba5df1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
neutron

@ -1 +1 @@
Subproject commit d59bf5ff6c75c9f32129cd4ee49c2acb23e8bb9c
Subproject commit 1374b01cfb37c4f1ff745a1614403e6d913bd13c