)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"991c96aba02cf9b654ab597af0f173e23481165a","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The test test_migrate_within_cell functional test takes 160 seconds to"},{"line_number":10,"context_line":"run."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Seems like the cleanup after the test function takes the extra time."},{"line_number":13,"context_line":"By looking at the test case it starts a migration then if that call"},{"line_number":14,"context_line":"returns 202 the test finishes. So the cleanup and the migration happen"},{"line_number":15,"context_line":"in parallel. By adding a wait for the server to finish migration"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_22c5820a","line":12,"range":{"start_line":12,"start_character":15,"end_line":12,"end_character":22},"updated":"2020-01-07 17:11:14.000000000","message":"What cleanup? Is it in the base class? The test just seems to care that a 202 is returned from the API and not about the actual result.","commit_id":"7845c0cdd310b5b25275aff239debfd72f7dceb8"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"bde67cd9a290335cf3aa6cd373011c536101c99c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The test test_migrate_within_cell functional test takes 160 seconds to"},{"line_number":10,"context_line":"run."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Seems like the cleanup after the test function takes the extra time."},{"line_number":13,"context_line":"By looking at the test case it starts a migration then if that call"},{"line_number":14,"context_line":"returns 202 the test finishes. So the cleanup and the migration happen"},{"line_number":15,"context_line":"in parallel. By adding a wait for the server to finish migration"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_57c5d308","line":12,"range":{"start_line":12,"start_character":15,"end_line":12,"end_character":22},"in_reply_to":"3fa7e38b_22c5820a","updated":"2020-01-08 09:07:55.000000000","message":"I think it is the generic fixtures.fixture.Fixture.cleanUp() that runs for every functional test fixture tearing down nova dependencies like the DB that causes the excessive runtime as it makes the parallel running migration to hit the db that is down, producing such logs:\n\n\"The services table record for the nova-conductor service on host conductor is gone.\"\nhttp://paste.openstack.org/show/788119/","commit_id":"7845c0cdd310b5b25275aff239debfd72f7dceb8"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0bd2c2ef81806685fea2efc39df6f23a76ca4b68","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The test test_migrate_within_cell functional test takes 160 seconds to"},{"line_number":10,"context_line":"run."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Seems like the cleanup after the test function takes the extra time."},{"line_number":13,"context_line":"By looking at the test case it starts a migration then if that call"},{"line_number":14,"context_line":"returns 202 the test finishes. So the cleanup and the migration happen"},{"line_number":15,"context_line":"in parallel. By adding a wait for the server to finish migration"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_f45f1672","line":12,"range":{"start_line":12,"start_character":15,"end_line":12,"end_character":22},"in_reply_to":"3fa7e38b_57c5d308","updated":"2020-01-08 15:59:09.000000000","message":"OK, if that\u0027s what\u0027s causing that error it shows up in a lot of tests from what I\u0027ve seen, so we might have more than just this one. Nice catch though.","commit_id":"7845c0cdd310b5b25275aff239debfd72f7dceb8"}]}