)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16237,"name":"venkatamahesh","email":"venkatamaheshkotha@gmail.com","username":"venkatamahesh"},"change_message_id":"dec61b6ce530bffdc59c9e4f59581c2eba42b2c6","unresolved":false,"context_lines":[{"line_number":15,"context_line":"Tests updated appropriately."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I1e635db8b06d3d7a78573e6e8a2a4c25b7fb458d"},{"line_number":18,"context_line":"Closes-Bug: 1500583"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a1a9d01_1bef12ea","line":18,"updated":"2015-09-29 18:11:02.000000000","message":"Closes-Bug: #1500583","commit_id":"4f0e28756ac78ada21b4ddd2a22d234590402700"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"96cd2a87b20307396eb54a08a36f818555fb7e12","unresolved":false,"context_lines":[{"line_number":15,"context_line":"Tests updated appropriately."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I1e635db8b06d3d7a78573e6e8a2a4c25b7fb458d"},{"line_number":18,"context_line":"Closes-Bug: 1500583"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a1a9d01_76e66516","line":18,"in_reply_to":"9a1a9d01_1bef12ea","updated":"2015-09-29 18:28:45.000000000","message":"What am I missing here?  That seems to be the right number and the system recognized that this commit was against that bug.","commit_id":"4f0e28756ac78ada21b4ddd2a22d234590402700"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"36bf95e5b8b306099449641fc140af7b1259da5d","unresolved":false,"context_lines":[{"line_number":15,"context_line":"Tests updated appropriately."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I1e635db8b06d3d7a78573e6e8a2a4c25b7fb458d"},{"line_number":18,"context_line":"Closes-Bug: 1500583"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a1a9d01_77465550","line":18,"in_reply_to":"9a1a9d01_76e66516","updated":"2015-09-30 01:00:09.000000000","message":"Usually the format is \"Closes-Bug: #1500583\".  You missed \"#\".  However, the bug link works.  So update it if you are doing another patch, but it is not a blocking issue.","commit_id":"4f0e28756ac78ada21b4ddd2a22d234590402700"}],"cinder/tests/unit/test_dellscapi.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"36bf95e5b8b306099449641fc140af7b1259da5d","unresolved":false,"context_lines":[{"line_number":5215,"context_line":"        res \u003d self.scapi._find_sc_cg("},{"line_number":5216,"context_line":"            {},"},{"line_number":5217,"context_line":"            \u0027GUID3-0869559e-6881-454e-ba18-15c6726d33c1\u0027)"},{"line_number":5218,"context_line":"        self.assertEqual(None, res)"},{"line_number":5219,"context_line":""},{"line_number":5220,"context_line":"    @mock.patch.object(dell_storagecenter_api.HttpClient,"},{"line_number":5221,"context_line":"                       \u0027get\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a1a9d01_775df513","line":5218,"updated":"2015-09-30 01:00:09.000000000","message":"assertIsNone is recommended.","commit_id":"4f0e28756ac78ada21b4ddd2a22d234590402700"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"36bf95e5b8b306099449641fc140af7b1259da5d","unresolved":false,"context_lines":[{"line_number":5228,"context_line":"        res \u003d self.scapi._find_sc_cg("},{"line_number":5229,"context_line":"            {},"},{"line_number":5230,"context_line":"            \u0027GUID1-0869559e-6881-454e-ba18-15c6726d33c1\u0027)"},{"line_number":5231,"context_line":"        self.assertEqual(None, res)"},{"line_number":5232,"context_line":""},{"line_number":5233,"context_line":"    @mock.patch.object(dell_storagecenter_api.StorageCenterApi,"},{"line_number":5234,"context_line":"                       \u0027_find_sc_cg\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a1a9d01_179d99bb","line":5231,"updated":"2015-09-30 01:00:09.000000000","message":"Same here.","commit_id":"4f0e28756ac78ada21b4ddd2a22d234590402700"}],"cinder/volume/drivers/dell/dell_storagecenter_api.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"03cb94feaa94c45df15d7050b112a8309bf6d3b3","unresolved":false,"context_lines":[{"line_number":2103,"context_line":"        LOG.debug(\u0027Expiring consistency group replay %s\u0027, replayid)"},{"line_number":2104,"context_line":"        replays \u003d self._find_cg_replays(profile,"},{"line_number":2105,"context_line":"                                        replayid)"},{"line_number":2106,"context_line":"        LOG.debug(replays)"},{"line_number":2107,"context_line":"        for replay in replays:"},{"line_number":2108,"context_line":"            r \u003d self.client.post(\u0027StorageCenter/ScReplay/%s/Expire\u0027"},{"line_number":2109,"context_line":"                                 % self._get_id(replay),"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_22e2ef2f","line":2106,"updated":"2015-09-29 17:00:38.000000000","message":"I know this is a nit, but having LOG.debug messages that don\u0027t explain what the output is seems like really poor form.  Could we please add something here that gives some context?\n\nThanks.","commit_id":"1f6efc1add9e8aab510a3e490bc360db5cd6353f"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"422357479cbb8fd69ca6cf06e536f333e4e03778","unresolved":false,"context_lines":[{"line_number":2103,"context_line":"        LOG.debug(\u0027Expiring consistency group replay %s\u0027, replayid)"},{"line_number":2104,"context_line":"        replays \u003d self._find_cg_replays(profile,"},{"line_number":2105,"context_line":"                                        replayid)"},{"line_number":2106,"context_line":"        LOG.debug(replays)"},{"line_number":2107,"context_line":"        for replay in replays:"},{"line_number":2108,"context_line":"            r \u003d self.client.post(\u0027StorageCenter/ScReplay/%s/Expire\u0027"},{"line_number":2109,"context_line":"                                 % self._get_id(replay),"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_42b28395","line":2106,"in_reply_to":"9a1a9d01_22e2ef2f","updated":"2015-09-29 17:12:23.000000000","message":"More to the point I don\u0027t need to be logging the entire structure.  Logging shuffled around with context added.","commit_id":"1f6efc1add9e8aab510a3e490bc360db5cd6353f"},{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"beca3a0e75ce81e15de41054ea7896f5141e71df","unresolved":false,"context_lines":[{"line_number":2083,"context_line":""},{"line_number":2084,"context_line":"            replays \u003d self._get_json(r)"},{"line_number":2085,"context_line":"        else:"},{"line_number":2086,"context_line":"            LOG.error(_LE(\u0027Unable to locate snapshot %s\u0027), replayid)"},{"line_number":2087,"context_line":""},{"line_number":2088,"context_line":"        return replays"},{"line_number":2089,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9a1a9d01_4e66f751","line":2086,"updated":"2015-09-30 18:05:57.000000000","message":"\".\" at EOL","commit_id":"e529e4d6cc71003c53126612dd869fa6fb23493a"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"b9131117b7d290eaf547e3bc1aebec4b087b1e0a","unresolved":false,"context_lines":[{"line_number":2083,"context_line":""},{"line_number":2084,"context_line":"            replays \u003d self._get_json(r)"},{"line_number":2085,"context_line":"        else:"},{"line_number":2086,"context_line":"            LOG.error(_LE(\u0027Unable to locate snapshot %s\u0027), replayid)"},{"line_number":2087,"context_line":""},{"line_number":2088,"context_line":"        return replays"},{"line_number":2089,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9a1a9d01_bf3d7bc5","line":2086,"in_reply_to":"9a1a9d01_4e66f751","updated":"2015-09-30 18:59:14.000000000","message":"DIE DIE DIE!!!!","commit_id":"e529e4d6cc71003c53126612dd869fa6fb23493a"}]}
