)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2964f22850cda89190eb90f22c42023d0f906abf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c38b8915_4f2c0967","updated":"2025-03-20 13:34:53.000000000","message":"recheck neutron-functional","commit_id":"dd7c7f4b2bea8081ccd7408560068e29710f1ac0"}],"neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovsdb_monitor.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"cb3722cfb0d7301d28d45a4d6e1d3b09e1fa50be","unresolved":true,"context_lines":[{"line_number":829,"context_line":"        self.sb_api.db_set("},{"line_number":830,"context_line":"            \u0027SB_Global\u0027, \u0027.\u0027,"},{"line_number":831,"context_line":"            (\u0027external_ids\u0027, {\u0027delete_agent\u0027: agent_id})).execute("},{"line_number":832,"context_line":"            check_error\u003dTrue)"},{"line_number":833,"context_line":"        self.sb_api.db_remove("},{"line_number":834,"context_line":"            \u0027SB_Global\u0027, \u0027.\u0027, \u0027external_ids\u0027, delete_agent\u003dagent_id,"},{"line_number":835,"context_line":"            if_exists\u003dTrue).execute(check_error\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ee24b2aa_c7d5c6c6","line":832,"updated":"2025-03-18 21:41:54.000000000","message":"Just wondering if a db_get() should be issued to make sure the entry is there before trying to remove it, or is this guaranteed to be in the ovsdb cache and that doesn\u0027t matter?","commit_id":"dd7c7f4b2bea8081ccd7408560068e29710f1ac0"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ac823dd09a40cb498b0356cf755ebf26f499c4a3","unresolved":false,"context_lines":[{"line_number":829,"context_line":"        self.sb_api.db_set("},{"line_number":830,"context_line":"            \u0027SB_Global\u0027, \u0027.\u0027,"},{"line_number":831,"context_line":"            (\u0027external_ids\u0027, {\u0027delete_agent\u0027: agent_id})).execute("},{"line_number":832,"context_line":"            check_error\u003dTrue)"},{"line_number":833,"context_line":"        self.sb_api.db_remove("},{"line_number":834,"context_line":"            \u0027SB_Global\u0027, \u0027.\u0027, \u0027external_ids\u0027, delete_agent\u003dagent_id,"},{"line_number":835,"context_line":"            if_exists\u003dTrue).execute(check_error\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"2e0a94d5_e5f092fb","line":832,"in_reply_to":"ee24b2aa_c7d5c6c6","updated":"2025-03-19 08:52:48.000000000","message":"By default this key is not present in SB_Global.external_ids. This is the same strategy used in [1] to generate a delete event.\n\n[1]https://github.com/openstack/neutron/blob/555eb6d32e31a95abdd64607a60ae75e479f6837/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py#L1561-L1569","commit_id":"dd7c7f4b2bea8081ccd7408560068e29710f1ac0"}]}