openstack
/
neutron
Code Issues Proposed changes

Merge "Fix SG interface to reflect the reality"

This commit is contained in:
Jenkins 2013-02-18 07:55:03 +00:00 committed by Gerrit Code Review
parent 27c52d7c8d 434534652d
commit 4298be01bf
2 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
quantum/db/securitygroups_db.py
View File

@ -398,11 +398,11 @@ class SecurityGroupDbMixin(ext_sg.SecurityGroupPluginBase):
raise ext_sg.SecurityGroupRuleNotFound(id=id)
return sgr
def delete_security_group_rule(self, context, sgrid):
def delete_security_group_rule(self, context, id):
if (cfg.CONF.SECURITYGROUP.proxy_mode and not context.is_admin):
raise ext_sg.SecurityGroupProxyModeNotAdmin()
with context.session.begin(subtransactions=True):
rule = self._get_security_group_rule(context, sgrid)
rule = self._get_security_group_rule(context, id)
context.session.delete(rule)
def _extend_port_dict_security_group(self, context, port):

11
quantum/extensions/securitygroup.py
View File

@ -15,6 +15,7 @@
# License for the specific language governing permissions and limitations
# under the License.
from abc import ABCMeta
from abc import abstractmethod
from oslo.config import cfg
@ -317,16 +318,14 @@ class Securitygroup(extensions.ExtensionDescriptor):
class SecurityGroupPluginBase(object):
__metaclass__ = ABCMeta
@abstractmethod
def create_security_group(self, context, security_group):
pass
@abstractmethod
def delete_security_group(self, context, security_group):
pass
@abstractmethod
def update_security_group(self, context, security_group):
def delete_security_group(self, context, id):
pass
@abstractmethod
@ -342,7 +341,7 @@ class SecurityGroupPluginBase(object):
pass
@abstractmethod
def delete_security_group_rule(self, context, sgrid):
def delete_security_group_rule(self, context, id):
pass
@abstractmethod