Fuel for OpenStack

a lot of deadlock warning in fuel-ui tests

Bug #1618852 reported by Bulat Gaifullin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Georgy Kibardin
Fuel for OpenStack 11.0
Mitaka
Won't Fix
High
Bulat Gaifullin
Fuel for OpenStack 9.1
Newton
Fix Committed
High
Georgy Kibardin
Fuel for OpenStack 10.1

Bug Description

Fuel-UI tests contains a lot of deadlock detector warnings:

https://ci.fuel-infra.org/job/verify-fuel-ui-on-fuel-web/4561/artifact/artifacts/nailgun.log

This warnings should be fixed.

Revision history for this message
Bulat Gaifullin (bulat.gaifullin) wrote :
Changed in fuel:
milestone: none → 10.0
Roman Prykhodchenko (romcheg)
tags: added: area-python
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/364832

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/364832
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=559a3b3931e27765f654c46cffc3bb7d958121a4
Submitter: Jenkins
Branch: master

commit 559a3b3931e27765f654c46cffc3bb7d958121a4
Author: Bulat Gaifullin <email address hidden>
Date: Fri Sep 2 13:43:46 2016 +0300

    Fixed dead-locks and race conditions in tasks execution

    Also imporved detecting dead-locks in chains, instead
    of card-coded chains use generic approach - collect
    knowledge about chains and check a new one against
    knowledge base.

    Change-Id: Ie1549c14d95a372a341e1e6e616226f8ee226a03
    Partial-Bug: 1618852

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/366126

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/mitaka)

Reviewed: https://review.openstack.org/366126
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=31fb1923bb32fb74ddd97058e86e9fe47cc30678
Submitter: Jenkins
Branch: stable/mitaka

commit 31fb1923bb32fb74ddd97058e86e9fe47cc30678
Author: Bulat Gaifullin <email address hidden>
Date: Fri Sep 2 13:43:46 2016 +0300

    Fixed dead-locks and race conditions in tasks execution

    Also imporved detecting dead-locks in chains, instead
    of card-coded chains use generic approach - collect
    knowledge about chains and check a new one against
    knowledge base.

    Change-Id: Ie1549c14d95a372a341e1e6e616226f8ee226a03
    Partial-Bug: 1618852

tags: added: in-stable-mitaka
Revision history for this message
Bulat Gaifullin (bulat.gaifullin) wrote :

deadlock for chain of tables was fixed.
Also there is a lot of warning about continues locking items and this warning is not valid in all cases. so detecting locks for chain of tables with ids of rows should be fixed and this is not critical for 9.1. Bug moved to next release.

Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 10.0 → 11.0
assignee: Bulat Gaifullin (bgaifullin) → Fuel Sustaining (fuel-sustaining-team)
Georgy Kibardin (gkibardin)
Changed in fuel:
assignee: Fuel Sustaining (fuel-sustaining-team) → Georgy Kibardin (gkibardin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/445869
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=2caa56fa6e6d70b62d60867b78f0dddb33aa6b18
Submitter: Jenkins
Branch: master

commit 2caa56fa6e6d70b62d60867b78f0dddb33aa6b18
Author: Georgy Kibardin <email address hidden>
Date: Wed Mar 15 13:36:05 2017 +0300

    Moving task cache update into task creation transaction

    Doing this we avoid inclusion of task cache update statement into the next
    transaction which may cause different problems such as deadlocks.
    (update happens inside make_astute_message() function)

    Change-Id: I865b98beb621bee089cf79f1304498fd3637d64f
    Closes-Bug: #1618852

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/449047

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/newton)

Reviewed: https://review.openstack.org/449047
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=d7ee3e8e4bfdd383c39e0b6ccd25cfdd72463150
Submitter: Jenkins
Branch: stable/newton

commit d7ee3e8e4bfdd383c39e0b6ccd25cfdd72463150
Author: Georgy Kibardin <email address hidden>
Date: Wed Mar 15 13:36:05 2017 +0300

    Moving task cache update into task creation transaction

    Doing this we avoid inclusion of task cache update statement into the next
    transaction which may cause different problems such as deadlocks.
    (update happens inside make_astute_message() function)

    Change-Id: I865b98beb621bee089cf79f1304498fd3637d64f
    Closes-Bug: #1618852
    (cherry picked from commit 2caa56fa6e6d70b62d60867b78f0dddb33aa6b18)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.