openstack/nova
Code Issues Proposed changes

Add a TODO and add info to a releasenote

Browse Source 
This updates the release notes for the new cellsv2 discover_hosts
command to be clear that it should be run at the API level. And adds a
TODO to provide a better error message if that is not done.

Change-Id: I51f113c2c445a9806675a9ee66f883d2b7f4f8f0
This commit is contained in:
Andrew Laski 2016-08-30 10:39:04 -04:00
parent b88677c2d1
commit 3eb4d1fd1d
2 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
nova/cmd/manage.py
View File

@ -1491,6 +1491,10 @@ class CellV2Commands(object):
"""
ctxt = context.RequestContext()
# TODO(alaski): If this is not run on a host configured to use the API
# database most of the lookups below will fail and may not provide a
# great error message. Add a check which will raise a useful error
# message about running this from an API host.
if cell_uuid:
cell_mappings = [objects.CellMapping.get_by_uuid(ctxt, cell_uuid)]
else:

9
releasenotes/notes/cells-discover-hosts-06a3079ba687e092.yaml
View File

@ -6,14 +6,17 @@ features:
then anytime a new host is added to a cell this new
"nova-manage cell_v2 discover_hosts" needs to be run before instances can
be booted on that host. If multiple hosts are added at one time the command
only needs to be run one time to discover all of them.
only needs to be run one time to discover all of them. This command should
be run from an API host, or a host that is configured to use the nova_api
database.
Please note that adding a host to a cell and not running this command could
lead to build failures/reschedules if that host is selected by the
scheduler. The discover_hosts command is necessary to route requests to the
host but is not necessary in order for the scheduler to be aware of the
host. In order to avoid that it is advised that new compute hosts are
disabled until the discover command has been run.
host. It is advised that nova-compute hosts are configured with
"enable_new_services=False" in order to avoid failures before the hosts
have been discovered.
issues:
- If a deployer has updated their deployment to using cellsv2 using either
the simple_cell_setup or the map_cell0/map_cell_and_hosts/map_instances