diff --git a/akanda/quantum/db/models_v2.py b/akanda/quantum/db/models_v2.py index a2320e8..7f8de0f 100644 --- a/akanda/quantum/db/models_v2.py +++ b/akanda/quantum/db/models_v2.py @@ -49,7 +49,6 @@ class PortForward(model_base.BASEV2, models_v2.HasId, models_v2.HasTenant): backref=orm.backref('forwards', cascade='all,delete')) - @validates('name') def validate_name(self, key, name): assert isinstance(name, basestring) diff --git a/akanda/quantum/extensions/portforward.py b/akanda/quantum/extensions/portforward.py index f9f5022..52962e2 100644 --- a/akanda/quantum/extensions/portforward.py +++ b/akanda/quantum/extensions/portforward.py @@ -117,8 +117,8 @@ class PortforwardResource(_authzbase.ResourceDelegate): resource.update(resource_dict) # FIXME(dhellmann): This variable is undefined # but I don't know what it should have been. - if not item['private_port']: - item['private_port'] = item['public_port'] + if not resource['private_port']: + resource['private_port'] = resource['public_port'] context.session.add(resource) return self.make_dict(resource) diff --git a/akanda/quantum/plugins/ovs_quantum_plugin.py b/akanda/quantum/plugins/ovs_quantum_plugin.py index b5d48d4..0b47938 100644 --- a/akanda/quantum/plugins/ovs_quantum_plugin.py +++ b/akanda/quantum/plugins/ovs_quantum_plugin.py @@ -2,6 +2,7 @@ from quantum.plugins.openvswitch import ovs_quantum_plugin from akanda.quantum.db import models_v2 + class OVSQuantumPluginV2(ovs_quantum_plugin.OVSQuantumPluginV2): supported_extension_aliases = ( ovs_quantum_plugin.OVSQuantumPluginV2.supported_extension_aliases + diff --git a/test/functional/visibility.py b/test/functional/visibility.py index 912cc89..1f574da 100644 --- a/test/functional/visibility.py +++ b/test/functional/visibility.py @@ -11,8 +11,6 @@ class AkandaClientWrapper(client.Client): portalias_path = '/dhportalias' portforward_path = '/dhportforward' - - # portalias crud @client.APIParamsCall def list_portalias(self, **params): @@ -209,7 +207,7 @@ class VisibilityTest(unittest.TestCase): class CanSeeTestCaseMixin(object): - + def test_addressgroup(self): ag = self.c.show_addressgroup(self.group['addressgroup']['id']) assert ag