)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ca0f2cf5a2d8029bed1ecbabbbf217d519ccdca3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8515f6bc_1d935d4b","updated":"2025-01-15 20:47:03.000000000","message":"recheck do additional functional check","commit_id":"956803819854546992a620cd6eb44acfd291fb79"}],"neutron/tests/functional/agent/common/test_ovsdb_monitor.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"7b9ccd51db66cc1b973dfa3a25972f3622e8d691","unresolved":true,"context_lines":[{"line_number":70,"context_line":"            remaining \u003d self._check_port_events("},{"line_number":71,"context_line":"                mon, ports_expected\u003dports_expected)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"            # Next time check only the ports not seen yet"},{"line_number":74,"context_line":"            ports_expected.clear()  # Python doesn\u0027t support {:} syntax"},{"line_number":75,"context_line":"            ports_expected.update(remaining)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"            return bool(ports_expected)  # True if there are remaining ports"},{"line_number":78,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1e3b0f85_495dca70","line":75,"range":{"start_line":73,"start_character":0,"end_line":75,"end_character":44},"updated":"2025-01-15 21:34:27.000000000","message":"```\nports_expected -\u003d remaining\n```\nmight be more idiomatic","commit_id":"956803819854546992a620cd6eb44acfd291fb79"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"a8be8085c418153ff432bfc983bae5d61653f087","unresolved":true,"context_lines":[{"line_number":70,"context_line":"            remaining \u003d self._check_port_events("},{"line_number":71,"context_line":"                mon, ports_expected\u003dports_expected)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"            # Next time check only the ports not seen yet"},{"line_number":74,"context_line":"            ports_expected.clear()  # Python doesn\u0027t support {:} syntax"},{"line_number":75,"context_line":"            ports_expected.update(remaining)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"            return bool(ports_expected)  # True if there are remaining ports"},{"line_number":78,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7eec9a37_358ce9ef","line":75,"range":{"start_line":73,"start_character":0,"end_line":75,"end_character":44},"in_reply_to":"1e3b0f85_495dca70","updated":"2025-01-15 21:49:46.000000000","message":"It was pointed to me this is not what we want here. We could have just replace L41 to the lines above avoiding introducing `process_new_events` helper. But this is good too.","commit_id":"956803819854546992a620cd6eb44acfd291fb79"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"b2ab45027ee44f631e33bf311645d2a3bb9c6314","unresolved":true,"context_lines":[{"line_number":70,"context_line":"            remaining \u003d self._check_port_events("},{"line_number":71,"context_line":"                mon, ports_expected\u003dports_expected)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"            # Next time check only the ports not seen yet"},{"line_number":74,"context_line":"            ports_expected.clear()  # Python doesn\u0027t support {:} syntax"},{"line_number":75,"context_line":"            ports_expected.update(remaining)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"            return bool(ports_expected)  # True if there are remaining ports"},{"line_number":78,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"6b87d877_7b59dd56","line":75,"range":{"start_line":73,"start_character":0,"end_line":75,"end_character":44},"in_reply_to":"1e3b0f85_495dca70","updated":"2025-01-15 21:44:54.000000000","message":"No. I\u0027d like to keep the set set to `remaining`, not vice versa.","commit_id":"956803819854546992a620cd6eb44acfd291fb79"}]}