commit 474bff078c9c9e736f86b7394320661bde70eaf4
Author: Terry Wilson <email address hidden>
Date: Thu Dec 19 09:17:50 2019 -0600
Work around potential double row.delete() call
The update_acls code can potentially iterate over the same ACL
twice. This temporary workaround silently ignores an attempt to
delete the same row twice in the same transaction, since that
should be safe. Ultimately, refactoring the ACL code to use sets
and possibly handle the fact that an ACL could be referenced from
multiple rows should be done.
Reviewed: https:/ /review. opendev. org/700007 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=474bff078c9 c9e736f86b73943 20661bde70eaf4
Committed: https:/
Submitter: Zuul
Branch: master
commit 474bff078c9c9e7 36f86b739432066 1bde70eaf4
Author: Terry Wilson <email address hidden>
Date: Thu Dec 19 09:17:50 2019 -0600
Work around potential double row.delete() call
The update_acls code can potentially iterate over the same ACL
twice. This temporary workaround silently ignores an attempt to
delete the same row twice in the same transaction, since that
should be safe. Ultimately, refactoring the ACL code to use sets
and possibly handle the fact that an ACL could be referenced from
multiple rows should be done.
Change-Id: I895eaf4006583f edc2657a4eb527d f1ff992c5bc
Related-bug: #1857016