)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11682,"name":"Ryan Moats","email":"rmoats@us.ibm.com","username":"regXboi"},"change_message_id":"8aa4894bcbf47fb7189586d44d731bf5268a22bb","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Yaguo Zhou \u003czhouyaguo@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-04-06 08:34:08 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix doc typo in SecurityGroupAgentRpcCallbackMixin"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Iee7719cf4dcb5e99a9f86f9b111ada0d573cd9c2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a061dce_7b76e9d7","line":7,"updated":"2016-04-07 02:18:19.000000000","message":"nit for a future respin: could use the TrivialFix tag here as well","commit_id":"d303065c6f1740b3d3bd846d31d8296e90846af5"},{"author":{"_account_id":20037,"name":"Petrônio Carlos Bezerra","email":"petroniocg@lsd.ufcg.edu.br","username":"petroniocg"},"change_message_id":"785cbbd0eff61e92b0892bc189c1e98554d3ef51","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Yaguo Zhou \u003czhouyaguo@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-04-06 08:34:08 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix doc typo in SecurityGroupAgentRpcCallbackMixin"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Iee7719cf4dcb5e99a9f86f9b111ada0d573cd9c2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a061dce_72698c25","line":7,"in_reply_to":"9a061dce_7b76e9d7","updated":"2016-04-08 13:02:59.000000000","message":"Done","commit_id":"d303065c6f1740b3d3bd846d31d8296e90846af5"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"258de631ecbba051aa4366cded44499204f1d8b4","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-04-26 21:06:43 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix doc typo in SecurityGroupAgentRpcCallbackMixin"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"TrivialFix"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Iee7719cf4dcb5e99a9f86f9b111ada0d573cd9c2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3ac371cc_cd298f39","line":8,"updated":"2016-08-16 09:28:01.000000000","message":"Please explain in the commit message why it\u0027s not the correct class. It\u0027s not obvious otherwise.","commit_id":"752f1cdb31a52dddd8025b7c27db506fc3f8d603"}],"neutron/api/rpc/handlers/securitygroups_rpc.py":[{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"44a5b0b67a6631b07ba97498c3051215f048bd92","unresolved":false,"context_lines":[{"line_number":176,"context_line":"    \"\"\"A mix-in that enable SecurityGroup support in agent implementations."},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"    This class implements the server side of an rpc interface.  The client side"},{"line_number":179,"context_line":"    can be found in SecurityGroupAgentRpcApiMixin. For more information on changing"},{"line_number":180,"context_line":"    rpc interfaces, see doc/source/devref/rpc_api.rst."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"    The sg_agent reference implementation is available in neutron/agent"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a061dce_3dcb20a7","line":179,"range":{"start_line":179,"start_character":20,"end_line":179,"end_character":49},"updated":"2016-04-05 15:30:55.000000000","message":"It\u0027s not a typo.\nPlease refer to:\n class SecurityGroupServerRpcApi(object):\n     \"\"\"RPC client for security group methods in the plugin.\n \n     This class implements the client side of an rpc interface.","commit_id":"1f3077677229b0a4c8d34849b618cf982159ffd7"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"ca0114af9dd93a10dde3955ac310b56debdcf686","unresolved":false,"context_lines":[{"line_number":176,"context_line":"    \"\"\"A mix-in that enable SecurityGroup support in agent implementations."},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"    This class implements the server side of an rpc interface.  The client side"},{"line_number":179,"context_line":"    can be found in SecurityGroupAgentRpcApiMixin. For more information on changing"},{"line_number":180,"context_line":"    rpc interfaces, see doc/source/devref/rpc_api.rst."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"    The sg_agent reference implementation is available in neutron/agent"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a061dce_21fbf6c2","line":179,"range":{"start_line":179,"start_character":75,"end_line":179,"end_character":83},"updated":"2016-04-05 17:53:42.000000000","message":"This line is too long.","commit_id":"1f3077677229b0a4c8d34849b618cf982159ffd7"},{"author":{"_account_id":17549,"name":"Yaguo Zhou","email":"zhouyaguo@unionpay.com","username":"zhouyaguo"},"change_message_id":"1ec58739d38f1aad53c0b49f6ce7c9467462bbc5","unresolved":false,"context_lines":[{"line_number":176,"context_line":"    \"\"\"A mix-in that enable SecurityGroup support in agent implementations."},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"    This class implements the server side of an rpc interface.  The client side"},{"line_number":179,"context_line":"    can be found in SecurityGroupAgentRpcApiMixin. For more information on changing"},{"line_number":180,"context_line":"    rpc interfaces, see doc/source/devref/rpc_api.rst."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"    The sg_agent reference implementation is available in neutron/agent"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a061dce_e9e39803","line":179,"range":{"start_line":179,"start_character":75,"end_line":179,"end_character":83},"in_reply_to":"9a061dce_21fbf6c2","updated":"2016-04-05 23:17:02.000000000","message":"Thanks for ur review and sorry for the pep8 error","commit_id":"1f3077677229b0a4c8d34849b618cf982159ffd7"},{"author":{"_account_id":17549,"name":"Yaguo Zhou","email":"zhouyaguo@unionpay.com","username":"zhouyaguo"},"change_message_id":"55c4ba64622c6efe447e147b02637e50c649ed7d","unresolved":false,"context_lines":[{"line_number":176,"context_line":"    \"\"\"A mix-in that enable SecurityGroup support in agent implementations."},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"    This class implements the server side of an rpc interface.  The client side"},{"line_number":179,"context_line":"    can be found in SecurityGroupAgentRpcApiMixin. For more information on changing"},{"line_number":180,"context_line":"    rpc interfaces, see doc/source/devref/rpc_api.rst."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"    The sg_agent reference implementation is available in neutron/agent"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a061dce_b0c9177b","line":179,"range":{"start_line":179,"start_character":20,"end_line":179,"end_character":49},"in_reply_to":"9a061dce_3dcb20a7","updated":"2016-04-05 15:49:51.000000000","message":"Thx, but i don\u0027t think so. \nSecurityGroupServerRpcApi and SecurityGroupServerRpcCallback are a pair, SecurityGroupAgentRpcApiMixin and SecurityGroupAgentRpcCallbackMixin should be a pair.","commit_id":"1f3077677229b0a4c8d34849b618cf982159ffd7"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"0261c3d1e0f1d360860df94f750b400f62fe8bc0","unresolved":false,"context_lines":[{"line_number":176,"context_line":"    \"\"\"A mix-in that enable SecurityGroup support in agent implementations."},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"    This class implements the server side of an rpc interface.  The client side"},{"line_number":179,"context_line":"    can be found in SecurityGroupAgentRpcApiMixin. For more information on changing"},{"line_number":180,"context_line":"    rpc interfaces, see doc/source/devref/rpc_api.rst."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"    The sg_agent reference implementation is available in neutron/agent"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a061dce_3e55d11c","line":179,"range":{"start_line":179,"start_character":20,"end_line":179,"end_character":49},"in_reply_to":"9a061dce_b0c9177b","updated":"2016-04-05 17:47:04.000000000","message":"True, you\u0027re right.\nI\u0027ve misread it and pasted wrong section. Here is extra info, that probably you\u0027re referring.:\n 122: agent side. The other side of this interface can be found in\n 123: SecurityGroupAgentRpcCallbackMixin.","commit_id":"1f3077677229b0a4c8d34849b618cf982159ffd7"}]}