commit c9358687cae598143cad20fffd16c6c191985ca4
Author: Kevin Benton <email address hidden>
Date: Wed Nov 9 23:25:01 2016 -0800
Separate floating IP port creation from transaction
This moves the floating IP port creation outside of the transaction
that creates the floating IP record. This eliminates the use of one
of the GUARD_TRANSACTION flags to prepare us for the enginefacade
switch and correct event notification semantics for push
notifications.
Note that this introduces a small window where the server can die
mid-process and leave an orphaned port without a floating IP. This
is similar to what can happen for router gateway ports. The intention
is to address these in a follow-up patch with a periodic garbage
collection task that will look for floating IP ports with a device
ID of 'PENDING' since the tenant cannot delete them on their own.
Reviewed: https:/ /review. openstack. org/396013 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=c9358687cae 598143cad20fffd 16c6c191985ca4
Committed: https:/
Submitter: Jenkins
Branch: master
commit c9358687cae5981 43cad20fffd16c6 c191985ca4
Author: Kevin Benton <email address hidden>
Date: Wed Nov 9 23:25:01 2016 -0800
Separate floating IP port creation from transaction
This moves the floating IP port creation outside of the transaction
that creates the floating IP record. This eliminates the use of one
of the GUARD_TRANSACTION flags to prepare us for the enginefacade
switch and correct event notification semantics for push
notifications.
Note that this introduces a small window where the server can die
mid-process and leave an orphaned port without a floating IP. This
is similar to what can happen for router gateway ports. The intention
is to address these in a follow-up patch with a periodic garbage
collection task that will look for floating IP ports with a device
ID of 'PENDING' since the tenant cannot delete them on their own.
Related-Bug: #1540844 Implements: blueprint push-notifications Implements: blueprint enginefacade-switch b64fbf06b11b0a2 9b018c6854f
Partially-
Partially-
Change-Id: Ia4c34c6654a5bf