diff --git a/neutron/extensions/securitygroup.py b/neutron/extensions/securitygroup.py index 3524117ee9e..cdbe29dcdec 100644 --- a/neutron/extensions/securitygroup.py +++ b/neutron/extensions/securitygroup.py @@ -174,7 +174,7 @@ def convert_ip_prefix_to_cidr(ip_prefix): def _validate_name_not_default(data, valid_values=None): - if data == "default": + if data.lower() == "default": raise SecurityGroupDefaultAlreadyExists() diff --git a/neutron/tests/unit/test_extension_security_group.py b/neutron/tests/unit/test_extension_security_group.py index e0061940b03..17484b965b8 100644 --- a/neutron/tests/unit/test_extension_security_group.py +++ b/neutron/tests/unit/test_extension_security_group.py @@ -332,6 +332,13 @@ class TestSecurityGroups(SecurityGroupDBTestCase): self.deserialize(self.fmt, res) self.assertEqual(res.status_int, webob.exc.HTTPConflict.code) + def test_create_default_security_group_check_case_insensitive(self): + name = 'DEFAULT' + description = 'my webservers' + res = self._create_security_group(self.fmt, name, description) + self.deserialize(self.fmt, res) + self.assertEqual(res.status_int, webob.exc.HTTPConflict.code) + def test_list_security_groups(self): with contextlib.nested(self.security_group(name='sg1', description='sg'),