python-tempestconf/config_tempest/flavors.py
Luigi Toscano 55e48c0a62 Fix: return the flavor ID on autodetection
The name was returned incorrectly (and inconsistently with the other
possible computation branches).

Also, fix a typo in a user-visible string.

Change-Id: I799a7239bad28c6c0d44ee399903aa67645aa45a
2018-08-06 20:32:34 +02:00

143 lines
5.6 KiB
Python

# Copyright 2016 Red Hat, Inc.
# All Rights Reserved.
#
# Licensed under the Apache License, Version 2.0 (the "License"); you may
# not use this file except in compliance with the License. You may obtain
# a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations
# under the License.
from operator import itemgetter
from config_tempest.constants import LOG
class Flavors(object):
def __init__(self, client, allow_creation, conf):
"""Init.
:type client: FlavorsClient object from tempest lib
:type allow_creation: boolean
:type conf: TempestConf object
"""
self.client = client
self.allow_creation = allow_creation
self._conf = conf
self.flavor_list = self.client.list_flavors()['flavors']
def create_tempest_flavors(self):
"""Find or create flavors and set them in conf.
If 'flavor_ref' and 'flavor_ref_alt' are specified in conf, it will
try to find them, if not found, it raises an Exception.
Otherwise it will try finding or creating 'm1.nano' and 'm1.micro'
flavors and set their ids in conf.
"""
prefs = [
{'key': 'flavor_ref', 'name': 'm1.nano', 'ram': 64},
{'key': 'flavor_ref_alt', 'name': 'm1.micro', 'ram': 128}
]
for pref in prefs:
flavor_id = None
if self._conf.has_option('compute', pref['key']):
flavor_id = self._conf.get('compute', pref['key'])
flavor_id = self.find_flavor_by_id(flavor_id)
if flavor_id is None:
raise Exception("%s id '%s' specified by user doesn't"
" exist", pref['key'], flavor_id)
else:
# create m1.nano/m1.micro flavor
flavor_id = self.create_flavor(pref['name'], ram=pref['ram'])
self._conf.set('compute', pref['key'], flavor_id)
def create_flavor(self, flavor_name, ram=64, vcpus=1, disk=0):
"""Create flavors or try to discover two smallest ones available.
:param flavor_name: flavor name to be created (usually m1.nano or
m1.micro)
:param ram: memory of created flavor in MB
:param vcpus: number of VCPUs for the flavor
:param disk: size of disk for flavor in GB
"""
flavor_id = self.find_flavor_by_name(flavor_name)
if flavor_id is not None:
LOG.info("(no change) Found flavor '%s'", flavor_name)
return flavor_id
elif self.allow_creation:
LOG.info("Creating flavor '%s'", flavor_name)
resp = self.client.create_flavor(name=flavor_name,
ram=ram, vcpus=vcpus,
disk=disk, id=None)
return resp['flavor']['id']
else:
if len(self.flavor_list) < 2:
raise Exception("Creation of flavors is not allowed and not "
"enough available flavors found. Either use --"
"create argument or create flavors manually.")
# return id of the discovered flavor
return self.discover_smallest_flavor(flavor_name)
def find_flavor_by_id(self, flavor_id):
"""Look for a flavor by its id.
:type flavor_id: string
:return: flavor id or None if not found
:rtype: string or None
"""
found = [f for f in self.flavor_list if f['id'] == flavor_id]
if found:
LOG.info("Found flavor '%s' by it's id '%s'",
found[0]['name'], flavor_id)
# return flavor's id
return found[0]['id']
return None
def find_flavor_by_name(self, flavor_name):
"""Look for a flavor by its name.
:type flavor_name: string
:return: flavor id or None if not found
:rtype: string or None
"""
found = [f for f in self.flavor_list if f['name'] == flavor_name]
if found:
# return flavor's id
return found[0]['id']
return None
def discover_smallest_flavor(self, flavor_name=""):
"""Discover the two smallest available flavors in the system.
If flavor_name contains "micro", the method returns the second
smallest flavor found.
:param flavor_name: [m1.nano, m1.micro]
"""
LOG.warning("Flavor '%s' not found and creation is not allowed. "
"Trying to autodetect the smallest flavor available.",
flavor_name)
flavors = []
for flavor in self.flavor_list:
f = self.client.show_flavor(flavor['id'])['flavor']
flavors.append((f['name'], f['id'], f['ram'],
f['disk'], f['vcpus']))
# order by ram, disk size and vcpus number and take first two of them
flavors = sorted(flavors, key=itemgetter(2, 3, 4))[:2]
f = None
if "micro" in flavor_name:
# take the second smaller one
f = flavors[1]
else:
f = flavors[0]
LOG.warning("Found '%s' flavor (id: '%s', ram: '%s', disk: '%s', "
"vcpus: '%s') ", f[0], f[1], f[2], f[3], f[4])
# return flavor's id
return f[1]