)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"da2cc93c3529eb2d37057ae485d2ee0830af60eb","unresolved":false,"context_lines":[{"line_number":11,"context_line":"fixed the tests to pass the gate."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Some of these errors are actual errors in our test code, while other are"},{"line_number":14,"context_line":"changes to Mock sintax."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: If8eabfeacacc221e92dfece9f4937875bd70751e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba3cc151_b946be4d","line":14,"updated":"2015-07-10 12:06:52.000000000","message":"s/sintax/syntax/","commit_id":"4d96555942428430b6894efcc04a2b742b3b1dfb"},{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"b9f2e742c1184b5f28bfd8e08cea5b3d1f69c33c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"fixed the tests to pass the gate."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Some of these errors are actual errors in our test code, while other are"},{"line_number":14,"context_line":"changes to Mock syntax."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes-Bug: #1473454"},{"line_number":17,"context_line":"Change-Id: If8eabfeacacc221e92dfece9f4937875bd70751e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba3cc151_3c3ff588","line":14,"updated":"2015-07-10 14:35:57.000000000","message":"Would it make sense to add a link to the changes that happened in this version of mock? Just a thought; it would help clarify which changes were test errors and which were changes to make mock work again.","commit_id":"0f588ac59b998a8a12457ffdca87292d67a0584a"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"455360f98f6b5892f6d45f36db756485ae639cef","unresolved":false,"context_lines":[{"line_number":11,"context_line":"fixed the tests to pass the gate."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Some of these errors are actual errors in our test code, while other are"},{"line_number":14,"context_line":"changes to Mock syntax."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes-Bug: #1473454"},{"line_number":17,"context_line":"Change-Id: If8eabfeacacc221e92dfece9f4937875bd70751e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba3cc151_dc8e09d0","line":14,"in_reply_to":"ba3cc151_3c3ff588","updated":"2015-07-10 14:46:45.000000000","message":"It\u0027s a good idea, but at this point I\u0027m not going to change anything that\u0027s not strictly necessary, as this bug is holding us back.","commit_id":"0f588ac59b998a8a12457ffdca87292d67a0584a"}],"cinder/volume/drivers/pure.py":[{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"da2cc93c3529eb2d37057ae485d2ee0830af60eb","unresolved":false,"context_lines":[{"line_number":817,"context_line":""},{"line_number":818,"context_line":"        vol_name \u003d self._get_vol_name(volume)"},{"line_number":819,"context_line":"        host \u003d self._get_host(connector)"},{"line_number":820,"context_line":"        if host:"},{"line_number":821,"context_line":"            host_name \u003d host[\"name\"]"},{"line_number":822,"context_line":"            LOG.info(_LI(\"Re-using existing purity host %(host_name)r\"),"},{"line_number":823,"context_line":"                     {\"host_name\": host_name})"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_395aae2b","line":820,"updated":"2015-07-10 12:06:52.000000000","message":"looks like an accidental change.","commit_id":"4d96555942428430b6894efcc04a2b742b3b1dfb"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"77fde7702ad856b9cb86e826b5e4c83458e11475","unresolved":false,"context_lines":[{"line_number":817,"context_line":""},{"line_number":818,"context_line":"        vol_name \u003d self._get_vol_name(volume)"},{"line_number":819,"context_line":"        host \u003d self._get_host(connector)"},{"line_number":820,"context_line":"        if host:"},{"line_number":821,"context_line":"            host_name \u003d host[\"name\"]"},{"line_number":822,"context_line":"            LOG.info(_LI(\"Re-using existing purity host %(host_name)r\"),"},{"line_number":823,"context_line":"                     {\"host_name\": host_name})"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_59b452d3","line":820,"in_reply_to":"ba3cc151_395aae2b","updated":"2015-07-10 12:16:26.000000000","message":"Yep, you are totally right.  :-)\n\nWill update patch once I confirm that it passes the gates and I don\u0027t need to change anything else.","commit_id":"4d96555942428430b6894efcc04a2b742b3b1dfb"}]}