ServerDiskConfigTestJSON._get_alternative_flavor cannot get the correct alternative_flavor

Bug #1257181 reported by wangchy on 2013-12-03
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tempest
Undecided
wangchy

Bug Description

tempest/api/compute/servers/test_disk_config.py:ServerDiskConfigTestJSON._get_alternative_flavor cannot get the correct alternative_flavor:

The following condition can never return true because a init never equals to a string
if int(server['flavor']['id']) == self.flavor_ref:

Four tests will fail due to this:
>>> tempest.api.compute.servers.test_disk_config.ServerDiskConfigTestJSON.test_resize_server_from_auto_to_manual[gate] 0.45 sec 1
>>> tempest.api.compute.servers.test_disk_config.ServerDiskConfigTestJSON.test_resize_server_from_manual_to_auto[gate] 0.45 sec 1
>>> tempest.api.compute.servers.test_disk_config.ServerDiskConfigTestXML.test_resize_server_from_auto_to_manual[gate] 0.45 sec 1
>>> tempest.api.compute.servers.test_disk_config.ServerDiskConfigTestXML.test_resize_server_from_manual_to_auto[gate]

wangchy (wangchy) wrote :

A change has been submitted for this:
https://review.openstack.org/#/c/59635/

Matt Riedemann (mriedem) on 2013-12-03
Changed in tempest:
status: New → In Progress
assignee: nobody → wangchy (wangchy)
Changed in tempest:
assignee: wangchy (wangchy) → Matt Riedemann (mriedem)
Matt Riedemann (mriedem) on 2013-12-03
Changed in tempest:
assignee: Matt Riedemann (mriedem) → wangchy (wangchy)
Matt Riedemann (mriedem) wrote :

The change just merged in the gate. Somewhere along the way with the patch sets the bug tag was removed from the commit message which broke the link to launchpad, but we didn't see that until after it was approved.

Changed in tempest:
status: In Progress → Fix Committed
milestone: none → icehouse-2
Sean Dague (sdague) on 2013-12-12
Changed in tempest:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers