)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":27078,"name":"Manik Bindlish","email":"manikbindlish19@gmail.com","username":"manik"},"change_message_id":"61061db50f6846f28282bac6cb0f5c4e61f96c05","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     huanhongda \u003chongda.xun@easystack.cn\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-11-05 16:30:48 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Delete server before removing its host from aggregate"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In change Icd227b807a78bd37a77fa547880780ff9edad1e8 we don\u0027t"},{"line_number":10,"context_line":"allow to remove host from aggregate if the host has instances"},{"line_number":11,"context_line":"on it."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I19861b2832f140eac0ad5397091a6333e2f40618"},{"line_number":14,"context_line":"Related-Bug: #1782539"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_01836950","line":11,"range":{"start_line":7,"start_character":0,"end_line":11,"end_character":6},"updated":"2018-11-28 09:14:20.000000000","message":"I think \"Depends-On: \" should be mention in commit message as it depend\u0027s on nova\u0027 patch","commit_id":"cf96f557d9dbac9e902677d3414584d4db373516"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9d51590c2b1f99956fb660c085ce1c0d268bb288","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     huanhongda \u003chongda.xun@easystack.cn\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-11-05 16:30:48 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Delete server before removing its host from aggregate"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In change Icd227b807a78bd37a77fa547880780ff9edad1e8 we don\u0027t"},{"line_number":10,"context_line":"allow to remove host from aggregate if the host has instances"},{"line_number":11,"context_line":"on it."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I19861b2832f140eac0ad5397091a6333e2f40618"},{"line_number":14,"context_line":"Related-Bug: #1782539"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9fdfeff1_771bd272","line":11,"range":{"start_line":7,"start_character":0,"end_line":11,"end_character":6},"in_reply_to":"3f79a3b5_01836950","updated":"2019-02-06 19:59:53.000000000","message":"That would create a circular dependency, which you can\u0027t do.","commit_id":"cf96f557d9dbac9e902677d3414584d4db373516"}],"tempest/api/compute/admin/test_aggregates.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"f1a5fcd3a6f02d1882b7e64a51f02363f74059ed","unresolved":false,"context_lines":[{"line_number":226,"context_line":""},{"line_number":227,"context_line":"        self.client.add_host(aggregate[\u0027id\u0027], host\u003dhost)"},{"line_number":228,"context_line":"        self.addCleanup(self.client.remove_host, aggregate[\u0027id\u0027], host\u003dhost)"},{"line_number":229,"context_line":"        server \u003d self.create_test_server(availability_zone\u003daz_name,"},{"line_number":230,"context_line":"                                         wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":231,"context_line":"        server_host \u003d self.get_host_for_server(server[\u0027id\u0027])"},{"line_number":232,"context_line":"        self.assertEqual(host, server_host)"},{"line_number":233,"context_line":"        self.delete_server(server[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_7d87e4b4","line":230,"range":{"start_line":229,"start_character":0,"end_line":230,"end_character":61},"updated":"2018-11-05 05:43:27.000000000","message":"Better to add \"self.addCleanup(self.delete_server, server[\u0027id\u0027])\" after this, and, better to add a comment for the reason.\nBesides, we will wait to see the review opinion on corresponding nova patch.","commit_id":"4abe3f8dcff4048eeabbdca388340dfed6cabc2f"},{"author":{"_account_id":26286,"name":"huanhongda","email":"hongda.xun@easystack.cn","username":"huanhongda"},"change_message_id":"2ae97a322cf66820322a3c141fca05b7cb13fec2","unresolved":false,"context_lines":[{"line_number":226,"context_line":""},{"line_number":227,"context_line":"        self.client.add_host(aggregate[\u0027id\u0027], host\u003dhost)"},{"line_number":228,"context_line":"        self.addCleanup(self.client.remove_host, aggregate[\u0027id\u0027], host\u003dhost)"},{"line_number":229,"context_line":"        server \u003d self.create_test_server(availability_zone\u003daz_name,"},{"line_number":230,"context_line":"                                         wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":231,"context_line":"        server_host \u003d self.get_host_for_server(server[\u0027id\u0027])"},{"line_number":232,"context_line":"        self.assertEqual(host, server_host)"},{"line_number":233,"context_line":"        self.delete_server(server[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_db19608b","line":230,"range":{"start_line":229,"start_character":0,"end_line":230,"end_character":61},"in_reply_to":"3f79a3b5_7d87e4b4","updated":"2018-11-05 08:34:46.000000000","message":"Done","commit_id":"4abe3f8dcff4048eeabbdca388340dfed6cabc2f"}]}