Locks list shouldn't be cleared on flush call in DeadlockDetectingSession

[Alexander Kislitsky]

Locks in DB don't released on the flush call. Flush method in shouldn't call clean_locks. Cleaning of locks list on flush breaks deadlocks detecting.

For performance tests we should disable deadlocks detecting.

[Alexander Kislitsky (akislitsky)]

Looks like feature.
Bulletin for feature proposed in mailing list [openstack-dev] [Fuel] Transaction scheme

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/224799

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/224161
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=ad79aecc37d1b6e3003fffe132064433ba09e43d
Submitter: Jenkins
Branch: master

commit ad79aecc37d1b6e3003fffe132064433ba09e43d
Author: Alexander Kislitsky <email address hidden>
Date: Mon Sep 14 13:18:57 2015 +0300

    Detection of possible deadlocks improved

    Tracing of locking order of objects with ids added:
     - object modifications traced
     - object deletion traced
     - object getting traced
     - fetching first object traced
     - fetching one object traced
     - bulk upgrade query traced
     - bulk delete query traced
     - added mixin class for locking ordered objects with
       before deletion and updating

    Observing DB model added for ids tracing.
    Cleaning of locks on flush removed.
    Raising of errors on detection of possible deadlocks
    enabled only for deadlock_detector test.

    Change-Id: I73830cbcb84466171eb25f922fff4da2aaa25a1d
    Partial-Bug: #1433528
    Closes-Bug: #1452359

[OpenStack Infra (hudson-openstack)]

Change abandoned by Igor Kalnitsky (<email address hidden>) on branch: master
Review: https://review.openstack.org/224799
Reason: it was done in other patch

[Alexander Kurenyshev (akurenyshev)]

Because this bug couldn't be verified via usual methods, and there is a mandatory unit test for every commit that checks locks when flushing environment [1] , I assume that bug is gone. So I move state to Fix Released.

[1]https://github.com/openstack/fuel-web/blob/46fbd9e21e91af057335cfa7e1f29159c1eb82ac/nailgun/nailgun/test/unit/test_deadlock_detector.py#L80-L85