Merge lp:~sleepsonthefloor/nova/709057 into lp:~hudson-openstack/nova/trunk
Proposed by
Anthony Young
Status: | Merged |
---|---|
Approved by: | Vish Ishaya |
Approved revision: | 641 |
Merged at revision: | 644 |
Proposed branch: | lp:~sleepsonthefloor/nova/709057 |
Merge into: | lp:~hudson-openstack/nova/trunk |
Diff against target: |
150 lines (+68/-13) 3 files modified
nova/db/sqlalchemy/api.py (+14/-2) nova/db/sqlalchemy/models.py (+5/-1) nova/tests/test_compute.py (+49/-10) |
To merge this branch: | bzr merge lp:~sleepsonthefloor/nova/709057 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Soren Hansen (community) | Approve | ||
Thierry Carrez (community) | gfe | Approve | |
Devin Carlen (community) | Approve | ||
Rick Harris (community) | Approve | ||
Review via email: mp+47845@code.launchpad.net |
Description of the change
Fixes bug #709057
* Mark security_
* Mark security_
* Make SecurityGroup.
* Add tests
IMO, this is important to merge. Nebula has monitoring projects that continually launch and destroy instances, and I imagine other real-world deployments may have similar health checkers. After a few weeks such health checkers choke due to the inefficient queries related to #709057.
To post a comment you must log in.
I think this is a good candidate for an exception. It is a pretty annoying performance error for large deployments, and it is limited to the db layer.