Merge lp:~soren/nova/lp714577 into lp:~hudson-openstack/nova/trunk

[Soren Hansen]

Pass timestamps to the db layer in fixed_ip_disassociate_all_by_timeout rather than converting to strings ahead of time, otherwise comparison between timestamps would often fail.

Leave it to the db layer to get comparisons between timestamps right.

Right now, we convert from a datetime object to a string using isoformat(). This yields a string such as '2011-02-09T19:52:01.355919' which sorts a greater than e.g. '2011-02-09 20:01:43.789045' due to being compared alphanumerically rather than as timestamps. This caused the cleanup routing in the network manager's periodic_task to prematurely expire leases, leading them to be recycled before dnsmasq was actually willing to let it go.

[Jay Pipes (jaypipes)]

lgtm. thx for the excellent patience in narrowing down this issue.

review: Approve

[Soren Hansen (soren)]

https://hudson.linux2go.dk/job/Nova-user-test/buildTimeTrend

The impact of this patch is quite obvious from this graph.

The two failures towards the end was me trying to revert the change for a few runs to gather the info I provided on the bug.

[Vish Ishaya (vishvananda)]

lgtm. Looks like you are making some progress with automated testing as well. You should sync up with termie and jaypipes so we don't duplicate work.

review: Approve

[Devin Carlen (devcamcar)]

lgtm

review: Approve

=== modified file 'nova/db/sqlalchemy/api.py'
--- nova/db/sqlalchemy/api.py	2011-01-28 23:31:23 +0000
+++ nova/db/sqlalchemy/api.py	2011-02-09 20:15:10 +0000
@@ -579,7 +579,7 @@
                              'AND instance_id IS NOT NULL '
                              'AND allocated = 0',
                     {'host': host,
-                     'time': time.isoformat()})
+                     'time': time})
     return result.rowcount