testr should preserve existing OS_* environment variables values

Bug #1182416 reported by Roman Podoliaka
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Undecided
Roman Podoliaka

Bug Description

Currently .testr.conf specifies that tests are run by executing of the following command:
OS_STDOUT_CAPTURE=1 OS_STDERR_CAPTURE=1 OS_TEST_TIMEOUT=60 ${PYTHON:-python} -m subunit.run discover -t ./ ./nova/tests $LISTOPT $IDOPTION

It should be possible to override the values of OS_* environment variables without making changes to .testr.conf, so one can specify those env vars for each tox/runtests.sh call or for the current shell session using export syntax.

Tags: tests
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/29902

Changed in nova:
assignee: nobody → Roman Podolyaka (rpodolyaka)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/29902
Committed: http://github.com/openstack/nova/commit/e4b85c10a66075a8bd1241e9945843be549aa813
Submitter: Jenkins
Branch: master

commit e4b85c10a66075a8bd1241e9945843be549aa813
Author: Roman Podolyaka <email address hidden>
Date: Tue May 21 13:45:40 2013 +0300

    Make testr preserve existing OS_* env vars values

    Values for OS_TEST_TIMEOUT, OS_STDOUT_CAPTURE and OS_STDERR_CAPTURE
    environment variables are fixed in .testr.conf now. It should be possible
    to override them for each tox/runtests.sh call or for the current shell
    session without making changes to .testr.conf.

    Fixes bug 1182416.

    Change-Id: I6c7839fa78b53ac8ea358ce0143a00931da96a9e

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