Bug 1397649 - Beaker randomly picks OpenStack flavors that are larger than necessary
Summary: Beaker randomly picks OpenStack flavors that are larger than necessary
Alias: None
Product: Beaker
Classification: Community
Component: general
Version: 23
Hardware: Unspecified
OS: Unspecified
unspecified vote
Target Milestone: 24.0
Assignee: Hui Wang
QA Contact: tools-bugs
Keywords: EasyFix, Patch
Depends On:
TreeView+ depends on / blocked
Reported: 2016-11-23 04:42 UTC by Dan Callaghan
Modified: 2017-02-21 18:48 UTC (History)
4 users (show)

Clone Of:
Last Closed: 2017-02-21 18:48:36 UTC

Attachments (Terms of Use)

Description Dan Callaghan 2016-11-23 04:42:49 UTC
When Beaker is dynamically creating an OpenStack instance to run a recipe, it's supposed to pick the "cheapest" flavor which meets the recipe's host requirements. We approximate "cheapest" by sorting flavors in order of least to greatest RAM, and then picking the first applicable one.

However when Beaker applies host requirements in needproperty.xml filter_openstack_flavor() methods, some methods are using set() which means the ordering of the passed-in list is discarded. As a result the sorting has no effect, Beaker effectively picks any random flavor meeting the host requirements, not the one with the least RAM.

Comment 1 Hui Wang 2016-12-12 08:42:26 UTC

Comment 3 Hui Wang 2016-12-22 06:02:43 UTC
Verified this issue.
The result is PASS.
Version Beaker 24.0.git.227.68bc3de 


Submit a job that has no memory host requirement.
It will take a instance with 1G memory(the cheapest one) to install in openstack.

Comment 4 Dan Callaghan 2017-02-21 18:48:36 UTC
Beaker 24.0 has been released.

Note You need to log in before you can comment on or make changes to this bug.