Browse Source
In addition to binding:vif_type, the neutron core plugin needs to supply various information to nova's VIF driver, such as VIF security details and PCI details when SR-IOV is being used. This information is read-only, requires admin privileges, and is not intended for normal users. Rather than add separate mechanisms throughout the stack for each such requirement, the binding:capabilities port attibute, which is a dictionary and is not currently not used by nova, is renamed to binding:vif_details to serve as a general-purpose mechanism for supplying binding-specific details to the VIF driver. This patch does not remove or replace the CAP_PORT_FILTER boolean previously used in binding:capabilities. A separate patch should implement the specific key/value pairs carried by binding:vif_details to implement VIF security. Another patch will implement the key/value pairs needed for SR-IOV. The ML2 plugin now allows the bound mechanism driver to supply the binding:vif_details dictionary content, instead of just the CAP_PORT_FILTER boolean previously carried by the binding:capabilities attribute. DocImpact: Need to update portbinding extension API, but no impact on user or administrator documentation. Implements: blueprint vif-details Related-Bug: 1112912 Change-Id: I34be746fcfa73c70f72b4f9add8eff3ac88c723fchanges/52/72452/6
Bob Kukura
9 years ago
28 changed files with 231 additions and 79 deletions
@ -0,0 +1,59 @@
|
||||
# vim: tabstop=4 shiftwidth=4 softtabstop=4 |
||||
# |
||||
# Copyright 2014 OpenStack Foundation |
||||
# |
||||
# 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. |
||||
# |
||||
|
||||
"""ml2 binding:vif_details |
||||
|
||||
Revision ID: 50d5ba354c23 |
||||
Revises: 27cc183af192 |
||||
Create Date: 2014-02-11 23:21:59.577972 |
||||
|
||||
""" |
||||
|
||||
# revision identifiers, used by Alembic. |
||||
revision = '50d5ba354c23' |
||||
down_revision = '27cc183af192' |
||||
|
||||
# Change to ['*'] if this migration applies to all plugins |
||||
|
||||
migration_for_plugins = [ |
||||
'neutron.plugins.ml2.plugin.Ml2Plugin' |
||||
] |
||||
|
||||
from alembic import op |
||||
import sqlalchemy as sa |
||||
|
||||
from neutron.db import migration |
||||
|
||||
|
||||
def upgrade(active_plugins=None, options=None): |
||||
if not migration.should_run(active_plugins, migration_for_plugins): |
||||
return |
||||
|
||||
op.add_column('ml2_port_bindings', |
||||
sa.Column('vif_details', sa.String(length=4095), |
||||
nullable=False, server_default='')) |
||||
op.drop_column('ml2_port_bindings', 'cap_port_filter') |
||||
|
||||
|
||||
def downgrade(active_plugins=None, options=None): |
||||
if not migration.should_run(active_plugins, migration_for_plugins): |
||||
return |
||||
|
||||
op.add_column('ml2_port_bindings', |
||||
sa.Column('cap_port_filter', sa.Boolean(), |
||||
nullable=False, server_default=False)) |
||||
op.drop_column('ml2_port_bindings', 'vif_details') |
||||
Loading…
Reference in new issue