Code review comment for lp:~yamahata/glance/lp802883

Revision history for this message
Jay Pipes (jaypipes) wrote :

OK, turns out that a recent-ish change that made configure_db() call migrate.db_sync() was causing startup times for the servers to slow down. This caused the somewhat random failure to start servers test failures because the timeout was too short.

Rather than increase the timeout, I reverted the commit that made migration run on startup (we go back to the Nova way of making the user run nova-manage db_sync to migrate the registry database). Because of this change, I was able to reset the registry database used in most functional tests to the in-memory SQLite database. This alone decreased the test run time from 50 seconds to 30 seconds or less on most runs...

I've pushed the changes to lp:~jaypipes/glance/bug802883 and am going to merge propose that. If the tests all pass, it'll merge into trunk and I'll close this proposal as Merged...

« Back to merge proposal