commit 83b6ce9e9ec2f0261fed630e18cbe9cfa363a0dd
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/ +/834536 /opendev. org/openstack/ neutron/ commit/ 83b6ce9e9ec2f02 61fed630e18cbe9 cfa363a0dd
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit 83b6ce9e9ec2f02 61fed630e18cbe9 cfa363a0dd
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
Change-Id: I5b96b3a91aacff e118ddbb91a75c4 892818ba97a