)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"03f0c2db37cf2761f5e5b5f806e61608575ecb30","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This reverts commit 1feb19c83ff61a3782ce862d71b8d04e21d7aa00."},{"line_number":10,"context_line":"Better fix in I23c5faaeab69aede1fd038a36f4a0b8f928498ce but that"},{"line_number":11,"context_line":"make take time due to dependency on a nova patch."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Iaa4d85f63cb8e058d09858636ce1434352e39bed"},{"line_number":14,"context_line":"Closes-Bug: #1673910"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a1ced50_db6860bd","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":4},"updated":"2017-03-18 03:44:31.000000000","message":"mini nit : may","commit_id":"ce3f96f324b21f8ef3b3a4c02adc678411163a84"}],"neutron/plugins/ml2/drivers/linuxbridge/agent/linuxbridge_neutron_agent.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"03f0c2db37cf2761f5e5b5f806e61608575ecb30","unresolved":false,"context_lines":[{"line_number":487,"context_line":"                if bridge_lib.BridgeDevice(bridge_name).addif(tap_device_name):"},{"line_number":488,"context_line":"                    return False"},{"line_number":489,"context_line":"        else:"},{"line_number":490,"context_line":"            data \u003d {\u0027tap_device_name\u0027: tap_device_name,"},{"line_number":491,"context_line":"                    \u0027device_owner\u0027: device_owner,"},{"line_number":492,"context_line":"                    \u0027bridge_name\u0027: bridge_name}"},{"line_number":493,"context_line":"            LOG.debug(\"Skip adding device %(tap_device_name)s to \""},{"line_number":494,"context_line":"                      \"%(bridge_name)s. It is owned by %(device_owner)s and \""},{"line_number":495,"context_line":"                      \"thus added elsewhere.\", data)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_5be9d019","line":492,"range":{"start_line":490,"start_character":0,"end_line":492,"end_character":47},"updated":"2017-03-18 03:44:31.000000000","message":"nit: Isnt the data dictionary same as in line#483 ?\nYou can just add device_owner in it later on? Though I guess then the message would not be exactly in the prder you want","commit_id":"ce3f96f324b21f8ef3b3a4c02adc678411163a84"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"03f0c2db37cf2761f5e5b5f806e61608575ecb30","unresolved":false,"context_lines":[{"line_number":492,"context_line":"                    \u0027bridge_name\u0027: bridge_name}"},{"line_number":493,"context_line":"            LOG.debug(\"Skip adding device %(tap_device_name)s to \""},{"line_number":494,"context_line":"                      \"%(bridge_name)s. It is owned by %(device_owner)s and \""},{"line_number":495,"context_line":"                      \"thus added elsewhere.\", data)"},{"line_number":496,"context_line":"        return True"},{"line_number":497,"context_line":""},{"line_number":498,"context_line":"    def plug_interface(self, network_id, network_segment, tap_name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_9bf3480a","line":495,"range":{"start_line":495,"start_character":24,"end_line":495,"end_character":44},"updated":"2017-03-18 03:44:31.000000000","message":"Query : Do we need this line ??","commit_id":"ce3f96f324b21f8ef3b3a4c02adc678411163a84"}]}