)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"626e8b433d9fbee6375080e7210d73dbf4d54c68","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0dc8a213_e7f8486a","updated":"2022-09-13 07:32:58.000000000","message":"Nice! I was not aware a subnet could be used for the allowed-address pairs! Perhaps worth to also cover it on the functional testing side\n\n(adding -1 just due to the failing gates, the test coverage is just a suggestion, not sure how simple/complex that could be)","commit_id":"f7caa0667419efcfb8f2d8a428e009c421488dc7"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"de9e949bbe06ee2799554502844278abcd1caf48","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3aa9d82b_e3d6efa8","updated":"2022-09-13 02:51:57.000000000","message":"recheck","commit_id":"f7caa0667419efcfb8f2d8a428e009c421488dc7"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"8804a622b08dc59dd32891715a90140e546c2203","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e19eb36a_38e55565","updated":"2022-09-15 00:46:45.000000000","message":"recheck","commit_id":"f7caa0667419efcfb8f2d8a428e009c421488dc7"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"aad5e82b7cda1f89e337c8e426e109fdfc3ba918","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"22421d4a_1c0632f0","updated":"2023-07-07 05:41:08.000000000","message":"recheck some functional tests timeout","commit_id":"6203c1b7b8cb446621eb3f658ab3b47647329228"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"2e361b9585e6da06fc44df47b44ef2ba587bbc1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bbf9a414_946d1b72","updated":"2023-07-07 03:29:15.000000000","message":"recheck some functional tests timeout","commit_id":"6203c1b7b8cb446621eb3f658ab3b47647329228"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"71c91029891609e70de00837a8042d6d7f4f3e38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"377bf2cd_3722081b","updated":"2023-07-07 08:03:46.000000000","message":"1) This feature could work but needs to be tested in a tempest test.\n\n2) Apart from this, you need to mention in the documentation that the allowed address CIDR won\u0027t work with FIPS: https://bugzilla.redhat.com/show_bug.cgi?id\u003d1986337\n\n3) Please add a release note and documentation. We are modifying the user input (removing the IP/32 address and providing only the CIDR, if a netmask if given)","commit_id":"c64f42ea52da2c3a017867c85e7f996041f73395"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"03c4d60ad5361a8f37493eaf36ba1abaf787e08a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"524de2c6_cae44913","in_reply_to":"377bf2cd_3722081b","updated":"2023-07-10 06:54:42.000000000","message":"hi rodolfo, as you said 1), this feature is already supported. Now it\u0027s just fixing a bug that doesn\u0027t work in one situation. Is it necessary to add a release note?","commit_id":"c64f42ea52da2c3a017867c85e7f996041f73395"}],"neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovn_client.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"71c91029891609e70de00837a8042d6d7f4f3e38","unresolved":true,"context_lines":[{"line_number":96,"context_line":"                    nb_ovn \u003d self.mech_driver.nb_ovn"},{"line_number":97,"context_line":"                    lsp \u003d nb_ovn.lsp_get(port_data[\u0027id\u0027]).execute()"},{"line_number":98,"context_line":"                    self.assertIsNotNone(lsp)"},{"line_number":99,"context_line":"                    expected \u003d \u0027%s %s 10.0.0.0/26\u0027 % ("},{"line_number":100,"context_line":"                        port_data[\u0027mac_address\u0027],"},{"line_number":101,"context_line":"                        port_data[\u0027fixed_ips\u0027][0][\u0027ip_address\u0027])"},{"line_number":102,"context_line":"                    self.assertEqual([expected], lsp.port_security)"}],"source_content_type":"text/x-python","patch_set":6,"id":"29209b32_567663a9","line":99,"range":{"start_line":99,"start_character":38,"end_line":99,"end_character":49},"updated":"2023-07-07 08:03:46.000000000","message":"+1, nice test","commit_id":"c64f42ea52da2c3a017867c85e7f996041f73395"}]}