lockutils.synchronized unit test contains a sleep

Bug #1068316 reported by Michael Still
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo-incubator
Fix Released
Wishlist
Davanum Srinivas (DIMS)
Grizzly
Fix Released
Wishlist
Davanum Srinivas (DIMS)

Bug Description

It's a horrible hack and I should fix it.

Tags: lockutils
Changed in openstack-common:
status: New → In Progress
Mark McLoughlin (markmc)
affects: openstack-common → oslo
Revision history for this message
Michael Still (mikal) wrote :

We've talked about this a little and haven't come up with a good plan to eliminate the sleep, so I am going to wishlist this bug for now until we can think of something.

Changed in oslo:
importance: Undecided → Wishlist
status: In Progress → Triaged
assignee: Michael Still (mikalstill) → nobody
Revision history for this message
Mark McLoughlin (markmc) wrote :

This is the abandoned patch that attempted to fix this: https://review.openstack.org/14556

description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo-incubator (master)

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

Changed in oslo:
assignee: nobody → Davanum Srinivas (dims-v)
status: Triaged → In Progress
Mark McLoughlin (markmc)
tags: added: lockutils
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo-incubator (master)

Reviewed: https://review.openstack.org/19027
Committed: http://github.com/openstack/oslo-incubator/commit/6024316a3ac02e4b27b5f6f379ef5cce59b05ce9
Submitter: Jenkins
Branch: master

commit 6024316a3ac02e4b27b5f6f379ef5cce59b05ce9
Author: Davanum Srinivas <email address hidden>
Date: Fri Jan 4 22:40:36 2013 -0500

    Eliminate sleep in the lockutils test case (across processes)

    Fork many processes and try to lock the same set of files
    using flock without blocking and make sure we wait for all the processes
    to finish as well.

    Fixes LP #1068316

    Change-Id: I09964b2c5af63f31b5ddee1f18eaf646f8d8ba58

Changed in oslo:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in oslo:
milestone: none → grizzly-2
status: Fix Committed → 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.