)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"c6eabf129c7843b523203a801dd237b90945c580","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update port pair group failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Set port pair for port pair group failed because the type of \u0027existing\u0027 is error"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I198e745fccff76184d9813805d1c1915aff5a273"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf659307_c6c3dc56","line":9,"range":{"start_line":9,"start_character":75,"end_line":9,"end_character":80},"updated":"2018-04-09 14:09:37.000000000","message":"\"is not a list\"","commit_id":"d2d4323020658d5d19d7199dc42230a08850df65"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"08b0466912fe5074781402f95e8f5debb672c313","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update port pair group failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Set port pair for port pair group failed because the type of \u0027existing\u0027 is error"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I198e745fccff76184d9813805d1c1915aff5a273"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf659307_a00e9038","line":9,"updated":"2018-04-11 12:35:35.000000000","message":"I don\u0027t know why a searchlight bug is related.","commit_id":"d2d4323020658d5d19d7199dc42230a08850df65"},{"author":{"_account_id":26972,"name":"chenyaguang","email":"chenyaguang@szzt.com.cn","username":"Cyg"},"change_message_id":"587156125267baf35ad252b138cfba245c5aa990","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update port pair group failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Set port pair for port pair group failed because the type of \u0027existing\u0027 is error"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I198e745fccff76184d9813805d1c1915aff5a273"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf659307_9e9ab911","line":9,"in_reply_to":"bf659307_1e48e92a","updated":"2018-04-12 01:57:34.000000000","message":"Change to: https://bugs.launchpad.net/python-neutronclient/+bug/1762884","commit_id":"d2d4323020658d5d19d7199dc42230a08850df65"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"f14c1af76fd8fdba6bd4eb783a10f8172b823a50","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update port pair group failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Set port pair for port pair group failed because the type of \u0027existing\u0027 is error"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I198e745fccff76184d9813805d1c1915aff5a273"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf659307_792edf43","line":9,"in_reply_to":"bf659307_9e9ab911","updated":"2018-04-12 02:29:56.000000000","message":"Add a bug link to the commit message.","commit_id":"d2d4323020658d5d19d7199dc42230a08850df65"},{"author":{"_account_id":26972,"name":"chenyaguang","email":"chenyaguang@szzt.com.cn","username":"Cyg"},"change_message_id":"45d86f1feeb6bda8c353d7a6d2198ae6d94c58a3","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update port pair group failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Set port pair for port pair group failed because the type of \u0027existing\u0027 is error"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I198e745fccff76184d9813805d1c1915aff5a273"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf659307_1e48e92a","line":9,"in_reply_to":"bf659307_a00e9038","updated":"2018-04-12 01:32:58.000000000","message":"Sorry，I made a mistake. I will amend it as soon as possible","commit_id":"d2d4323020658d5d19d7199dc42230a08850df65"},{"author":{"_account_id":26972,"name":"chenyaguang","email":"chenyaguang@szzt.com.cn","username":"Cyg"},"change_message_id":"5cc25ad642488a9b5e16dd9b5fae4e0ff293e83c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update port pair group failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Set port pair for port pair group failed because the type of \u0027existing\u0027 is error"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I198e745fccff76184d9813805d1c1915aff5a273"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf659307_dcd37d37","line":9,"range":{"start_line":9,"start_character":75,"end_line":9,"end_character":80},"in_reply_to":"bf659307_c6c3dc56","updated":"2018-04-11 01:39:05.000000000","message":"Is that like this?https://bugs.launchpad.net/searchlight/+bug/1762884","commit_id":"d2d4323020658d5d19d7199dc42230a08850df65"},{"author":{"_account_id":26972,"name":"chenyaguang","email":"chenyaguang@szzt.com.cn","username":"Cyg"},"change_message_id":"5b7bb99cb899cfa5f949d2400fa61a00eb131823","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update port pair group failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Set port pair for port pair group failed because the type of \u0027existing\u0027 is error"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I198e745fccff76184d9813805d1c1915aff5a273"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf659307_7050740b","line":9,"range":{"start_line":9,"start_character":75,"end_line":9,"end_character":80},"in_reply_to":"bf659307_c6c3dc56","updated":"2018-04-10 01:10:07.000000000","message":"It is like \"[[]]\"","commit_id":"d2d4323020658d5d19d7199dc42230a08850df65"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"1750baa0cde91cc644e4157205b9b592b129db0c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Set or unset port pair group failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Set or unset port pair for port pair group failed"},{"line_number":10,"context_line":"because the type of \u0027existing\u0027 is error"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I198e745fccff76184d9813805d1c1915aff5a273"},{"line_number":13,"context_line":"Closes-Bug: #1762884"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f6a8fd7_0b9fa85f","line":10,"range":{"start_line":10,"start_character":34,"end_line":10,"end_character":39},"updated":"2018-04-18 08:01:25.000000000","message":"\u0027wrong\u0027 would be better here.","commit_id":"80104c44ea012bfd39864ea8b6c105b0c1f26109"},{"author":{"_account_id":26972,"name":"chenyaguang","email":"chenyaguang@szzt.com.cn","username":"Cyg"},"change_message_id":"62a2ca698c5f1fb29bf5677e0476a4c89bd46bd4","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Set or unset port pair group failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Set or unset port pair for port pair group failed"},{"line_number":10,"context_line":"because the type of \u0027existing\u0027 is error"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I198e745fccff76184d9813805d1c1915aff5a273"},{"line_number":13,"context_line":"Closes-Bug: #1762884"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f6a8fd7_2b738c4f","line":10,"range":{"start_line":10,"start_character":34,"end_line":10,"end_character":39},"in_reply_to":"9f6a8fd7_0b9fa85f","updated":"2018-04-18 08:03:00.000000000","message":"Thank you!","commit_id":"80104c44ea012bfd39864ea8b6c105b0c1f26109"}],"neutronclient/osc/v2/sfc/sfc_port_pair_group.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"abc448056f41c03280f8cdb5db37bc399849dc5d","unresolved":false,"context_lines":[{"line_number":188,"context_line":"            if parsed_args.no_port_pair:"},{"line_number":189,"context_line":"                existing \u003d []"},{"line_number":190,"context_line":"            else:"},{"line_number":191,"context_line":"                existing \u003d client.find_resource("},{"line_number":192,"context_line":"                    resource, parsed_args.port_pair_group,"},{"line_number":193,"context_line":"                    cmd_resource\u003d\u0027sfc_port_pair_group\u0027)[\u0027port_pairs\u0027]"},{"line_number":194,"context_line":"            attrs[\u0027port_pairs\u0027] \u003d sorted(list(set(existing) | set(added)))"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf659307_aed3cc00","line":191,"updated":"2018-04-12 04:07:26.000000000","message":"(Not directly to this patch) There is no need to call find_resource() on sfc_port_pair_group twice. L.179 does the first call. It is much better to store a retrieved port pair group object and reuse it here.","commit_id":"d2d4323020658d5d19d7199dc42230a08850df65"},{"author":{"_account_id":26972,"name":"chenyaguang","email":"chenyaguang@szzt.com.cn","username":"Cyg"},"change_message_id":"956dd1b1e38ff2ef68fdd38b3abfc3fdb846ff50","unresolved":false,"context_lines":[{"line_number":188,"context_line":"            if parsed_args.no_port_pair:"},{"line_number":189,"context_line":"                existing \u003d []"},{"line_number":190,"context_line":"            else:"},{"line_number":191,"context_line":"                existing \u003d client.find_resource("},{"line_number":192,"context_line":"                    resource, parsed_args.port_pair_group,"},{"line_number":193,"context_line":"                    cmd_resource\u003d\u0027sfc_port_pair_group\u0027)[\u0027port_pairs\u0027]"},{"line_number":194,"context_line":"            attrs[\u0027port_pairs\u0027] \u003d sorted(list(set(existing) | set(added)))"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f6a8fd7_88a5fe06","line":191,"in_reply_to":"bf659307_aed3cc00","updated":"2018-04-18 06:54:24.000000000","message":"I\u0027m sorry for the late reply. I think you are right.","commit_id":"d2d4323020658d5d19d7199dc42230a08850df65"}]}