Merge "Add rescue_interface to node DB table"

This commit is contained in:
Zuul 2017-11-16 21:33:11 +00:00 committed by Gerrit Code Review
commit 5d1b3847c5
3 changed files with 40 additions and 0 deletions

View File

@ -0,0 +1,32 @@
# Licensed under the Apache License, Version 2.0 (the "License"); you may
# not use this file except in compliance with the License. You may obtain
# a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations
# under the License.
"""add rescue interface to nodes
Revision ID: 405cfe08f18d
Revises: 868cb606a74a
Create Date: 2017-02-01 16:32:32.098742
"""
# revision identifiers, used by Alembic.
revision = '405cfe08f18d'
down_revision = '868cb606a74a'
from alembic import op
import sqlalchemy as sa
def upgrade():
op.add_column('nodes', sa.Column('rescue_interface',
sa.String(255),
nullable=True))

View File

@ -175,6 +175,7 @@ class Node(Base):
management_interface = Column(String(255), nullable=True)
network_interface = Column(String(255), nullable=True)
raid_interface = Column(String(255), nullable=True)
rescue_interface = Column(String(255), nullable=True)
storage_interface = Column(String(255), nullable=True)
power_interface = Column(String(255), nullable=True)
vendor_interface = Column(String(255), nullable=True)

View File

@ -651,6 +651,13 @@ class MigrationCheckersMixin(object):
self.assertIsInstance(table.c.version.type,
sqlalchemy.types.String)
def _check_405cfe08f18d(self, engine, data):
nodes = db_utils.get_table(engine, 'nodes')
col_names = [column.name for column in nodes.c]
self.assertIn('rescue_interface', col_names)
self.assertIsInstance(nodes.c.rescue_interface.type,
sqlalchemy.types.String)
def test_upgrade_and_version(self):
with patch_with_engine(self.engine):
self.migration_api.upgrade('head')