)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2b94781c7e4afd118d3aee3f483ce011bf6a573e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2767670c_f9f7a32d","updated":"2023-04-17 13:57:13.000000000","message":"I didn\u0027t check the CI: py3 is failing","commit_id":"bfe77059a92f04e55bb2dec740bc0e567b74855a"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"2a1b48dce9f5d4e5ca4cd375d4f333e273729ca2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c63ddd34_d1b40d7a","updated":"2023-04-19 08:48:16.000000000","message":"Wait for the test to be completed...","commit_id":"2974523ec70c8cbbe7860be8d6f0030d81585d08"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"dcb053a5abfe36055dbeea2620afc1b242046eb7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e07a8534_ecdaa673","updated":"2023-04-18 11:58:06.000000000","message":"recheck","commit_id":"2974523ec70c8cbbe7860be8d6f0030d81585d08"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"00a1bfdc022becfa9744422be171611e4adefc6c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1126a021_4689c72e","updated":"2023-04-19 09:54:57.000000000","message":"Already tested.","commit_id":"2490db67d5a74407f2bf1f4b58b3e16788d71797"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"a8c307db14bf77e127e6da42cd3c0baf8c527a33","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"28a452da_3f978d1b","updated":"2023-04-20 07:01:22.000000000","message":"I have some doubts on request/response bodies, let\u0027s discuss","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"8a0bbb2471034e2538add2a8ce644e0213e3cfcd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"6509a5c6_f692d40d","updated":"2023-04-21 06:02:39.000000000","message":"recheck","commit_id":"872807640227193bf3bfb4800b2253ea975538e9"}],"api-ref/source/v2/samples/ports/port-add-allowed-address-pairs-request.json":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"59364f4a310fb22a5ed93dbf904db5fc49e805f2","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"   \"port\" : {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","}],"source_content_type":"application/json","patch_set":7,"id":"cd4fc760_769bbeaf","line":2,"range":{"start_line":2,"start_character":11,"end_line":2,"end_character":13},"updated":"2023-04-19 09:45:32.000000000","message":"Do we need the port ID here?","commit_id":"2490db67d5a74407f2bf1f4b58b3e16788d71797"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"0c1b193f3b5093236e43bca9dd824660b01adffe","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"   \"port\" : {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","}],"source_content_type":"application/json","patch_set":7,"id":"e638a073_ba3ca72b","line":2,"range":{"start_line":2,"start_character":11,"end_line":2,"end_character":13},"in_reply_to":"cd4fc760_769bbeaf","updated":"2023-04-19 11:42:14.000000000","message":"Done","commit_id":"2490db67d5a74407f2bf1f4b58b3e16788d71797"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"a8c307db14bf77e127e6da42cd3c0baf8c527a33","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"   \"port\" : {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","}],"source_content_type":"application/json","patch_set":8,"id":"3d6ddc95_3e7e97dd","line":2,"range":{"start_line":2,"start_character":3,"end_line":2,"end_character":13},"updated":"2023-04-20 07:01:22.000000000","message":"so if it\u0027s a Port action why need `port` and not just `allowed_address_pairs` in body?","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ce1b1ddbe929fedc0c5a2488271816738a81f5d5","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"   \"port\" : {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","}],"source_content_type":"application/json","patch_set":8,"id":"a965238f_dbe69557","line":2,"range":{"start_line":2,"start_character":3,"end_line":2,"end_character":13},"in_reply_to":"1b32dba5_a681e987","updated":"2023-04-20 08:57:25.000000000","message":"Because all resource actions are inside a dictionary with the resource as key. Please check any other API. Another very similar to this one is router-add-extraroutes-request","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"14b9440d5e20341bea12514bfc8b2b5abf0348ce","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"   \"port\" : {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","}],"source_content_type":"application/json","patch_set":8,"id":"cd263d32_ebfd992a","line":2,"range":{"start_line":2,"start_character":3,"end_line":2,"end_character":13},"in_reply_to":"3d6ddc95_3e7e97dd","updated":"2023-04-20 07:18:57.000000000","message":"yeah, i think so, let me modify it next patch.","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"fd6dca38cd51bb32aedde9c62e2c67668ea214d6","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"   \"port\" : {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","}],"source_content_type":"application/json","patch_set":8,"id":"3776a194_6a4a348e","line":2,"range":{"start_line":2,"start_character":3,"end_line":2,"end_character":13},"in_reply_to":"a965238f_dbe69557","updated":"2023-04-20 09:03:09.000000000","message":"Before sending other patch, let\u0027s agree on the expected input and output.","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"745e69c84439a0afc580f971ff28d9b53769d0be","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"   \"port\" : {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","}],"source_content_type":"application/json","patch_set":8,"id":"56c10af9_57ab16fc","line":2,"range":{"start_line":2,"start_character":3,"end_line":2,"end_character":13},"in_reply_to":"a965238f_dbe69557","updated":"2023-04-20 09:07:20.000000000","message":"We should do it in the same way as we did \"extraroutes-atomic\" extension for routers API: https://github.com/openstack/neutron-lib/blob/master/api-ref/source/v2/routers.inc#add-extra-routes-to-router so it should be inside \"port\" dict.","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"21ae2bec108910dce82ab6a2d2adfa9c3e1f8010","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"   \"port\" : {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","}],"source_content_type":"application/json","patch_set":8,"id":"1b32dba5_a681e987","line":2,"range":{"start_line":2,"start_character":3,"end_line":2,"end_character":13},"in_reply_to":"cd263d32_ebfd992a","updated":"2023-04-20 08:31:05.000000000","message":"Done","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"}],"api-ref/source/v2/samples/ports/port-add-allowed-address-pairs-response.json":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"a8c307db14bf77e127e6da42cd3c0baf8c527a33","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"port\": {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","},{"line_number":6,"context_line":"                \"mac_address\": \"fa:14:2a:b3:cb:f0\""},{"line_number":7,"context_line":"            }"},{"line_number":8,"context_line":"        ]"},{"line_number":9,"context_line":"    }"},{"line_number":10,"context_line":"}"}],"source_content_type":"application/json","patch_set":8,"id":"ee640b0b_fa007d03","line":10,"range":{"start_line":1,"start_character":0,"end_line":10,"end_character":1},"updated":"2023-04-20 07:01:22.000000000","message":"not sure we need response body at all, moreover it might be misleading: looks like port has only this single allowed address pair","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"745e69c84439a0afc580f971ff28d9b53769d0be","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"port\": {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","},{"line_number":6,"context_line":"                \"mac_address\": \"fa:14:2a:b3:cb:f0\""},{"line_number":7,"context_line":"            }"},{"line_number":8,"context_line":"        ]"},{"line_number":9,"context_line":"    }"},{"line_number":10,"context_line":"}"}],"source_content_type":"application/json","patch_set":8,"id":"f6148f47_05d4e673","line":10,"range":{"start_line":1,"start_character":0,"end_line":10,"end_character":1},"in_reply_to":"a698da0b_342408b1","updated":"2023-04-20 09:07:20.000000000","message":"Again, let\u0027s be consistent with atomic extraroutes and return body here: https://github.com/openstack/neutron-lib/blob/master/api-ref/source/v2/samples/routers/router-add-extraroutes-response.json","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"14b9440d5e20341bea12514bfc8b2b5abf0348ce","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"port\": {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","},{"line_number":6,"context_line":"                \"mac_address\": \"fa:14:2a:b3:cb:f0\""},{"line_number":7,"context_line":"            }"},{"line_number":8,"context_line":"        ]"},{"line_number":9,"context_line":"    }"},{"line_number":10,"context_line":"}"}],"source_content_type":"application/json","patch_set":8,"id":"a698da0b_342408b1","line":10,"range":{"start_line":1,"start_character":0,"end_line":10,"end_character":1},"in_reply_to":"ee640b0b_fa007d03","updated":"2023-04-20 07:18:57.000000000","message":"Done","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"}],"api-ref/source/v2/samples/ports/port-remove-allowed-address-pairs-request.json":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"59364f4a310fb22a5ed93dbf904db5fc49e805f2","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"   \"port\" : {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","}],"source_content_type":"application/json","patch_set":7,"id":"cb605405_b8a01575","line":2,"range":{"start_line":2,"start_character":10,"end_line":2,"end_character":11},"updated":"2023-04-19 09:45:32.000000000","message":"Same question: do we need the port ID?","commit_id":"2490db67d5a74407f2bf1f4b58b3e16788d71797"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"0c1b193f3b5093236e43bca9dd824660b01adffe","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"   \"port\" : {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": ["},{"line_number":4,"context_line":"            {"},{"line_number":5,"context_line":"                \"ip_address\": \"12.12.11.12\","}],"source_content_type":"application/json","patch_set":7,"id":"0eb50ccd_016e3e84","line":2,"range":{"start_line":2,"start_character":10,"end_line":2,"end_character":11},"in_reply_to":"cb605405_b8a01575","updated":"2023-04-19 11:42:14.000000000","message":"Done","commit_id":"2490db67d5a74407f2bf1f4b58b3e16788d71797"}],"api-ref/source/v2/samples/ports/port-remove-allowed-address-pairs-response.json":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"a8c307db14bf77e127e6da42cd3c0baf8c527a33","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"port\": {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": []"},{"line_number":4,"context_line":"    }"},{"line_number":5,"context_line":"}"}],"source_content_type":"application/json","patch_set":8,"id":"cada090d_294fc50d","line":5,"range":{"start_line":1,"start_character":0,"end_line":5,"end_character":1},"updated":"2023-04-20 07:01:22.000000000","message":"this is even more confusing. I think we either need to return full port data or no body at all, just response status code","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"745e69c84439a0afc580f971ff28d9b53769d0be","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"port\": {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": []"},{"line_number":4,"context_line":"    }"},{"line_number":5,"context_line":"}"}],"source_content_type":"application/json","patch_set":8,"id":"e311297c_1ebc8b6b","line":5,"range":{"start_line":1,"start_character":0,"end_line":5,"end_character":1},"in_reply_to":"5787f40a_c8a84af7","updated":"2023-04-20 09:07:20.000000000","message":"Here also, lets be consistent with extraroutes which says \"The response codes and response body are the same as to the update of the routes parameter. That is the whole router object is returned including the routes parameter which represents the result of the removal.\" - see https://github.com/openstack/neutron-lib/blob/master/api-ref/source/v2/routers.inc#remove-extra-routes-from-router","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"14b9440d5e20341bea12514bfc8b2b5abf0348ce","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"port\": {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": []"},{"line_number":4,"context_line":"    }"},{"line_number":5,"context_line":"}"}],"source_content_type":"application/json","patch_set":8,"id":"5787f40a_c8a84af7","line":5,"range":{"start_line":1,"start_character":0,"end_line":5,"end_character":1},"in_reply_to":"cada090d_294fc50d","updated":"2023-04-20 07:18:57.000000000","message":"Done","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"e4c09239af03eaf088cfbb02c80a9d89d09fdee0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"port\": {"},{"line_number":3,"context_line":"        \"allowed_address_pairs\": []"},{"line_number":4,"context_line":"    }"},{"line_number":5,"context_line":"}"}],"source_content_type":"application/json","patch_set":8,"id":"dbe44e3e_f81ddf43","line":5,"range":{"start_line":1,"start_character":0,"end_line":5,"end_character":1},"in_reply_to":"e311297c_1ebc8b6b","updated":"2023-04-20 09:29:30.000000000","message":"Because i think it would be more expensive if we query the port body.\nOK, let me upgrade it in next patch.","commit_id":"18233f6eb5d34325f7d0b51491a6e525ffe90bce"}],"neutron_lib/api/definitions/allowedaddresspairs_atomic.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"70c21b7d9914bb20f8347e80a80506d683318bd4","unresolved":true,"context_lines":[{"line_number":33,"context_line":"        \u0027remove_allowed_address_pairs\u0027: \u0027PUT\u0027,"},{"line_number":34,"context_line":"    }"},{"line_number":35,"context_line":"}"},{"line_number":36,"context_line":"REQUIRED_EXTENSIONS \u003d [port.ALIAS, allowedaddresspairs.ALIAS]"},{"line_number":37,"context_line":"OPTIONAL_EXTENSIONS \u003d []"},{"line_number":38,"context_line":"ACTION_STATUS \u003d {}"}],"source_content_type":"text/x-python","patch_set":4,"id":"c00cd222_02d26423","line":36,"range":{"start_line":36,"start_character":23,"end_line":36,"end_character":33},"updated":"2023-04-17 14:00:37.000000000","message":"\"port\" is not an extension, \"allowed-address-pairs\". You need to remove port.ALIAS","commit_id":"bfe77059a92f04e55bb2dec740bc0e567b74855a"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"3ffabf6647fdbee48836c87581dbf65f9c5d63a5","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        \u0027remove_allowed_address_pairs\u0027: \u0027PUT\u0027,"},{"line_number":34,"context_line":"    }"},{"line_number":35,"context_line":"}"},{"line_number":36,"context_line":"REQUIRED_EXTENSIONS \u003d [port.ALIAS, allowedaddresspairs.ALIAS]"},{"line_number":37,"context_line":"OPTIONAL_EXTENSIONS \u003d []"},{"line_number":38,"context_line":"ACTION_STATUS \u003d {}"}],"source_content_type":"text/x-python","patch_set":4,"id":"2ec09f88_bfe583a3","line":36,"range":{"start_line":36,"start_character":23,"end_line":36,"end_character":33},"in_reply_to":"bd03e2b8_5caed2af","updated":"2023-04-18 10:45:30.000000000","message":"Done","commit_id":"bfe77059a92f04e55bb2dec740bc0e567b74855a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"93024bfef608df803d36edfafb10a808c6be17e1","unresolved":true,"context_lines":[{"line_number":33,"context_line":"        \u0027remove_allowed_address_pairs\u0027: \u0027PUT\u0027,"},{"line_number":34,"context_line":"    }"},{"line_number":35,"context_line":"}"},{"line_number":36,"context_line":"REQUIRED_EXTENSIONS \u003d [port.ALIAS, allowedaddresspairs.ALIAS]"},{"line_number":37,"context_line":"OPTIONAL_EXTENSIONS \u003d []"},{"line_number":38,"context_line":"ACTION_STATUS \u003d {}"}],"source_content_type":"text/x-python","patch_set":4,"id":"bd03e2b8_5caed2af","line":36,"range":{"start_line":36,"start_character":23,"end_line":36,"end_character":33},"in_reply_to":"c00cd222_02d26423","updated":"2023-04-17 14:01:29.000000000","message":"(message re-written):\n\"port\" is not an extension, \"allowed-address-pairs\" is. You need to remove port.ALIAS","commit_id":"bfe77059a92f04e55bb2dec740bc0e567b74855a"}],"releasenotes/notes/allowedaddresspairs-atomic-apidef-6f9d6865854vwfrs":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7f707062933689cd6abc4ca7a4392781e9a3cb7a","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    New API definition: ``allowedaddresspairs-atomic``."}],"source_content_type":"application/octet-stream","patch_set":4,"id":"ed1a9b09_7f131323","line":4,"range":{"start_line":4,"start_character":54,"end_line":4,"end_character":55},"updated":"2023-04-17 13:56:21.000000000","message":"nit: you can also add the two new API endpoints added: \"add_allowed_address_pair\" and \"remove_allowed_address_pairs\".\n\nPlease if you re-spin this patch, add them.","commit_id":"bfe77059a92f04e55bb2dec740bc0e567b74855a"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"3ffabf6647fdbee48836c87581dbf65f9c5d63a5","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    New API definition: ``allowedaddresspairs-atomic``."}],"source_content_type":"application/octet-stream","patch_set":4,"id":"6f40e885_83e32d86","line":4,"range":{"start_line":4,"start_character":54,"end_line":4,"end_character":55},"in_reply_to":"ed1a9b09_7f131323","updated":"2023-04-18 10:45:30.000000000","message":"Done","commit_id":"bfe77059a92f04e55bb2dec740bc0e567b74855a"}]}