)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"06ec32b968ea44cb47617f992373d28a1e26b416","unresolved":false,"context_lines":[{"line_number":15,"context_line":"This patch fixes that race condition, and updates the regression test to"},{"line_number":16,"context_line":"reflect the correct results."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Blueprint: return-alternate-hosts"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Closes-Bug #1741125"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9f91af0f_a3168bf0","line":18,"updated":"2018-01-08 15:59:26.000000000","message":"Don\u0027t really need this in here. The bug existed outside of this blueprint.","commit_id":"db711bc06fcc234488b107dd22dd2a0272939174"}],"nova/compute/manager.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b9dce8606feb5f5c84e38bed7e0b0aa183a6aa44","unresolved":false,"context_lines":[{"line_number":4086,"context_line":"                exc_info \u003d sys.exc_info()"},{"line_number":4087,"context_line":"                if migration:"},{"line_number":4088,"context_line":"                    # Need to revert first to avoid a race on the retry"},{"line_number":4089,"context_line":"                    self._revert_allocation(context, instance, migration)"},{"line_number":4090,"context_line":"                self._reschedule_resize_or_reraise(context, image, instance,"},{"line_number":4091,"context_line":"                        exc_info, instance_type, request_spec,"},{"line_number":4092,"context_line":"                        filter_properties)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f91af0f_60a2b9ff","line":4089,"updated":"2018-01-08 15:28:55.000000000","message":"We need to store the result of this to check it in the condition on line 4098 below to determine if we need to handle the allocation delete the old way. Actually, now that I say that, we should just move that entire block up here because if _revert_allocation returns False, it could mean the allocation was created before we were using the migration swap stuff in conductor, and we need to remove the allocation the old way, which gets us right back into the race scenario.","commit_id":"7bd739c3773ef70a3958f935e2f522e8df480b96"}],"nova/tests/functional/regressions/test_bug_1741125.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"24668454a452a4dd7aa88953bcb5559e8f2326b2","unresolved":false,"context_lines":[{"line_number":82,"context_line":"        server \u003d self._wait_for_state_change(self.api, created_server,"},{"line_number":83,"context_line":"                \u0027VERIFY_RESIZE\u0027)"},{"line_number":84,"context_line":"        ctxt \u003d context.get_admin_context()"},{"line_number":85,"context_line":"        instance \u003d objects.Instance.get_by_uuid(ctxt, server[\"id\"])"},{"line_number":86,"context_line":"        self.assertEqual(int(self.flavor2[\"id\"]), instance.flavor.id)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f91af0f_007c0d70","line":86,"range":{"start_line":85,"start_character":8,"end_line":86,"end_character":69},"updated":"2018-01-08 15:29:55.000000000","message":"We can determine the flavor from the server response from _wait_for_state_change.","commit_id":"7bd739c3773ef70a3958f935e2f522e8df480b96"}]}