)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"668cd4564bc1dc4591d5b5221ba3268cbea57eff","unresolved":false,"context_lines":[{"line_number":17,"context_line":"This leaves default behavior as long as targeted plugin"},{"line_number":18,"context_line":"derives from the common base class."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Fixes Bug #1211513"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I53b17d3cbb1a67b21330c5e0da8d5240ac010f3a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":33,"id":"AAAATn%2F975U%3D","line":20,"updated":"2013-12-04 09:28:21.000000000","message":"can you please get rid of the # above. just\n\nCloses-Bug: 1211513\n\nmeans that gerrit will render the bug id as a uri to the bug report\n\nthanks!","commit_id":"94f8dc0443ca5644fc2937b6603674076d71c609"},{"author":{"_account_id":7448,"name":"Carl Baldwin","email":"carl@ecbaldwin.net","username":"carl-baldwin"},"change_message_id":"1a6af2862af6a749e5309c22200cc1644ac737c8","unresolved":false,"context_lines":[{"line_number":17,"context_line":"This leaves default behavior as long as targeted plugin"},{"line_number":18,"context_line":"derives from the common base class."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Fixes Bug #1211513"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I53b17d3cbb1a67b21330c5e0da8d5240ac010f3a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":33,"id":"AAAATn%2F9IVM%3D","line":20,"in_reply_to":"AAAATn%2F975U%3D","updated":"2013-12-23 17:15:12.000000000","message":"The \u0027#\u0027 is fine.  See this link for details about the standard format for this line:  https://wiki.openstack.org/wiki/Gerrit_Workflow#Committing_Changes","commit_id":"94f8dc0443ca5644fc2937b6603674076d71c609"}],"neutron/tests/unit/test_extensions.py":[{"author":{"_account_id":7448,"name":"Carl Baldwin","email":"carl@ecbaldwin.net","username":"carl-baldwin"},"change_message_id":"1a6af2862af6a749e5309c22200cc1644ac737c8","unresolved":false,"context_lines":[{"line_number":34,"context_line":"import neutron.tests.unit.extensions"},{"line_number":35,"context_line":"from neutron.tests.unit import testlib_api"},{"line_number":36,"context_line":"from neutron import wsgi"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"ROOTDIR \u003d os.path.dirname(os.path.dirname(__file__))"}],"source_content_type":"text/x-python","patch_set":33,"id":"AAAATn%2F8pwQ%3D","line":37,"updated":"2013-12-23 17:15:12.000000000","message":"Nit:  Try not to change whitespace without reason.  Most files have two blank lines after the imports section.  No reason to change this.","commit_id":"94f8dc0443ca5644fc2937b6603674076d71c609"}],"neutron/tests/unit/test_quota_ext.py":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"363f5b0cb9ba89e2397b701d22131ccd37900c8e","unresolved":false,"context_lines":[{"line_number":66,"context_line":"            \u0027quota_items\u0027,"},{"line_number":67,"context_line":"            [\u0027network\u0027, \u0027subnet\u0027, \u0027port\u0027, \u0027extra1\u0027],"},{"line_number":68,"context_line":"            group\u003d\u0027QUOTAS\u0027)"},{"line_number":69,"context_line":"        #quota.QUOTAS \u003d quota.QuotaEngine()"},{"line_number":70,"context_line":"        quota.register_resources_from_config()"},{"line_number":71,"context_line":"        self._plugin_patcher \u003d mock.patch(TARGET_PLUGIN, autospec\u003dTrue)"},{"line_number":72,"context_line":"        self.plugin \u003d self._plugin_patcher.start()"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAASn%2F%2F3EQ%3D","line":69,"updated":"2013-08-28 18:39:35.000000000","message":"not sure I understand why you commented this out; as for the other fixes, are there really part of this patch?","commit_id":"09dc247adfbc9b85a14af57591ce7263b1374e28"},{"author":{"_account_id":6522,"name":"Sean McCully","email":"sean_mccully@yahoo.com","username":"sean-mccully"},"change_message_id":"f2b5578a6a3b2178aa63fd2807232ae4a83d7f64","unresolved":false,"context_lines":[{"line_number":66,"context_line":"            \u0027quota_items\u0027,"},{"line_number":67,"context_line":"            [\u0027network\u0027, \u0027subnet\u0027, \u0027port\u0027, \u0027extra1\u0027],"},{"line_number":68,"context_line":"            group\u003d\u0027QUOTAS\u0027)"},{"line_number":69,"context_line":"        #quota.QUOTAS \u003d quota.QuotaEngine()"},{"line_number":70,"context_line":"        quota.register_resources_from_config()"},{"line_number":71,"context_line":"        self._plugin_patcher \u003d mock.patch(TARGET_PLUGIN, autospec\u003dTrue)"},{"line_number":72,"context_line":"        self.plugin \u003d self._plugin_patcher.start()"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAASn%2F%2F3CA%3D","line":69,"in_reply_to":"AAAASn%2F%2F3EQ%3D","updated":"2013-08-28 18:44:45.000000000","message":"This was breaking the tests, probably because the import chain changed. So yes","commit_id":"09dc247adfbc9b85a14af57591ce7263b1374e28"}]}