On 5/31/2012 4:28 PM, Daryl Walleck wrote:
This Nova review broke that test: https://review.openstack.org/#/c/7735/
I haven't heard a final answer on how the Nova dev team wants to handle this. I 
thought it was at least initially reverted.

Daryl
It seems that is not the case even though the bug tickets involved say fix committed. But I think the tempest test is still wrong. Nova only needs to hold onto flavors until there are no instances that are built with it. Our test does not create any instances with the new flavor before deleting it. This is similar to the issue of cleaning up cached images on the compute nodes. Dan, do you have any comment about this?

 -David











On May 31, 2012, at 3:10 PM, David Kranz wrote:

I am still seeing the below error with the latest versions of everything. Is 
this test passing for any of you? It seems to pass in jenkins so I don't know 
what the problem could be. Also, that code says:

        # Delete the flavor
        resp, _ = self.admin_client.delete_flavor(self.new_flavor_id)
        self.assertEqual(resp.status, 202)

???        # Deleted flavors can be seen via detailed GET
        resp, flavor = self.admin_client.get_flavor_details(self.new_flavor_id)
        self.assertEqual(resp.status, 200)
        self.assertEqual(flavor['name'], self.flavor_name)

        # Deleted flavors should not show up in a list however
        resp, flavors = self.admin_client.list_flavors_with_detail()

Is it really part of the spec that deleted flavors are never purged from the 
system?





======================================================================
ERROR: Delete a flavor and ensure it is not listed
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\source\tempest\tempest\tests\compute\admin\test_flavors.py", line 
110, in test_get_flavor_details_for_deleted_flavor
    resp, flavor = self.admin_client.get_flavor_details(self.new_flavor_id)
  File "C:\source\tempest\tempest\services\nova\json\flavors_client.py", line 
39, in get_flavor_details
    resp, body = self.get("flavors/%s" % str(flavor_id))
  File "C:\source\tempest\tempest\common\rest_client.py", line 155, in get
    return self.request('GET', url)
  File "C:\source\tempest\tempest\common\rest_client.py", line 189, in request
    raise exceptions.NotFound(resp_body)
NotFound: Object not found
Details: Object not found
Details: {"itemNotFound": {"message": "The resource could not be found.", 
"code": 404}}
-------------------->>  begin captured logging<<  --------------------
tempest.common.rest_client: ERROR: Request URL: 
http://172.18.0.156:8774/v2/6b12f7a923c04f89bffb95bff997c3ef/flavors/1234
tempest.common.rest_client: ERROR: Request Body: None
tempest.common.rest_client: ERROR: Response Headers: {'date': 'Thu, 31 May 2012 
19:57:42 GMT', 'status': '404', 'content-length': '78', 'content-type': 
'application/json; charset=UTF-8', 'x-compute-request-id': 
'req-b62a1f74-9919-4f8f-9d81-1006932894c9'}
tempest.common.rest_client: ERROR: Response Body: {"itemNotFound": {"message": "The resource 
could not be found.", "code": 404}}
--------------------->>  end captured logging<<  ---------------------


--
Mailing list: https://launchpad.net/~openstack-qa-team
Post to     : openstack-qa-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openstack-qa-team
More help   : https://help.launchpad.net/ListHelp


--
Mailing list: https://launchpad.net/~openstack-qa-team
Post to     : openstack-qa-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openstack-qa-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to