)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"change_message_id":"7c138241b87d2f9ba7bf8373c766b93330426753","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Apply security group when attach network"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fix bug: when attach new network on a container which has add"},{"line_number":10,"context_line":"a security group before, security group doesn\u0027t apply"},{"line_number":11,"context_line":"on new port."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df87a7cf_1e3f1cb6","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":9},"updated":"2017-12-13 06:00:06.000000000","message":"nit: It\u0027s unnecessary.","commit_id":"8021f8b24b62e6343d3eb620573286f496981528"},{"author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"change_message_id":"7c138241b87d2f9ba7bf8373c766b93330426753","unresolved":false,"context_lines":[{"line_number":11,"context_line":"on new port."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I091a05078001c7e649afdaf21505e1ed2614da6b"},{"line_number":14,"context_line":"Closes-bug: bug/527369"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df87a7cf_5e55a4f9","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":22},"updated":"2017-12-13 06:00:06.000000000","message":"Please correct this.","commit_id":"8021f8b24b62e6343d3eb620573286f496981528"}],"zun/tests/unit/container/docker/test_docker_driver.py":[{"author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"change_message_id":"44fdf1e15074af72a8dbaac59b5007c3235d5410","unresolved":false,"context_lines":[{"line_number":593,"context_line":"    @mock.patch(\u0027zun.common.utils.get_security_group_ids\u0027)"},{"line_number":594,"context_line":"    @mock.patch(\u0027zun.network.kuryr_network.KuryrNetwork\u0027"},{"line_number":595,"context_line":"                \u0027.connect_container_to_network\u0027)"},{"line_number":596,"context_line":"    @mock.patch(\u0027zun.network.kuryr_network.KuryrNetwork\u0027"},{"line_number":597,"context_line":"                \u0027.disconnect_container_from_network\u0027)"},{"line_number":598,"context_line":"    @mock.patch(\u0027zun.network.kuryr_network.KuryrNetwork\u0027"},{"line_number":599,"context_line":"                \u0027.list_networks\u0027)"},{"line_number":600,"context_line":"    def test_network_attach_with_security_group(self, mock_list,"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_126e43c8","line":597,"range":{"start_line":596,"start_character":4,"end_line":597,"end_character":53},"updated":"2017-12-12 15:13:18.000000000","message":"It\u0027s unnecessary mock here, not see we calls disconnect_container_from_network in the code.","commit_id":"0ec9e329118a58541faf373fa5572fbcc9de5118"},{"author":{"_account_id":17812,"name":"chengyang","email":"374519141@qq.com","username":"chengyang"},"change_message_id":"ddde4c9399743da5e93ae84bc3c4dc4454341961","unresolved":false,"context_lines":[{"line_number":593,"context_line":"    @mock.patch(\u0027zun.common.utils.get_security_group_ids\u0027)"},{"line_number":594,"context_line":"    @mock.patch(\u0027zun.network.kuryr_network.KuryrNetwork\u0027"},{"line_number":595,"context_line":"                \u0027.connect_container_to_network\u0027)"},{"line_number":596,"context_line":"    @mock.patch(\u0027zun.network.kuryr_network.KuryrNetwork\u0027"},{"line_number":597,"context_line":"                \u0027.disconnect_container_from_network\u0027)"},{"line_number":598,"context_line":"    @mock.patch(\u0027zun.network.kuryr_network.KuryrNetwork\u0027"},{"line_number":599,"context_line":"                \u0027.list_networks\u0027)"},{"line_number":600,"context_line":"    def test_network_attach_with_security_group(self, mock_list,"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_6acf86b0","line":597,"range":{"start_line":596,"start_character":4,"end_line":597,"end_character":53},"in_reply_to":"df87a7cf_126e43c8","updated":"2017-12-13 02:34:28.000000000","message":"\u003e It\u0027s unnecessary mock here, not see we calls disconnect_container_from_network\n \u003e in the code.\n\ngood catch. done","commit_id":"0ec9e329118a58541faf373fa5572fbcc9de5118"}]}