Warn when a RPC call is made while a lock is held

Bug #1063222 reported by Michael Still
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Unassigned

Bug Description

We shouldn't be making RPC calls while a lock is held, as it blocks other users of the lock waiting on the RPC to return. This came up for me with the iptables lock and a call to nova-network which is being tracked in bug 1062314. I would like to see a generic warning being issued when this scenario occurs, so that we know to track down these things when they happen.

Chuck Short (zulcss)
Changed in nova:
status: New → Confirmed
Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :

Any ideas on how this can be implemented?

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :

Looks like this is in progress with a patch against oslo - https://review.openstack.org/#/c/14804/

Mark McLoughlin (markmc)
Changed in nova:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/21406
Committed: http://github.com/openstack/nova/commit/1cfc526f50686286e64ac07566f715e981916d3c
Submitter: Jenkins
Branch: master

commit 1cfc526f50686286e64ac07566f715e981916d3c
Author: Michael Still <email address hidden>
Date: Thu Feb 7 16:04:17 2013 +1100

    Update modules from common required for rpc with lock detection.

    This change updates the versions of the local, lockutils and rpc
    modules from common. This is required so that I can implement
    automated detection of rpcs being made while holding locks, which
    will come in a later review.

    Sneaks up on bug 1063222.

    Change-Id: I3689c133d1efc7425a44662abbdb434b5f36c64e

Changed in nova:
status: In Progress → Fix Committed
Michael Still (mikal)
Changed in nova:
status: Fix Committed → In Progress
Michael Still (mikal)
Changed in nova:
assignee: Michael Still (mikalstill) → nobody
status: In Progress → Triaged
Revision history for this message
Tracy Jones (tjones-i) wrote :

Michael - is this still an issue? A patch was merged for it...

Matt Riedemann (mriedem)
Changed in nova:
importance: High → Low
Revision history for this message
Joe Gordon (jogo) wrote :

unclear why this bug was re-opened. Michael can you elaborate.

Changed in nova:
status: Triaged → Incomplete
Revision history for this message
Sean Dague (sdague) wrote :

I'm moving back to fixed because no one knows why the bug was reopened.

Changed in nova:
status: Incomplete → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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