)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34041,"name":"Diwei Zhu","email":"zhu.diw@northeastern.edu","username":"DiweiZhu"},"change_message_id":"22c0fb671b9164ae3dd458ee20416f63c8390d06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b20c6711_750bb180","updated":"2021-11-26 15:14:19.000000000","message":"Seems this one is forgot. Please have a look at it when you feel comfortable to. Thank you so much. ","commit_id":"d3a4176be0e439cfa094b56f64d22e8da58f4c2b"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"08022930bac8689f190cb70fb413a7658a7a02df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"33954e92_6c9f967c","updated":"2021-11-29 10:39:47.000000000","message":"You need a release note 😄 Other than that, this looks good to me. Nice work again","commit_id":"d3a4176be0e439cfa094b56f64d22e8da58f4c2b"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"193d962a93c5a90c0cb80df1db0be6fdb9702778","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8fadaca5_cb5d1557","updated":"2021-11-30 16:13:28.000000000","message":"This looks good now. Another great piece of work. Kudos","commit_id":"f82afc7f379daebd1994d9133eff801f790c0d32"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"a8f3ea7b073967d7a6d707b643e2590e0b783e75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"148f3ef2_cac7e824","updated":"2021-11-30 16:14:24.000000000","message":"great","commit_id":"f82afc7f379daebd1994d9133eff801f790c0d32"}],"openstackclient/compute/v2/server.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"08022930bac8689f190cb70fb413a7658a7a02df","unresolved":true,"context_lines":[{"line_number":3729,"context_line":"            if inf.net_id \u003d\u003d net_id:"},{"line_number":3730,"context_line":"                compute_client.delete_server_interface("},{"line_number":3731,"context_line":"                    inf.port_id,"},{"line_number":3732,"context_line":"                    server\u003dserver,"},{"line_number":3733,"context_line":"                )"},{"line_number":3734,"context_line":""},{"line_number":3735,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"25e5cfb3_53b2d2d3","line":3732,"updated":"2021-11-29 10:39:47.000000000","message":"I think you also want\n\n  ignore_missing\u003dTrue\n\nhere, right?","commit_id":"d3a4176be0e439cfa094b56f64d22e8da58f4c2b"},{"author":{"_account_id":34041,"name":"Diwei Zhu","email":"zhu.diw@northeastern.edu","username":"DiweiZhu"},"change_message_id":"85b666bd042d6a493a8deb6b66e0739ac375b415","unresolved":true,"context_lines":[{"line_number":3729,"context_line":"            if inf.net_id \u003d\u003d net_id:"},{"line_number":3730,"context_line":"                compute_client.delete_server_interface("},{"line_number":3731,"context_line":"                    inf.port_id,"},{"line_number":3732,"context_line":"                    server\u003dserver,"},{"line_number":3733,"context_line":"                )"},{"line_number":3734,"context_line":""},{"line_number":3735,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fd6c6342_c7d684d5","line":3732,"in_reply_to":"25e5cfb3_53b2d2d3","updated":"2021-11-29 14:33:37.000000000","message":"I think True is the default value. Is there some situation we need to explicitely add it as an argument?","commit_id":"d3a4176be0e439cfa094b56f64d22e8da58f4c2b"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"ebc2e2d9c5280a193de32e71807dd3af5c27ab00","unresolved":false,"context_lines":[{"line_number":3729,"context_line":"            if inf.net_id \u003d\u003d net_id:"},{"line_number":3730,"context_line":"                compute_client.delete_server_interface("},{"line_number":3731,"context_line":"                    inf.port_id,"},{"line_number":3732,"context_line":"                    server\u003dserver,"},{"line_number":3733,"context_line":"                )"},{"line_number":3734,"context_line":""},{"line_number":3735,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"a07c2d05_206020ef","line":3732,"in_reply_to":"fd6c6342_c7d684d5","updated":"2021-11-30 16:12:04.000000000","message":"Apologies. You\u0027re correct. This is the default. There\u0027s no need to provide it explicitly, no","commit_id":"d3a4176be0e439cfa094b56f64d22e8da58f4c2b"}],"openstackclient/tests/functional/compute/v2/test_server.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"08022930bac8689f190cb70fb413a7658a7a02df","unresolved":true,"context_lines":[{"line_number":1107,"context_line":""},{"line_number":1108,"context_line":"        # remove network and check \u0027public\u0027 is not in server show"},{"line_number":1109,"context_line":"        self.openstack("},{"line_number":1110,"context_line":"            \u0027server remove network \u0027 + name + \u0027 public\u0027)"},{"line_number":1111,"context_line":""},{"line_number":1112,"context_line":"        wait_time \u003d 0"},{"line_number":1113,"context_line":"        while wait_time \u003c 60:"}],"source_content_type":"text/x-python","patch_set":2,"id":"805ec87c_398d3d35","line":1110,"updated":"2021-11-29 10:39:47.000000000","message":"nit: This would all fit on one line, I think?","commit_id":"d3a4176be0e439cfa094b56f64d22e8da58f4c2b"},{"author":{"_account_id":34041,"name":"Diwei Zhu","email":"zhu.diw@northeastern.edu","username":"DiweiZhu"},"change_message_id":"85b666bd042d6a493a8deb6b66e0739ac375b415","unresolved":false,"context_lines":[{"line_number":1107,"context_line":""},{"line_number":1108,"context_line":"        # remove network and check \u0027public\u0027 is not in server show"},{"line_number":1109,"context_line":"        self.openstack("},{"line_number":1110,"context_line":"            \u0027server remove network \u0027 + name + \u0027 public\u0027)"},{"line_number":1111,"context_line":""},{"line_number":1112,"context_line":"        wait_time \u003d 0"},{"line_number":1113,"context_line":"        while wait_time \u003c 60:"}],"source_content_type":"text/x-python","patch_set":2,"id":"c09f35fc_d8a15b1d","line":1110,"in_reply_to":"805ec87c_398d3d35","updated":"2021-11-29 14:33:37.000000000","message":"Done","commit_id":"d3a4176be0e439cfa094b56f64d22e8da58f4c2b"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"08022930bac8689f190cb70fb413a7658a7a02df","unresolved":true,"context_lines":[{"line_number":1125,"context_line":"        addresses \u003d cmd_output[\u0027addresses\u0027]"},{"line_number":1126,"context_line":"        self.assertNotIn(\u0027public\u0027, addresses)"},{"line_number":1127,"context_line":""},{"line_number":1128,"context_line":"        # create port, record one of its ip address"},{"line_number":1129,"context_line":"        port_name \u003d \u0027test-port\u0027"},{"line_number":1130,"context_line":""},{"line_number":1131,"context_line":"        cmd_output \u003d json.loads(self.openstack("}],"source_content_type":"text/x-python","patch_set":2,"id":"139d6fd3_822fea6e","line":1128,"updated":"2021-11-29 10:39:47.000000000","message":"nit: It might be better to split this into two tests? Less chance of them interfering with each other","commit_id":"d3a4176be0e439cfa094b56f64d22e8da58f4c2b"},{"author":{"_account_id":34041,"name":"Diwei Zhu","email":"zhu.diw@northeastern.edu","username":"DiweiZhu"},"change_message_id":"85b666bd042d6a493a8deb6b66e0739ac375b415","unresolved":false,"context_lines":[{"line_number":1125,"context_line":"        addresses \u003d cmd_output[\u0027addresses\u0027]"},{"line_number":1126,"context_line":"        self.assertNotIn(\u0027public\u0027, addresses)"},{"line_number":1127,"context_line":""},{"line_number":1128,"context_line":"        # create port, record one of its ip address"},{"line_number":1129,"context_line":"        port_name \u003d \u0027test-port\u0027"},{"line_number":1130,"context_line":""},{"line_number":1131,"context_line":"        cmd_output \u003d json.loads(self.openstack("}],"source_content_type":"text/x-python","patch_set":2,"id":"7291f21b_b359e524","line":1128,"in_reply_to":"139d6fd3_822fea6e","updated":"2021-11-29 14:33:37.000000000","message":"Done","commit_id":"d3a4176be0e439cfa094b56f64d22e8da58f4c2b"}]}