openstack/kolla
Code Issues Proposed changes

Fix invalid test in cinder data migration script

Browse Source 
The extend_start.sh script for cinder-api uses an incorrect test to
check if the MAX_NUMBER variable is set. This leads to a incorrect
cinder-manage command being run if this variable is not set, making the
database migration fail.

This change fixes the test to properly use the MAX_NUMBER variable.

Change-Id: Ie46da709db0de09a9cf641b3c154275282f213fc
Closes-Bug: #1866827
This commit is contained in:
Gauvain Pocentek 2020-03-10 12:46:17 +01:00
parent 031b17226a
commit ff42ff2004
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
docker/cinder/cinder-api/extend_start.sh
View File

@ -9,7 +9,7 @@ if [[ "${!KOLLA_BOOTSTRAP[@]}" ]]; then
fi
if [[ "${!KOLLA_OSM[@]}" ]]; then
if [[ -z ${MAX_NUMBER} ]]; then
if [[ "${!MAX_NUMBER[@]}" ]]; then
cinder-manage db online_data_migrations --max_count ${MAX_NUMBER}
else
cinder-manage db online_data_migrations

5
releasenotes/notes/bug-1866827-5351ec43486d7f33.yaml Normal file
View File

@ -0,0 +1,5 @@
---
fixes:
- |
Fixes the MAX_NUMBER variable usage when running the database online
migrations for cinder.