Hardcoded paths prevent tests from being parallelized

Bug #1240052 reported by David Stanek
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Low
Morgan Fainberg

Bug Description

The Keystone tests create config files and database files on the fly during the course for a test run. The paths are static and not using tempfile. This means that the tests cannot be run in parallel.

David Stanek (dstanek)
Changed in keystone:
assignee: nobody → David Stanek (dstanek)
Dolph Mathews (dolph)
Changed in keystone:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

Changed in keystone:
assignee: David Stanek (dstanek) → Morgan Fainberg (mdrnstm)
status: New → In Progress
Changed in keystone:
assignee: Morgan Fainberg (mdrnstm) → Dolph Mathews (dolph)
Changed in keystone:
assignee: Dolph Mathews (dolph) → Morgan Fainberg (mdrnstm)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/65870
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=be995bcf80eb946e584922155302e38e59112255
Submitter: Jenkins
Branch: master

commit be995bcf80eb946e584922155302e38e59112255
Author: Morgan Fainberg <email address hidden>
Date: Wed Mar 19 15:42:55 2014 -0700

    Add support for parallel testr workers in Keystone

    Full support for parallel testr workers now works in keystone. The
    major item to be aware of is that if non-SQLite backends or a LIVE
    LDAP backend is being used for testing, the concurrency must be set
    to 1 to avoid conflicts. To artificially set the concurrency set
    the env variable ``TEST_RUN_CONCURRENCY`` to the desired number of
    testr workers. The default is to run one worker. To use 1 per
    available core, set ``TEST_RUN_CONCURRENCY`` to 0.

    Each worker uses a subdirectory of the test temp dir based upon
    the PID. When the worker exists (via the use of atexit), the
    worker-specific PID directory is automatically removed.

    The first time concurrent tests are run in a given environment,
    ensure that old .testrepository directories are cleaned up as they
    may introduce bad learning data on how to bin the tests based upon
    test runs that occured without concurrency.

    Closes-Bug: #1240052
    Change-Id: I5c3385a431774ce574266a82065ed6237e8b3665

Changed in keystone:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in keystone:
milestone: none → icehouse-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: icehouse-rc1 → 2014.1
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.