nova-cpu.conf shouldn't have database config in it
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
devstack |
Fix Released
|
Medium
|
Dan Smith |
Bug Description
I noticed this locally but confirmed in a CI run on both stein and queens, the nova-cpu.conf generated by devstack contains database configuration in it, which is mostly harmless since we don't have "local conductor" in nova anymore (nova-compute can't reach the database directly), but it's confusing nonetheless:
[database]
connection = mysql+pymysql:
[api_database]
connection = mysql+pymysql:
The nova_cell1.conf looks good:
[database]
connection = mysql+pymysql:
I think the problem is here:
Because if you're not running cells v1 we set this variable above:
local compute_
Which is the nova.conf that the controller services (nova-api, nova-scheduler, etc) use which configures the API database and the cell database is pointed at cell0. So we copy that to create nova-cpu.conf which is convenient but also kind of wrong and confusing because of the DB setup.
An easy fix would be to just remove the nova-cpu.conf ini group entries for the database.
tags: | added: low-hanging-fruit |
Changed in devstack: | |
importance: | Low → Medium |
Changed in devstack: | |
assignee: | nobody → Matt Riedemann (mriedem) |
status: | Triaged → In Progress |
Changed in devstack: | |
assignee: | Matt Riedemann (mriedem) → Dan Smith (danms) |
Reviewed: https:/ /review. openstack. org/609755 /git.openstack. org/cgit/ openstack- dev/devstack/ commit/ ?id=7d0003ef7ed 2c2609ffe6782e0 ce6147f9c9a3bf
Committed: https:/
Submitter: Zuul
Branch: master
commit 7d0003ef7ed2c26 09ffe6782e0ce61 47f9c9a3bf
Author: Dan Smith <email address hidden>
Date: Thu Oct 11 08:59:26 2018 -0700
Make sure nova-compute is not configured with database access
Apparently we're inheriting some database config from the main file,
which should not be set for nova-compute. If we're properly in superconductor
mode where we have a dedicated config for compute, remove those lines
if present.
Closes-Bug: #1797413
Change-Id: I4820abe57a0230 50dd8d067c77e26 028801ff288