)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":20553,"name":"Brian Ober","email":"bober@us.ibm.com","username":"ober37"},"change_message_id":"173b593465bb3c54c112a762fcfacec3ded9aedf","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Previously SwiftLogFormatter would make two checks. One to see if the"},{"line_number":10,"context_line":"transaction id was already in the message field and another check to make sure"},{"line_number":11,"context_line":"the log level wasn\u0027t set to info. If either of these was true, then it would"},{"line_number":12,"context_line":"not log the transaction ID in the transaction ID field."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This commit removes the check for the info log. Now transaction IDs will be"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a122d0e_b0ec86f4","line":11,"range":{"start_line":11,"start_character":71,"end_line":11,"end_character":76},"updated":"2016-04-22 17:20:27.000000000","message":"Want to reformat this.  Take a look at the following:\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"88c3788f4ceca22985401c81bba46c0680b94f53"}],"test/unit/common/test_utils.py":[{"author":{"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},"change_message_id":"3b3c7db7d5ee96fcbb498a8c4a4f380688ec1f8e","unresolved":false,"context_lines":[{"line_number":1666,"context_line":"            logger.error(\u0027test\u0027)"},{"line_number":1667,"context_line":"            log_msg \u003d strip_value(sio)"},{"line_number":1668,"context_line":"            self.assertTrue(\u0027txn\u0027 in log_msg)"},{"line_number":1669,"context_line":"            self.assertTrue(\u002712345\u0027 in log_msg)"},{"line_number":1670,"context_line":"            # test txn already in message"},{"line_number":1671,"context_line":"            self.assertEqual(logger.txn_id, \u002712345\u0027)"},{"line_number":1672,"context_line":"            logger.warning(\u0027test 12345 test\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_d02bf831","line":1669,"updated":"2016-04-22 22:20:56.000000000","message":"Would it be better to assert that the log info did have the transaction id included?  Instead of removing the lines, you could keep them all in, but change the \"not in\" to be \"in\".  With the changes here, I could revert the changes in utils and there would be no test failure.","commit_id":"94f5d4ffe298e21b3033129476723e427f472c6d"}]}