commit b25f6448d530b51eea4a36b3d087515e8495f53f
Author: Kevin Benton <email address hidden>
Date: Thu May 12 09:17:56 2016 -0700
Ensure ML2's create/update_port methods not in transaction
This adds a check to the ML2 create and update port methods which
are called by other services to manipulate ports. This check prevents
them from passing in a context that is already part of an ongoing DB
session because we do not want DB rollbacks to be allowed after the
ML2 plugin calls postcommit methods on drivers.
This also adds a temporary hack to set an attribute on the context
to skip this check to accomadate two L3 code-paths and a subnet
code-path that have port manipulations entangled in transactions.
This attribute will ultimately be removed once these paths are
refactored.
Reviewed: https:/ /review. openstack. org/275110 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=b25f6448d53 0b51eea4a36b3d0 87515e8495f53f
Committed: https:/
Submitter: Jenkins
Branch: master
commit b25f6448d530b51 eea4a36b3d08751 5e8495f53f
Author: Kevin Benton <email address hidden>
Date: Thu May 12 09:17:56 2016 -0700
Ensure ML2's create/update_port methods not in transaction
This adds a check to the ML2 create and update port methods which
are called by other services to manipulate ports. This check prevents
them from passing in a context that is already part of an ongoing DB
session because we do not want DB rollbacks to be allowed after the
ML2 plugin calls postcommit methods on drivers.
This also adds a temporary hack to set an attribute on the context
to skip this check to accomadate two L3 code-paths and a subnet
code-path that have port manipulations entangled in transactions.
This attribute will ultimately be removed once these paths are
refactored.
Closes-Bug: #1540844 336ab0b76c0826b 506e2dc44b6
Change-Id: I5aa099c2264636