commit 757983a4cfe3107ea6ffd0b416790ae23d91ef2e
Author: Chris Dent <email address hidden>
Date: Wed Aug 29 13:36:16 2018 +0100
[placement] Make _ensure_aggregate context not independent
The use of the independent context on _ensure_aggregate appears to
be unnecessary. It causes file-based uses of SQLite dbs to fail
(with database locked errors, as reported in the associated bug,
1789633) and thus may mask issues with other databases. Adding the
independent context manager was the result of a series of "throw
stuff at the wall and see what sticks" patches, but it looks now
that it is not required, and in some situations causes problems.
Runs through the gate show that the behavior it was fixing (as
described in bug 1786703) is not happening.
Reviewed: https:/ /review. openstack. org/597486 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=757983a4cfe 3107ea6ffd0b416 790ae23d91ef2e
Committed: https:/
Submitter: Zuul
Branch: master
commit 757983a4cfe3107 ea6ffd0b416790a e23d91ef2e
Author: Chris Dent <email address hidden>
Date: Wed Aug 29 13:36:16 2018 +0100
[placement] Make _ensure_aggregate context not independent
The use of the independent context on _ensure_aggregate appears to
be unnecessary. It causes file-based uses of SQLite dbs to fail
(with database locked errors, as reported in the associated bug,
1789633) and thus may mask issues with other databases. Adding the
independent context manager was the result of a series of "throw
stuff at the wall and see what sticks" patches, but it looks now
that it is not required, and in some situations causes problems.
Runs through the gate show that the behavior it was fixing (as
described in bug 1786703) is not happening.
Change-Id: I1f325d55ec256d b34a4c3bbd230dc d8a91bce542
Related-Bug: #1786703
Closes-Bug: #1789633