)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},"change_message_id":"1faa0a1e651bd10b8fcd600a877f27bc993f886d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0d5c52ec_9f7a38cd","updated":"2023-08-02 07:07:35.000000000","message":"Can you continue remain this patch?","commit_id":"81978b36c8767ed796c6891d9cee71b452fd1054"},{"author":{"_account_id":12932,"name":"Andrew Bogott","email":"andrewbogott@gmail.com","username":"Andrew"},"change_message_id":"ec64c9db4cbf1a162350a5c1055b3b39d01588a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"08f1e757_cec3a01e","updated":"2023-01-10 14:40:45.000000000","message":"I\u0027m adding all the reviewers who looked at the patch I\u0027m trying to correct here. I\u0027m very happy to try an alternative approach.","commit_id":"81978b36c8767ed796c6891d9cee71b452fd1054"},{"author":{"_account_id":12932,"name":"Andrew Bogott","email":"andrewbogott@gmail.com","username":"Andrew"},"change_message_id":"7dce9f1494d2c7069c1a59dd0057e31de9518d35","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1bc1fc07_16c6b28d","updated":"2023-01-10 14:19:54.000000000","message":"recheck","commit_id":"81978b36c8767ed796c6891d9cee71b452fd1054"},{"author":{"_account_id":12932,"name":"Andrew Bogott","email":"andrewbogott@gmail.com","username":"Andrew"},"change_message_id":"60c9317b7079ea554ec598664523a69932df4270","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"80c799ad_e84ee146","updated":"2023-01-20 14:12:46.000000000","message":"recheck","commit_id":"81978b36c8767ed796c6891d9cee71b452fd1054"},{"author":{"_account_id":12932,"name":"Andrew Bogott","email":"andrewbogott@gmail.com","username":"Andrew"},"change_message_id":"3552caf04732b697f70b8cb99f221ef6da0a0363","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8c75864f_fac6e7c6","updated":"2023-01-11 14:21:31.000000000","message":"recheck","commit_id":"81978b36c8767ed796c6891d9cee71b452fd1054"},{"author":{"_account_id":12932,"name":"Andrew Bogott","email":"andrewbogott@gmail.com","username":"Andrew"},"change_message_id":"a93c31e7e9e382655f89e6ac0a295f83826742c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e424a9ad_0f3adb95","in_reply_to":"0d5c52ec_9f7a38cd","updated":"2023-08-31 18:04:51.000000000","message":"Bo, I\u0027m not sure I understand your comment but I have rebased and resolved the conflict.","commit_id":"81978b36c8767ed796c6891d9cee71b452fd1054"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"a6667c86e5c88d94da12f6a87c0347b7060cfac3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"651e977b_adf9c29b","in_reply_to":"8c75864f_fac6e7c6","updated":"2023-01-12 01:21:15.000000000","message":"Hi, I have tried to  fix the CI, but it still not work.  this seems caused by the tox 4.2.6, the new tox  doesn\u0027t install trove package in virtual venv, I am digging into it now . thanks.","commit_id":"81978b36c8767ed796c6891d9cee71b452fd1054"},{"author":{"_account_id":12932,"name":"Andrew Bogott","email":"andrewbogott@gmail.com","username":"Andrew"},"change_message_id":"32c15f59b2008fed39c3770201a9c2876deec7bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a3880fca_dd956a6f","updated":"2023-11-29 21:50:54.000000000","message":"*bump* this patch is still needed","commit_id":"362f510aa635fc6c4a21233f41d6359455991f5c"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"869c4afae19cd208abbc54b069bf342eb4f2b6f1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9a72ea50_c3e0469d","updated":"2023-11-30 02:03:50.000000000","message":"ok, i will review it soon.","commit_id":"362f510aa635fc6c4a21233f41d6359455991f5c"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"e4b621730b97f3dd4242bee61a91476c0ebcb1f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"08036708_5e0e2575","updated":"2023-12-21 07:23:14.000000000","message":"recheck","commit_id":"237e977135d80580fd36078a0a5b7fb02478b906"}],"trove/instance/models.py":[{"author":{"_account_id":31737,"name":"Hirotaka Wakabayashi","email":"hiwkby@yahoo.com","username":"hiwkby"},"change_message_id":"32385cb461ee1a4d07b5fa8f9a36a8c293af900f","unresolved":true,"context_lines":[{"line_number":141,"context_line":""},{"line_number":142,"context_line":"def load_simple_instance_addresses(context, db_info):"},{"line_number":143,"context_line":"    \"\"\"Get addresses of the instance from Neutron.\"\"\""},{"line_number":144,"context_line":"    if \u0027BUILDING\u0027 \u003d\u003d db_info.task_status.action and not db_info.cluster_id:"},{"line_number":145,"context_line":"        db_info.addresses \u003d []"},{"line_number":146,"context_line":"        return"},{"line_number":147,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1110cf3f_a3744c26","side":"PARENT","line":144,"range":{"start_line":144,"start_character":0,"end_line":144,"end_character":6},"updated":"2023-01-24 15:36:41.000000000","message":"Hello Andrew, Thanks for the patch! I think this logic is required for the instances \nbelong to a cluster because the task_status of creating a cluster is determined\nafter initializing a cluster.","commit_id":"1d8231d31ffde24387fcb63faa55689bbb81ad36"},{"author":{"_account_id":12932,"name":"Andrew Bogott","email":"andrewbogott@gmail.com","username":"Andrew"},"change_message_id":"337c8d7b2c67b08e0682b5c7922c84346b0cb098","unresolved":true,"context_lines":[{"line_number":141,"context_line":""},{"line_number":142,"context_line":"def load_simple_instance_addresses(context, db_info):"},{"line_number":143,"context_line":"    \"\"\"Get addresses of the instance from Neutron.\"\"\""},{"line_number":144,"context_line":"    if \u0027BUILDING\u0027 \u003d\u003d db_info.task_status.action and not db_info.cluster_id:"},{"line_number":145,"context_line":"        db_info.addresses \u003d []"},{"line_number":146,"context_line":"        return"},{"line_number":147,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"ef173969_e22b101a","side":"PARENT","line":144,"range":{"start_line":144,"start_character":0,"end_line":144,"end_character":6},"in_reply_to":"1110cf3f_a3744c26","updated":"2023-05-26 02:26:58.000000000","message":"Hello, and sorry for the delay in responding to this. I\u0027m not sure I understand your comment... this address is needed for DNS setup which happens while the instance is still in the \u0027building\u0027 state. My tests confirm that leaving this section in breaks DNS. Does removing it break something else?","commit_id":"1d8231d31ffde24387fcb63faa55689bbb81ad36"},{"author":{"_account_id":31737,"name":"Hirotaka Wakabayashi","email":"hiwkby@yahoo.com","username":"hiwkby"},"change_message_id":"1da2b6cd4bb2b677c8ac3fb5e5807f657b624a4a","unresolved":true,"context_lines":[{"line_number":141,"context_line":""},{"line_number":142,"context_line":"def load_simple_instance_addresses(context, db_info):"},{"line_number":143,"context_line":"    \"\"\"Get addresses of the instance from Neutron.\"\"\""},{"line_number":144,"context_line":"    if \u0027BUILDING\u0027 \u003d\u003d db_info.task_status.action and not db_info.cluster_id:"},{"line_number":145,"context_line":"        db_info.addresses \u003d []"},{"line_number":146,"context_line":"        return"},{"line_number":147,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"f6b9efef_4bc4a419","side":"PARENT","line":144,"range":{"start_line":144,"start_character":0,"end_line":144,"end_character":6},"in_reply_to":"ef173969_e22b101a","updated":"2023-08-31 22:55:22.000000000","message":"Sorry for the delay. I think this condition is needed for database cluster function. I will test this patch later.","commit_id":"1d8231d31ffde24387fcb63faa55689bbb81ad36"}]}