)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"7dc7d03dd761235c104982ef9afec58d0540e303","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The initial bug was that the initial \u0027Starting\u0027 log provided by"},{"line_number":10,"context_line":"requestlog had a different request id from the rest of the log"},{"line_number":11,"context_line":"messages for the same request. The initial assumption was that this"},{"line_number":12,"context_line":"was because since a request id was not available, one was being"},{"line_number":13,"context_line":"generated that later got replaced by the request id middleware."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"In the process of debugging that it became clear that the id was"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3a98d1f4_e0375ee1","line":12,"range":{"start_line":12,"start_character":4,"end_line":12,"end_character":17},"updated":"2016-09-28 21:05:04.000000000","message":"because","commit_id":"4b35954467c7aa014437e64c96b385d7d642cf66"}],"nova/api/openstack/placement/deploy.py":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"c18c7fb7af8d87c806872f4448cdbfebaeab2cd1","unresolved":false,"context_lines":[{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    for middleware in (microversion_middleware,"},{"line_number":53,"context_line":"                       fault_wrap,"},{"line_number":54,"context_line":"                       request_log,"},{"line_number":55,"context_line":"                       context_middleware,"},{"line_number":56,"context_line":"                       auth_middleware,"},{"line_number":57,"context_line":"                       req_id_middleware,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a8bc5a2_8fca216d","line":54,"range":{"start_line":54,"start_character":23,"end_line":54,"end_character":35},"updated":"2016-09-23 06:32:31.000000000","message":"Maybe we can test this like unittest at https://github.com/openstack/nova/blob/master/nova/tests/unit/api/openstack/placement/test_requestlog.py#L49\n\nMake fake PlacementHandler(), and fake a request id, also mock one of method in the requestlog middleware. In the mock method, we ensure the request id is the expected one.\n\n\nOr maybe we should add a note at here to notice this order is on purpose.","commit_id":"ffb6dac4677d767e2bc38802a026fa2e049079b8"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"d1080eb62cdd6442d407f7f1fb7334dd209b016c","unresolved":false,"context_lines":[{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    for middleware in (microversion_middleware,"},{"line_number":53,"context_line":"                       fault_wrap,"},{"line_number":54,"context_line":"                       request_log,"},{"line_number":55,"context_line":"                       context_middleware,"},{"line_number":56,"context_line":"                       auth_middleware,"},{"line_number":57,"context_line":"                       req_id_middleware,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a8bc5a2_52fd1439","line":54,"in_reply_to":"5a8bc5a2_8fca216d","updated":"2016-09-23 11:40:40.000000000","message":"Both sounds like a good idea, I\u0027ll see what I can make happen.","commit_id":"ffb6dac4677d767e2bc38802a026fa2e049079b8"}]}