Passing functional tests leaving test data around

Bug #788920 reported by Rick Harris
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Low
Jay Pipes

Bug Description

Looks like two functional tests that are passing are still leaving test data around in the /tmp directory.

e.g.

Ran 127 tests in 27.366s

OK
rick@maverick:~/openstack/glance/lp779311$ ls /tmp
test.95547 test.99353

We should figure out which tests are doing this, and ensure that the tearDown does a proper rmtree on those dirs.

Related branches

Changed in glance:
importance: Undecided → Low
Revision history for this message
Jay Pipes (jaypipes) wrote :

So.... any test that does not call stop_servers() at the end of the test case will leave data around. This was on purpose. I wanted the test directory to persist if there was an exception/assert during a test case.

From looking through the code, looks like tests.functional.TestCurlApi.test_queued_process_flow() and test_filtered_images() are the culprits. I'll get the fixes proposed shortly.

-jay

Changed in glance:
status: New → In Progress
milestone: none → diablo-1
assignee: nobody → Jay Pipes (jaypipes)
Revision history for this message
Brian Waldon (bcwaldon) wrote :

I love having Jay around to clean up my mistakes :)

Revision history for this message
Dan Prince (dan-prince) wrote : RE: [Bug 788920] Re: Passing functional tests leaving test data around

Yep. We are all human here. We'll get some of his too.

Dan

-----Original Message-----
From: "Brian Waldon" <email address hidden>
Sent: Friday, May 27, 2011 10:51am
To: <email address hidden>
Subject: [Bug 788920] Re: Passing functional tests leaving test data around

I love having Jay around to clean up my mistakes :)

--
You received this bug notification because you are subscribed to Glance.
https://bugs.launchpad.net/bugs/788920

Title:
  Passing functional tests leaving test data around

Status in OpenStack Image Registry and Delivery Service (Glance):
  In Progress

Bug description:
  Looks like two functional tests that are passing are still leaving
  test data around in the /tmp directory.

  e.g.

  Ran 127 tests in 27.366s

  OK
  rick@maverick:~/openstack/glance/lp779311$ ls /tmp
  test.95547 test.99353

  We should figure out which tests are doing this, and ensure that the
  tearDown does a proper rmtree on those dirs.

Revision history for this message
Jay Pipes (jaypipes) wrote :

Ha! It was my mistake, too :)

Revision history for this message
Joshua McKenty (joshua-mckenty) wrote :

Who you calling human? I'm fairly certain Termie is a cyborg.

Joshua McKenty
Piston Cloud Computing, Inc.
(650) 283-6846
<email address hidden>

On 2011-05-27, at 8:05 AM, Dan Prince wrote:

> Yep. We are all human here. We'll get some of his too.
>
> Dan
>
> -----Original Message-----
> From: "Brian Waldon" <email address hidden>
> Sent: Friday, May 27, 2011 10:51am
> To: <email address hidden>
> Subject: [Bug 788920] Re: Passing functional tests leaving test data around
>
> I love having Jay around to clean up my mistakes :)
>
> --
> You received this bug notification because you are subscribed to Glance.
> https://bugs.launchpad.net/bugs/788920
>
> Title:
> Passing functional tests leaving test data around
>
> Status in OpenStack Image Registry and Delivery Service (Glance):
> In Progress
>
> Bug description:
> Looks like two functional tests that are passing are still leaving
> test data around in the /tmp directory.
>
> e.g.
>
> Ran 127 tests in 27.366s
>
> OK
> rick@maverick:~/openstack/glance/lp779311$ ls /tmp
> test.95547 test.99353
>
> We should figure out which tests are doing this, and ensure that the
> tearDown does a proper rmtree on those dirs.
>
> --
> You received this bug notification because you are a member of Glance
> Bug Team, which is subscribed to Glance.
> https://bugs.launchpad.net/bugs/788920
>
> Title:
> Passing functional tests leaving test data around
>
> Status in OpenStack Image Registry and Delivery Service (Glance):
> In Progress
>
> Bug description:
> Looks like two functional tests that are passing are still leaving
> test data around in the /tmp directory.
>
> e.g.
>
> Ran 127 tests in 27.366s
>
> OK
> rick@maverick:~/openstack/glance/lp779311$ ls /tmp
> test.95547 test.99353
>
> We should figure out which tests are doing this, and ensure that the
> tearDown does a proper rmtree on those dirs.

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in glance:
milestone: diablo-1 → 2011.3
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.