openstack/ironic-inspector
Code Issues Proposed changes

Merge "PGSQL: create Enum before using"

Browse Source
This commit is contained in:
Jenkins 2017-07-29 21:40:20 +00:00 committed by Gerrit Code Review
commit 75199063e9
2 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ironic_inspector/migrations/versions/d2e48801c8ef_introducing_node_state_attribute.py
View File

@ -36,10 +36,12 @@ Node = sql.table('nodes',
def upgrade(): def upgrade():
state_enum = sa.Enum(*istate.States.all(), name='node_state')
state_enum.create(op.get_bind())
op.add_column('nodes', sa.Column('version_id', sa.String(36), op.add_column('nodes', sa.Column('version_id', sa.String(36),
server_default='')) server_default=''))
op.add_column('nodes', sa.Column('state', sa.Enum(*istate.States.all(), op.add_column('nodes', sa.Column('state', state_enum,
name='node_state'),
nullable=False, nullable=False,
default=istate.States.finished, default=istate.States.finished,
server_default=istate.States.finished)) server_default=istate.States.finished))

5
releasenotes/notes/pgsql-imperative-enum-dda76f150a205d0a.yaml Normal file
View File

@ -0,0 +1,5 @@
---
fixes:
- |
Create the node_state enum explicitly in the migration, as it is required
by PostgreSQL