)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"9aea9b8af2add08033a4e82d3895e6d60b4142fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"86d91a2d_c70d983c","updated":"2025-02-11 14:14:08.000000000","message":"So if https://review.opendev.org/c/openstack/neutron/+/941154 is \"no need to backport\", we should just drop the extra parenthesis directly in this backport?","commit_id":"d1d8097e0f6b9b81e5e84d815eb06f93bccb4bb2"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"40b45b5cff2bf2e3e6d0d89da95d2bad76be0287","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"70ccb906_32bbd645","in_reply_to":"86d91a2d_c70d983c","updated":"2025-02-11 21:41:49.000000000","message":"Right, we need to drop the parens here but don\u0027t need to backport that change since py39 seems to be ok with it.","commit_id":"d1d8097e0f6b9b81e5e84d815eb06f93bccb4bb2"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"177b077c50097ed763ed285bd3a490c9d3b97aca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1a368308_63298f85","updated":"2025-02-18 08:05:04.000000000","message":"PEP8 happy with the parenthesis, rest of the backport looked good already","commit_id":"502df59a93de07b1e2f0025bcbcd137942e1d33b"}],"neutron/tests/unit/agent/linux/test_external_process.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"291b75b05635d43bc8c1c2f3445d5945a242e220","unresolved":true,"context_lines":[{"line_number":326,"context_line":"        else:"},{"line_number":327,"context_line":"            expected_cmd \u003d [\u0027kill\u0027, \u0027-9\u0027, 4]"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"        with (mock.patch.object(ep.ProcessManager, \u0027pid\u0027) as pid):"},{"line_number":330,"context_line":"            pid.__get__ \u003d mock.Mock(return_value\u003d4)"},{"line_number":331,"context_line":"            with mock.patch.object(ep.ProcessManager, \u0027active\u0027) as active:"},{"line_number":332,"context_line":"                active.__get__ \u003d mock.Mock(return_value\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"c8709295_d6c3f6e6","line":329,"updated":"2025-02-10 19:43:58.000000000","message":"I hadn\u0027t even noticed the added parens here, but py38 doesn\u0027t like it, and they aren\u0027t necessary. I proposed a change on master to remove them https://review.opendev.org/c/openstack/neutron/+/941154","commit_id":"d1d8097e0f6b9b81e5e84d815eb06f93bccb4bb2"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"630c036d05bfc29433a36875a3d86047a94edf94","unresolved":false,"context_lines":[{"line_number":326,"context_line":"        else:"},{"line_number":327,"context_line":"            expected_cmd \u003d [\u0027kill\u0027, \u0027-9\u0027, 4]"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"        with (mock.patch.object(ep.ProcessManager, \u0027pid\u0027) as pid):"},{"line_number":330,"context_line":"            pid.__get__ \u003d mock.Mock(return_value\u003d4)"},{"line_number":331,"context_line":"            with mock.patch.object(ep.ProcessManager, \u0027active\u0027) as active:"},{"line_number":332,"context_line":"                active.__get__ \u003d mock.Mock(return_value\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"021c279f_23cb2496","line":329,"in_reply_to":"c8709295_d6c3f6e6","updated":"2025-02-18 06:06:41.000000000","message":"Done","commit_id":"d1d8097e0f6b9b81e5e84d815eb06f93bccb4bb2"}]}