]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Default security group table
authorAnn Kamyshnikova <akamyshnikova@mirantis.com>
Fri, 12 Dec 2014 12:30:06 +0000 (15:30 +0300)
committerAnn Kamyshnikova <akamyshnikova@mirantis.com>
Thu, 29 Jan 2015 11:08:29 +0000 (14:08 +0300)
commit79c97120de9cff4d0992b5d41ff4bbf05e890f89
tree06c97e8bcbbc81b6ef61be83188b40b5803893f8
parent910470de36a58f9742e8c2163ebdfa100d5b2795
Default security group table

This change prevents the race condition by enforcing a single default
security group via new table default_security_group. It has tenant_id
as primary key and security_group_id, which is id of default
security group. Migration that inroduces this table has sanity check that
verifies that there is no duplicate default security group in any
tenant.

This idea has come up from discussion in comments to
https://review.openstack.org/135006

DocImpact

Closes-bug: #1194579

Change-Id: Ifa8fbddd22bce4c50836cf443ebe10dff37443ef
neutron/db/migration/alembic_migrations/versions/14be42f3d0a5_default_sec_group_table.py [new file with mode: 0644]
neutron/db/migration/alembic_migrations/versions/HEAD
neutron/db/migration/cli.py
neutron/db/securitygroups_db.py
neutron/tests/functional/db/test_migrations.py
neutron/tests/unit/test_db_migration.py