diff --git a/octavia/tests/functional/db/test_repositories.py b/octavia/tests/functional/db/test_repositories.py index ceb73070a4..6a6043c1c1 100644 --- a/octavia/tests/functional/db/test_repositories.py +++ b/octavia/tests/functional/db/test_repositories.py @@ -426,6 +426,11 @@ class AllRepositoriesTest(base.OctaviaDBTestBase): self.assertEqual(pool, pool_dm_dict) def test_create_load_balancer_tree(self): + self.skipTest("SLQAlchemy/PySqlite transaction handling is broken. " + "Version 1.3.16 of sqlachemy changes how sqlite3 " + "transactions are handled and this test fails as " + "The LB created early in this process now disappears " + "from the transaction context.") project_id = uuidutils.generate_uuid() member = {'project_id': project_id, 'ip_address': '11.0.0.1', 'protocol_port': 80, 'enabled': True, 'backup': False, @@ -505,6 +510,11 @@ class AllRepositoriesTest(base.OctaviaDBTestBase): self.assertIsInstance(db_lb, models.LoadBalancer) def test_sqlite_transactions_broken(self): + self.skipTest("SLQAlchemy/PySqlite transaction handling is broken. " + "Version 1.3.16 of sqlachemy changes how sqlite3 " + "transactions are handled and this test fails as " + "The LB created early in this process now disappears " + "from the transaction context.") """This test is a canary for pysqlite fixing transaction handling. When this test starts failing, we can fix and un-skip the deadlock