x/quark
Code Issues Proposed changes

Fixes RM7946

Browse Source 
Now that we have alembic migrations, the need for quark to automatically
create the schema is no longer required. This commit
removes two lines that cause sqlalchemy to create the schema in the
database if one does not exist.

Additionally, quite a few port and subnet tests were accidentally
leaking past the test abstraction and hitting the database. This patch
unearthed that problem, so several tests were modified to fix the issue.
This commit is contained in:
Matt Dietz
2014-07-30 20:35:29 +00:00
parent 208e3450ce
commit 90ee82ec61
3 changed files with 70 additions and 54 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
quark/plugin.py
View File

@@ -16,7 +16,6 @@
"""
v2 Neutron Plug-in API Quark Implementation
"""
from neutron.db import api as neutron_db_api
from neutron.extensions import securitygroup as sg_ext
from neutron import neutron_plugin_base_v2
from neutron.openstack.common import log as logging
@@ -24,7 +23,6 @@ from neutron import quota
from oslo.config import cfg
from quark.api import extensions
from quark.db import models
from quark.plugin_modules import ip_addresses
from quark.plugin_modules import ip_policies
from quark.plugin_modules import mac_address_ranges
@@ -92,8 +90,6 @@ class Plugin(neutron_plugin_base_v2.NeutronPluginBaseV2,
def __init__(self):
LOG.info("Starting quark plugin")
neutron_db_api.configure_db()
neutron_db_api.register_models(base=models.BASEV2)
def _fix_missing_tenant_id(self, context, resource):
"""Will add the tenant_id to the context from body.