ImagesOneServerTestJSON failure in setup not logged

Bug #1218475 reported by Andrea Frittoli
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tempest
Fix Released
Wishlist
Unassigned

Bug Description

The setup method calls teardown in case create server fails.
However in case of failure during teardown itself, the original error will be lost.

@classmethod
    def setUpClass(cls):
        super(ImagesOneServerTestJSON, cls).setUpClass()
        cls.client = cls.images_client
        if not cls.config.service_available.glance:
            skip_msg = ("%s skipped as glance is not available" % cls.__name__)
            raise cls.skipException(skip_msg)

        try:
            resp, cls.server = cls.create_server(wait_until='ACTIVE')
        except Exception:
            cls.tearDownClass()
            raise

Other tests which take a similar approach log the error before calling teardown:

        except Exception as exc:
            LOG.exception(exc)
            cls.tearDownClass()
            raise

Revision history for this message
Andrea Frittoli (andrea-frittoli) wrote :

Issue is still there.

Sean Dague (sdague)
Changed in tempest:
importance: Undecided → Wishlist
status: New → Confirmed
Shweta P (shweta-ap05)
Changed in tempest:
assignee: nobody → Shweta P (shweta-ap05)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tempest (master)

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

Changed in tempest:
status: Confirmed → In Progress
Revision history for this message
Yaroslav Lobankov (ylobankov) wrote :

The proposed patch (https://review.openstack.org/76720) is abandoned, but the status of the bug is "In progress". So moving the status back to "Confirmed".

Changed in tempest:
status: In Progress → Confirmed
assignee: Shweta P (shweta-ap05) → nobody
Revision history for this message
Ghanshyam Mann (ghanshyammann) wrote :

This is fixed by I3eae85f8e42f8ed58c87e6336792faf66c8f13aa

Revision history for this message
Ghanshyam Mann (ghanshyammann) wrote :
Changed in tempest:
status: Confirmed → 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.