)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4e524babc3141a51dc25aabef41e87195ffc4d3d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add quota limit check and enhance roll back for cg create"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"hen a consistency group is created from a source consistency group"},{"line_number":10,"context_line":"and the number of volumes associated with the source CG exceeds"},{"line_number":11,"context_line":"the volumes quota, the creation of the CG will fail and will leave"},{"line_number":12,"context_line":"volumes in \"creating\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a71b18c_79ba0de3","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":3},"updated":"2016-12-02 01:53:27.000000000","message":"When","commit_id":"5674ed21dfa3914ec7005fd9fe9870cf7752dced"}],"cinder/tests/unit/api/contrib/test_consistencygroups.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4e524babc3141a51dc25aabef41e87195ffc4d3d","unresolved":false,"context_lines":[{"line_number":1534,"context_line":""},{"line_number":1535,"context_line":"        self.assertEqual(400, res.status_int)"},{"line_number":1536,"context_line":"        self.assertIn(\u0027message\u0027, res_dict[\u0027badRequest\u0027])"},{"line_number":1537,"context_line":"        message \u003d (\u0027The server could not comply with the request since it is \u0027"},{"line_number":1538,"context_line":"                   \u0027either malformed or otherwise incorrect.\u0027)"},{"line_number":1539,"context_line":"        self.assertEqual(message, res_dict[\u0027badRequest\u0027][\u0027message\u0027])"},{"line_number":1540,"context_line":""},{"line_number":1541,"context_line":"        cg \u003d objects.ConsistencyGroupList.get_all(self.ctxt)"},{"line_number":1542,"context_line":"        # The new cg has been deleted already."}],"source_content_type":"text/x-python","patch_set":2,"id":"3a71b18c_19a11995","line":1539,"range":{"start_line":1537,"start_character":8,"end_line":1539,"end_character":68},"updated":"2016-12-02 01:53:27.000000000","message":"I\u0027m not a big fan of asserting against text messages like this. It could change for some reason and not be obvious that it is being used in unit tests.\n\nI think the assert for the status_int is probably sufficient. If you need to do another update, I\u0027d prefer if this was removed.","commit_id":"5674ed21dfa3914ec7005fd9fe9870cf7752dced"},{"author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"change_message_id":"66d9430bba535f29b949868ba3e0622b35c27dee","unresolved":false,"context_lines":[{"line_number":1534,"context_line":""},{"line_number":1535,"context_line":"        self.assertEqual(400, res.status_int)"},{"line_number":1536,"context_line":"        self.assertIn(\u0027message\u0027, res_dict[\u0027badRequest\u0027])"},{"line_number":1537,"context_line":"        message \u003d (\u0027The server could not comply with the request since it is \u0027"},{"line_number":1538,"context_line":"                   \u0027either malformed or otherwise incorrect.\u0027)"},{"line_number":1539,"context_line":"        self.assertEqual(message, res_dict[\u0027badRequest\u0027][\u0027message\u0027])"},{"line_number":1540,"context_line":""},{"line_number":1541,"context_line":"        cg \u003d objects.ConsistencyGroupList.get_all(self.ctxt)"},{"line_number":1542,"context_line":"        # The new cg has been deleted already."}],"source_content_type":"text/x-python","patch_set":2,"id":"3a71b18c_282661ce","line":1539,"range":{"start_line":1537,"start_character":8,"end_line":1539,"end_character":68},"in_reply_to":"3a71b18c_19a11995","updated":"2016-12-02 14:31:13.000000000","message":"Done","commit_id":"5674ed21dfa3914ec7005fd9fe9870cf7752dced"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4e524babc3141a51dc25aabef41e87195ffc4d3d","unresolved":false,"context_lines":[{"line_number":1584,"context_line":"        self.assertEqual(400, res.status_int)"},{"line_number":1585,"context_line":"        self.assertIn(\u0027message\u0027, res_dict[\u0027badRequest\u0027])"},{"line_number":1586,"context_line":"        message \u003d (\u0027The server could not comply with the request since it is \u0027"},{"line_number":1587,"context_line":"                   \u0027either malformed or otherwise incorrect.\u0027)"},{"line_number":1588,"context_line":"        self.assertEqual(message, res_dict[\u0027badRequest\u0027][\u0027message\u0027])"},{"line_number":1589,"context_line":"        self.assertTrue(mock_validate.called)"},{"line_number":1590,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a71b18c_79b68ddd","line":1587,"updated":"2016-12-02 01:53:27.000000000","message":"Same.","commit_id":"5674ed21dfa3914ec7005fd9fe9870cf7752dced"}]}