)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28722,"name":"Anton Kurbatov","email":"anton.kurbatov@acronis.com","username":"akurbatov"},"change_message_id":"3659feca000b0128a1e005237e2278be84f718de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4564977d_037106c0","updated":"2022-12-15 12:54:39.000000000","message":"recheck","commit_id":"aa99f6c61478848514aae606a04154086a5731bd"},{"author":{"_account_id":28722,"name":"Anton Kurbatov","email":"anton.kurbatov@acronis.com","username":"akurbatov"},"change_message_id":"a4bf6ead0f1fab0470686fa67f2c129bb0235c8d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f71487f9_d6f38693","updated":"2022-11-30 15:28:25.000000000","message":"recheck\nFail in openstack-tox-cover not related to the patch.\nCreated a new bug for this fail: https://bugs.launchpad.net/neutron/+bug/1998343","commit_id":"aa99f6c61478848514aae606a04154086a5731bd"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"b19b31b55ebe489c96bf3b82ff1188d671b4a42f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"be158e13_11f01eed","updated":"2022-12-21 12:10:34.000000000","message":"recheck\nhttps://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867922?forceReload\u003dtrue was merged  to workaround tempest failures","commit_id":"aa99f6c61478848514aae606a04154086a5731bd"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"f21eabc01321a607ba17a35a69b2fb7ba622171e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"82cfe14b_95412474","updated":"2022-11-28 10:31:12.000000000","message":"recheck\ntempest test seems irrelevant","commit_id":"aa99f6c61478848514aae606a04154086a5731bd"},{"author":{"_account_id":28722,"name":"Anton Kurbatov","email":"anton.kurbatov@acronis.com","username":"akurbatov"},"change_message_id":"0f76fa146fed9c89803f7ffd0cf20c02f4825d91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0265b5da_766c300d","in_reply_to":"82cfe14b_95412474","updated":"2022-11-28 13:06:50.000000000","message":"Yes, the fail in CI really looks unrelated to the patch.\nI managed to reproduce the issue without the patch and create a new issue:\nhttps://bugs.launchpad.net/nova/+bug/1998110","commit_id":"aa99f6c61478848514aae606a04154086a5731bd"}],"neutron/plugins/ml2/drivers/mech_agent.py":[{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"f3f33f483dc61ad63131beb43318745300a88f6b","unresolved":true,"context_lines":[{"line_number":72,"context_line":"            # no point in putting in a block if the status is already ACTIVE"},{"line_number":73,"context_line":"            return"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"        if (port[\u0027device_owner\u0027] in ml2_consts.NO_PBLOCKS_TYPES or"},{"line_number":76,"context_line":"                port[\u0027device_owner\u0027] \u003d\u003d ml2_consts.DEFAULT_DEVICE_OWNER):"},{"line_number":77,"context_line":"            # do not set provisioning_block if it is neutron service port"},{"line_number":78,"context_line":"            # or the port has a default device_owner"}],"source_content_type":"text/x-python","patch_set":1,"id":"732b7c0a_75326cfc","line":75,"updated":"2023-05-23 00:51:37.000000000","message":"Seems nothing changed:\n\u003e\u003e\u003e a \u003d \u0027\u0027\n\u003e\u003e\u003e a \u003d\u003d \u0027\u0027\nTrue\n\u003e\u003e\u003e a in [\u0027\u0027, 1 ,2]\nTrue\n\u003e\u003e\u003e \u0027\u0027 in [a, 1 ,2 ]\nTrue","commit_id":"aa99f6c61478848514aae606a04154086a5731bd"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"f0fba6753da7c647426c90fd8a1b9a9063f1829e","unresolved":true,"context_lines":[{"line_number":73,"context_line":"            return"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"        if (port[\u0027device_owner\u0027] in ml2_consts.NO_PBLOCKS_TYPES or"},{"line_number":76,"context_line":"                port[\u0027device_owner\u0027] \u003d\u003d ml2_consts.DEFAULT_DEVICE_OWNER):"},{"line_number":77,"context_line":"            # do not set provisioning_block if it is neutron service port"},{"line_number":78,"context_line":"            # or the port has a default device_owner"},{"line_number":79,"context_line":"            return"}],"source_content_type":"text/x-python","patch_set":1,"id":"36a7f2f6_0401b31c","line":76,"range":{"start_line":76,"start_character":16,"end_line":76,"end_character":71},"updated":"2022-12-21 12:06:08.000000000","message":"what about the reversed logic? keep default_dev_owner in the no_pblocks_types list, and explicitly check for DEFAULT_DEVICE_OWNER where you need it for DHCP?","commit_id":"aa99f6c61478848514aae606a04154086a5731bd"}]}