Hi Jay!

I'm working to submit a fix for this but am waiting to be added to "OpenStack Contributors".  Are you by any chance one of the team administrators who can add me?  :)

Katherine

----
Katherine Elliott
You have only to observe my dogs to peer into my soul...
Pulsar - Test/QA
Manageability Solutions Lab
Hewlett-Packard
Ft. Collins, Colorado 
MS 36, 6UE12
970-898-3565 

-----Original Message-----
From: <email address hidden> [mailto:<email address hidden>] On Behalf Of Jay Pipes
Sent: Monday, May 14, 2012 1:41 PM
To: Elliott, Katherine (MSL - Ft. Collins)
Subject: [Bug 999256] Re: images_client uses incorrect exception: BuildErrorException

** Changed in: tempest
       Status: New => Triaged

** Changed in: tempest
   Importance: Undecided => Low

** Changed in: tempest
     Assignee: (unassigned) => Katherine Elliott (katherine-elliott)

-- 
You received this bug notification because you are subscribed to the bug
report.
https://bugs.launchpad.net/bugs/999256

Title:
  images_client uses incorrect exception: BuildErrorException

Status in Tempest:
  Triaged

Bug description:
  I'm seeing this on the latest-available branch.  In images_client.py,
  both wait_for_image_resp_code and wait_for_image_status use the
  BuildErrorException, which makes for some really confusing results to
  debug.  For example, in the test_images and test_list_images tests,
  when an added images fails to get to the "ACTIVE" state, we see this
  in the output:

  ======================================================================
  ERROR: An image for the provided server should be created
  ----------------------------------------------------------------------
  Traceback (most recent call last):
    File "/tempest/tempest/tests/test_images.py", line 46, in test_create_delete_image
      self.client.wait_for_image_status(image_id, 'ACTIVE')
    File "/tempest/tempest/services/nova/json/images_client.py", line 104, in wait_for_image_status
      raise exceptions.BuildErrorException
  BuildErrorException: Server %(server_id)s failed to build and is in ERROR status

  In addition, images_client is using the exception incorrectly, not
  passing in a "server_id" string (which should be an "image_id" string
  in a correct exception):

  def wait_for_image_resp_code(self, image_id, code):
          """
          Waits until the HTTP response code for the request matches the
          expected value
          """
          resp, body = self.client.get("images/%s" % str(image_id))
          start = int(time.time())

          while resp.status != code:
              time.sleep(self.build_interval)
              resp, body = self.client.get("images/%s" % str(image_id))

              if int(time.time()) - start >= self.build_timeout:
                  raise exceptions.BuildErrorException

  To my exceptions.py here, I've added the following exception:

  class ImageAddException(TempestException):
      message = "Image %(image_id) failed to become ACTIVE in the allotted time."

  and changed the image tests to use this.

To manage notifications about this bug go to:
https://bugs.launchpad.net/tempest/+bug/999256/+subscriptions