commit b053dfe3844aaf024668847aab3498c3af559dc0
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Wed Mar 16 16:32:31 2022 +0000
Remove exception ``IpAddressAllocationNotFound``
This patch removes the ``IpAddressAllocationNotFound`` exception. This
exception was raised when a IPAM register was called to be deleted
but not found.
As reported in the LP bug, this IPAM register deletion can be called
several times if a port fails during the creation. The IPAM register
deletion calls the DB deletion but doesn't raise any exception if the
register does not exist. The code ensures the IPAM register is
deleted and there is no need to fail if it is not present anymore.
This patch also removes the exception catch and try in "update_port",
that was added in [0] as a fix for [1]. That was added because the
subnet deletion code involved a port update call [2] during the
IP allocation deletion, if any port was still present in the subnet.
Since [3], this code is not needed because the subnet deletion does
not call a port update anymore.
Reviewed: https:/ /review. opendev. org/c/openstack /neutron/ +/834652 /opendev. org/openstack/ neutron/ commit/ b053dfe3844aaf0 24668847aab3498 c3af559dc0
Committed: https:/
Submitter: "Zuul (22348)"
Branch: stable/wallaby
commit b053dfe3844aaf0 24668847aab3498 c3af559dc0
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Wed Mar 16 16:32:31 2022 +0000
Remove exception ``IpAddressAllo cationNotFound` `
This patch removes the ``IpAddressAllo cationNotFound` ` exception. This
exception was raised when a IPAM register was called to be deleted
but not found.
As reported in the LP bug, this IPAM register deletion can be called
several times if a port fails during the creation. The IPAM register
deletion calls the DB deletion but doesn't raise any exception if the
register does not exist. The code ensures the IPAM register is
deleted and there is no need to fail if it is not present anymore.
This patch also removes the exception catch and try in "update_port",
that was added in [0] as a fix for [1]. That was added because the
subnet deletion code involved a port update call [2] during the
IP allocation deletion, if any port was still present in the subnet.
Since [3], this code is not needed because the subnet deletion does
not call a port update anymore.
[0]https:/ /review. opendev. org/c/openstack /neutron/ +/373536 /bugs.launchpad .net/neutron/ +bug/1622616 /github. com/openstack/ neutron/ blob/pike- em/neutron/ db/db_base_ plugin_ v2.py#L1017- L1018 /review. opendev. org/c/openstack /neutron/ +/713045
[1]https:/
[2]https:/
[3]https:/
Closes-Bug: #1965807
Related-Bug: #1954763
Related-Bug: #1622616
Conflicts:
neutron/ ipam/exceptions .py
Change-Id: I5b96b3a91aacff e118ddbb91a75c4 892818ba97a 61fed630e18cbe9 cfa363a0dd) 2c24d68774bf286 c9e6ae2d94)
(cherry picked from commit 83b6ce9e9ec2f02
(cherry picked from commit 2a6a4ab5dacc2e9