Skip to content

Commit 9106b29

Browse files
Zuulopenstack-gerrit
authored andcommitted
Merge "Init placement before nova"
2 parents aed9300 + e8bad5c commit 9106b29

2 files changed

Lines changed: 12 additions & 10 deletions

File tree

lib/placement

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,6 @@ PLACEMENT_UWSGI_CONF=$PLACEMENT_CONF_DIR/placement-uwsgi.ini
4444

4545
# The placement service can optionally use a separate database
4646
# connection. Set PLACEMENT_DB_ENABLED to True to use it.
47-
# NOTE(cdent): This functionality depends on some code that is not
48-
# yet merged in nova but is coming soon.
4947
PLACEMENT_DB_ENABLED=$(trueorfalse False PLACEMENT_DB_ENABLED)
5048

5149
if is_service_enabled tls-proxy; then
@@ -151,9 +149,9 @@ function create_placement_accounts {
151149
function init_placement {
152150
if [ "$PLACEMENT_DB_ENABLED" != False ]; then
153151
recreate_database placement
154-
time_start "dbsync"
155-
$NOVA_BIN_DIR/nova-manage --config-file $NOVA_CONF api_db sync
156-
time_stop "dbsync"
152+
# Database migration will be handled when nova does an api_db sync
153+
# TODO(cdent): When placement is extracted we'll do our own sync
154+
# here.
157155
fi
158156
create_placement_accounts
159157
}

stack.sh

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -894,6 +894,8 @@ if is_service_enabled neutron; then
894894
stack_install_service neutron
895895
fi
896896

897+
# Nova configuration is used by placement so we need to create nova.conf
898+
# first.
897899
if is_service_enabled nova; then
898900
# Compute service
899901
stack_install_service nova
@@ -1184,6 +1186,13 @@ if is_service_enabled cinder; then
11841186
init_cinder
11851187
fi
11861188

1189+
# Placement Service
1190+
# ---------------
1191+
1192+
if is_service_enabled placement; then
1193+
echo_summary "Configuring placement"
1194+
init_placement
1195+
fi
11871196

11881197
# Compute Service
11891198
# ---------------
@@ -1202,11 +1211,6 @@ if is_service_enabled nova; then
12021211
init_nova_cells
12031212
fi
12041213

1205-
if is_service_enabled placement; then
1206-
echo_summary "Configuring placement"
1207-
init_placement
1208-
fi
1209-
12101214

12111215
# Extras Configuration
12121216
# ====================

0 commit comments

Comments
 (0)