Should not read config options while update cells
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Invalid
|
Undecided
|
Zhenyu Zheng |
Bug Description
When perform ``nova-manage cell_v2 update_cell`` command,
we read the ``database_
configuration file if the caller don't provide this info:
http://
and they are latter saved with cell_mapping.save()
this is problematic because:
1.If I did not pass config file with --config-file, it will use the file from default route and most probably it will be cell0 file, when I did not pass ``transport_url``
and/or ``database_
by the value in the conf, which will fail in the latter uniqueness check as we already
have the cell_0 db mapped.
2.If I pass the nova-cell1.conf with --config-file, the command will probably unable to
run because I do not have api_database connection in the config file as I should not
perform upcall.
3. It seems not appropriate at all that I have to always provide the config file for
the cell I want to update when I call update cell, as I'm only updating a record in
the API_DB.
example:
root@ubuntu:
+------
| Name | UUID | Transport URL | Database Connection | Disabled |
+------
| cell0 | 00000000-
| cell1 | 8eea3d64-
+------
for case 1:
root@ubuntu:
transport_url and/or database_connection combination already exists for another cell with uuid 00000000-
for case2:
root@ubuntu:
An error has occurred:
Traceback (most recent call last):
File "/opt/stack/
ret = fn(*fn_args, **fn_kwargs)
File "/opt/stack/
cell_mapping = objects.
File "/usr/local/
result = fn(cls, context, *args, **kwargs)
File "/opt/stack/
db_mapping = cls._get_
File "/usr/local/
with self._transacti
File "/usr/lib/
return self.gen.next()
File "/usr/local/
context=
File "/usr/lib/
return self.gen.next()
File "/usr/local/
bind=
File "/usr/local/
self._start()
File "/usr/local/
engine_args, maker_args)
File "/usr/local/
"No sql_connection parameter is established")
CantStartEngine
description: | updated |
description: | updated |
description: | updated |
Changed in nova: | |
status: | In Progress → Invalid |
Fix proposed to branch: master /review. openstack. org/603998
Review: https:/