openstack/openstack
Code Issues Proposed changes

Update git submodules

Browse Source 
* Update kuryr-kubernetes from branch 'master'
  to 624a106fe25bfd25e446e64dea9daa4ee5c27053
  - Merge "Cleanup KuryrPort when Pod is missing"
  - Cleanup KuryrPort when Pod is missing
    
    We can easily imagine an user frustrated by his pod not getting deleted
    and opting to remove the finalizer from the Pod. If the cause of the
    deletion delay was the kuryr-controller being down, we end up with an
    orphaned KuryrPort. At the moment this causes crashes, which obviously
    it shouldn't. Moreover we should figure out how to clean up the Neutron
    port if that happens. This commit does so as explained below.
    
    1. KuryrPort on_present() will trigger its deletion when it detects that
       Pod does not longer exist.
    2. Turns out security_groups parameter passed to release_vif() was never
       used. I removed it from drivers and got rid of get_security_groups()
       call from on_finalize() as it's no longer necessary.
    3. When we cannot get the Pod in KuryrPort on_finalize() we attempt to
       gather info required to cleanup the KuryrPort and "mock" a Pod
       object. A precaution is added that any error from release_vif() is
       ignored in that case to make sure failed cleanup is not causing the
       system to go down.
    
    Change-Id: Iaf48296ff28394823f68d58362bcc87d38a2cd42
This commit is contained in:
Zuul 2022-09-22 10:46:46 +00:00 committed by Gerrit Code Review
parent 96be7c3b8e
commit 775c139989
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
kuryr-kubernetes

@ -1 +1 @@
Subproject commit 0501b798bd4e9467811d2d2c03809951e88ff859
Subproject commit 624a106fe25bfd25e446e64dea9daa4ee5c27053