When abnormal exit nova-network, the file lock was not released

Bug #1026892 reported by zhouliang
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Invalid
High
Unassigned
Nominated for Essex by Russell Bryant

Bug Description

When nova-network was launched and exit abnormally, the nova-iptable.lock was not deleted.
So the next time it was launched, it failed of getting file lock

Revision history for this message
zhouliang (liang-a-zhou) wrote :
Revision history for this message
Russell Bryant (russellb) wrote :

I've seen this come up on IRC multiple times.

Changed in nova:
status: New → Confirmed
importance: Undecided → High
Revision history for this message
Russell Bryant (russellb) wrote :

Based on the log, it looks like this was from Essex. The related code has been reworked significantly in Folsom, and should be fixed there. Here are the related commits:

commit 2c1524866acb9f9ac3f50dc7d33338cfb03fd08a
Author: Stanislaw Pitucha <email address hidden>
Date: Thu Aug 2 13:51:07 2012 +0100

    Improve external lock implementation

    Remove a number of limitations from the external locks.
    - They can be nested now
    - They do not need external cleanup in case of failures
    - They do not rely on lockfile or greenlet internal implementation

    New implementation is based on fcntl locks and any crashing process
    will drop the lock. It does not have to rely on any cleanup code or
    handling exceptions. Because no cleanup is needed, a number of tests
    have been removed.
    This implementation is not portable outside of POSIX/BSD/SVR4 systems.

    Fcntl locks should work correctly with NFS mounts.

    Locks are cleaned up after the tests finish running via run_tests.sh,
    even though it's not strictly needed.

    This change requires eventlet >= 0.9.17.

    bp improve-external-locking

    Change-Id: Idf5424c04645f25097733848a007b150145b0b27

commit 3f0ef8e28fd0ec5102a58be8d0013900194869fe
Author: Pádraig Brady <email address hidden>
Date: Tue Aug 7 01:51:58 2012 +0100

    Improve external locking on Windows

    commit 2c15248 improved external locking for POSIX.
    See that commit for advantages of the technique used.
    This implements the same technique for Windows systems.

    Fixes bug: 1033596
    Change-Id: I298309bb1c84499f2060c172213d355aa7ce54e5

tags: added: essex-backport
Revision history for this message
Russell Bryant (russellb) wrote :

I've nominated this bug for Essex, but will be marking the main bug as Invalid since it has already been fixed in the main tree. Thanks!

Changed in nova:
status: Confirmed → Invalid
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.