From 1c7b77655135b9901c60741a276b3c8300d35046 Mon Sep 17 00:00:00 2001 From: Stephen Finucane Date: Fri, 22 Oct 2021 10:40:59 +0100 Subject: [PATCH] Update git submodules * Update nova from branch 'master' to fb083138eb03ca6de3e90e07a9c603b717e73fbe - db: Increase timeout for migration tests We're seeing quite a few timeout failures on the following tests in 'nova.tests.unit.db.main.test_migrations': - TestModelsLegacySyncMySQL.test_models_sync - TestMigrationsWalkMySQL.test_walk_versions - TestModelsSyncMySQL.test_innodb_tables - TestModelsSyncMySQL.test_models_sync Evidently MySQL is particularly affected here. Test run times are slow even on a relatively powered machine like my localhost (Lenovo T460s w/ Intel Core i7-6600U CPU + 20G RAM) and the CI machines are only making matters worse. Local experiments with alternative MySQL libraries, such 'mysqlclient', did not yield any improvements in performance so we must simply live with this for now. Do so by setting 'TIMEOUT_SCALING_FACTOR' for these tests to 4, meaning these tests will now get a whopping 640 seconds (or over 10 minutes) to execute (we set OS_TEST_TIMEOUT to 160 in 'tox.ini'). We set this for both main and API DB migrations, even though only the former is currently exhibiting issues, to head off future problems. An alternative to this would be to override the timeout on a test-by-test basis, as Cinder has done [1], but that seems more complicated for no good reason. Yet another alternative would be to reintroduce the serialization of these tests first introduced in change I6ce930fa86c82da1008089791942b1fff7d04c18, but that is left until later in the hopes that simply increasing the timeout will resolve the issues. [1] https://github.com/openstack/cinder/blob/19.0.0/cinder/tests/unit/db/test_migrations.py Change-Id: I82b9a064d77251945ff1ae99d7049f367ddde92e Signed-off-by: Stephen Finucane --- nova | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/nova b/nova index e14eef0719..fb083138eb 160000 --- a/nova +++ b/nova @@ -1 +1 @@ -Subproject commit e14eef0719eceef35e7e96b3e3d242ec79a80969 +Subproject commit fb083138eb03ca6de3e90e07a9c603b717e73fbe