openstack
/
nova
Code Issues Proposed changes

TrivialFix: Add a space between messages 

The 345 DB migration shows error messages by combining base_msg
and each specific message, but it was not readable due to lack of
a space. This patch fixes it.

This comes from a review of Ie83e7bd807c2c79e5cbe1337292c2d1989d4ac03

Change-Id: I277d0bf9e392c969fcf0721ee1588e3e65d4650a
This commit is contained in:
Ken'ichi Ohmichi 2018-02-06 12:13:11 -08:00 committed by Stephen Finucane
parent 2c1874a0ec
commit 791803de47
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
nova/db/sqlalchemy/migrate_repo/versions/345_require_online_migration_completion.py
View File

@ -39,7 +39,7 @@ def upgrade(migrate_engine):
instance_types.c.deleted == 0).scalar()
if count:
msg = (base_msg +
_('There are still %(count)i unmigrated flavors. ') % {
_(' There are still %(count)i unmigrated flavors. ') % {
'count': count})
raise exception.ValidationError(detail=msg)
@ -47,7 +47,7 @@ def upgrade(migrate_engine):
keypairs.c.deleted == 0).scalar()
if count:
msg = (base_msg +
_('There are still %(count)i unmigrated keypairs. ') % {
_(' There are still %(count)i unmigrated keypairs. ') % {
'count': count})
raise exception.ValidationError(detail=msg)
@ -55,7 +55,7 @@ def upgrade(migrate_engine):
aggregates.c.deleted == 0).scalar()
if count:
msg = (base_msg +
_('There are still %(count)i unmigrated aggregates. ') % {
_(' There are still %(count)i unmigrated aggregates. ') % {
'count': count})
raise exception.ValidationError(detail=msg)
@ -63,6 +63,6 @@ def upgrade(migrate_engine):
instance_groups.c.deleted == 0).scalar()
if count:
msg = (base_msg +
_('There are still %(count)i unmigrated instance groups. ') % {
_(' There are still %(count)i unmigrated instance groups. ') % {
'count': count})
raise exception.ValidationError(detail=msg)