scenario manager cleans up in reverse order of creation

Bug #1278255 reported by John Griffith
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
tempest
Expired
Medium
Unassigned

Bug Description

tempst/scenario/manager.py has a teradown class that explicitly deletes resources in reverse order of creation, just popping them off the stack.

The way the tests are written this is great, assuming that that tests are successful. The problem is that if there's an issue as there often is in the boot_from_volume test this might not work.

Common issue being seen in the gates currently is:
1. Boot from volume is called
2. bootable volume is created and succeeds
3. nova boot is called against that volume
        This call fails frequently timing out waiting to become ready

In this situation the test fails and bails out... and then calls clean up. Now, clean up is in a situation where it tries to pop "things" (specifically the volume) and delete it. The problem is that the volume is still in an "in-use" status from the "nova boot" process, so the cleanup fails.

Probably need to implement addCleanup here like we've done in other places that clears up the instance and frees up the volume.

Revision history for this message
John Griffith (john-griffith) wrote :
Revision history for this message
John Griffith (john-griffith) wrote :

Looking at a number of cases, one thought was to add an admin client for cinder that does a force delete in the teardown routines. This would avoid the confusion of failed instances or other things causing the teardown to fail and confusing folks when they're trying to find and log the recheck bug.

Thoughts?

Revision history for this message
Mauro S M Rodrigues (maurorodrigues) wrote :
Changed in tempest:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Andrea Frittoli (andrea-frittoli) wrote :

I don't see any hit in Kibana anymore, is this still an issue?

Revision history for this message
Luz Cazares (luz-cazares) wrote :

Looks like code has changed a lot since this bug was opened.
Currently, resource clean ups are done via waiters class and called via tearDown on the BaseTestCase instead of tempest/scenario/manager.py

Please add additional information if this is still valid

Changed in tempest:
status: Confirmed → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for tempest because there has been no activity for 60 days.]

Changed in tempest:
status: Incomplete → Expired
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.