Also, On a related note we should do something similar for /servers as well:
https://bugs.launchpad.net/nova/+bug/1007116 I'm going to take a stab at that this evening and when I do I'll push up a new Tempest test case as well. Dan ----- Original Message ----- > From: "Dan Prince" <dpri...@redhat.com> > To: "Daryl Walleck" <daryl.wall...@rackspace.com> > Cc: "openstack-qa-team" <openstack-qa-team@lists.launchpad.net>, "David > Kranz" <david.kr...@qrclab.com> > Sent: Thursday, May 31, 2012 4:45:01 PM > Subject: Re: [Openstack-qa-team] deleted flavor failure > > > > ----- Original Message ----- > > From: "Daryl Walleck" <daryl.wall...@rackspace.com> > > To: "David Kranz" <david.kr...@qrclab.com> > > Cc: "<openstack-qa-team@lists.launchpad.net>" > > <openstack-qa-team@lists.launchpad.net> > > Sent: Thursday, May 31, 2012 4:28:09 PM > > Subject: Re: [Openstack-qa-team] deleted flavor failure > > > > 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. > > Hi Daryl, > > We actually went the other way... I just updated the Tempest test > case yesterday to account for this: > > https://review.openstack.org/#/c/7932/ > > I think it is this new version of the test case that David is asking > about. > > Dan > > > > > > Daryl > > > > 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 > > -- 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
