)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"a4b60acea35d85655a77d595c3b0873158dbc048","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a394f227_929622b6","updated":"2023-01-06 22:14:16.000000000","message":"Looks like this was fixed in https://review.opendev.org/c/openstack/neutron/+/865697 - if so we can abandon this and mark the bug fixed.","commit_id":"c08ad5ee8754481a1f20fbc2463386dcf677a5e5"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"8566e288546fb81568e687d746c5f929fd67b120","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"56aa7e51_1d073a2f","in_reply_to":"a394f227_929622b6","updated":"2023-01-09 07:20:10.000000000","message":"@Brian No the other patch doesn\u0027t fix #1974149. changes in neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py are included in other patch so will just rebase this patch on master and update commit message so this patch is specific to fixing test. But yes this patch does similar fix like other patch.","commit_id":"c08ad5ee8754481a1f20fbc2463386dcf677a5e5"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3dcf2b4a46b3649fc6777c6cf09430cca35041ee","unresolved":true,"context_lines":[{"line_number":326,"context_line":"        # don\u0027t update the AgentCache. We use chassis_private.chassis to return"},{"line_number":327,"context_line":"        # data about the agent."},{"line_number":328,"context_line":"        return event \u003d\u003d self.ROW_CREATE or ("},{"line_number":329,"context_line":"            hasattr(old, \u0027nb_cfg\u0027) and not"},{"line_number":330,"context_line":"            (self.table \u003d\u003d \u0027Chassis_Private\u0027 and not row.chassis))"},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"    def run(self, event, row, old):"}],"source_content_type":"text/x-python","patch_set":1,"id":"b53dd4cf_26847cfe","line":329,"range":{"start_line":329,"start_character":12,"end_line":329,"end_character":34},"updated":"2022-06-23 14:14:41.000000000","message":"good catch","commit_id":"c08ad5ee8754481a1f20fbc2463386dcf677a5e5"}],"neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovsdb_monitor.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3dcf2b4a46b3649fc6777c6cf09430cca35041ee","unresolved":true,"context_lines":[{"line_number":473,"context_line":"        nb_cfg_timestamp \u003d timestamp * 1000"},{"line_number":474,"context_line":"        self.sb_api.db_set(\u0027Chassis_Private\u0027, self.chassis_name, ("},{"line_number":475,"context_line":"            \u0027nb_cfg_timestamp\u0027, nb_cfg_timestamp)).execute(check_error\u003dTrue)"},{"line_number":476,"context_line":"        # Also increment nb_cfg by 1 to trigger ChassisAgentWriteEvent which"},{"line_number":477,"context_line":"        # is responsible to update AgentCache"},{"line_number":478,"context_line":"        old_nb_cfg \u003d self.sb_api.db_get(\u0027Chassis_Private\u0027, self.chassis_name,"},{"line_number":479,"context_line":"            \u0027nb_cfg\u0027).execute(check_error\u003dTrue)"},{"line_number":480,"context_line":"        self.sb_api.db_set(\u0027Chassis_Private\u0027, self.chassis_name, ("},{"line_number":481,"context_line":"            \u0027nb_cfg\u0027, old_nb_cfg + 1)).execute(check_error\u003dTrue)"},{"line_number":482,"context_line":"        try:"},{"line_number":483,"context_line":"            n_utils.wait_until_true(check_agent_ts, timeout\u003d5)"},{"line_number":484,"context_line":"        except n_utils.WaitTimeout:"}],"source_content_type":"text/x-python","patch_set":1,"id":"18bf22e8_aa78b3bd","line":481,"range":{"start_line":476,"start_character":8,"end_line":481,"end_character":64},"updated":"2022-06-23 14:14:41.000000000","message":"We are already triggering this event with the previous call. If you want to check it, create another test based on this.","commit_id":"c08ad5ee8754481a1f20fbc2463386dcf677a5e5"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"aa55a1e4aea8399d4e8a0432354cb59716269410","unresolved":true,"context_lines":[{"line_number":473,"context_line":"        nb_cfg_timestamp \u003d timestamp * 1000"},{"line_number":474,"context_line":"        self.sb_api.db_set(\u0027Chassis_Private\u0027, self.chassis_name, ("},{"line_number":475,"context_line":"            \u0027nb_cfg_timestamp\u0027, nb_cfg_timestamp)).execute(check_error\u003dTrue)"},{"line_number":476,"context_line":"        # Also increment nb_cfg by 1 to trigger ChassisAgentWriteEvent which"},{"line_number":477,"context_line":"        # is responsible to update AgentCache"},{"line_number":478,"context_line":"        old_nb_cfg \u003d self.sb_api.db_get(\u0027Chassis_Private\u0027, self.chassis_name,"},{"line_number":479,"context_line":"            \u0027nb_cfg\u0027).execute(check_error\u003dTrue)"},{"line_number":480,"context_line":"        self.sb_api.db_set(\u0027Chassis_Private\u0027, self.chassis_name, ("},{"line_number":481,"context_line":"            \u0027nb_cfg\u0027, old_nb_cfg + 1)).execute(check_error\u003dTrue)"},{"line_number":482,"context_line":"        try:"},{"line_number":483,"context_line":"            n_utils.wait_until_true(check_agent_ts, timeout\u003d5)"},{"line_number":484,"context_line":"        except n_utils.WaitTimeout:"}],"source_content_type":"text/x-python","patch_set":1,"id":"db54bfdf_0debeb0d","line":481,"range":{"start_line":476,"start_character":8,"end_line":481,"end_character":64},"in_reply_to":"18bf22e8_aa78b3bd","updated":"2022-06-23 14:24:01.000000000","message":"@ralonsoh you mean Event is triggered with below call? If yes then in my tests i don\u0027t see it triggered, also not seeing what can trigger it looking https://github.com/openstack/neutron/blob/dda8c70fbe3948de45bf72c5f9be7c657ebe42a9/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py#L328-L330. If it\u0027s something else, then yes please do share the reference i will check that out.\n\n\n  self.sb_api.db_set(\u0027Chassis_Private\u0027, self.chassis_name, (\n            \u0027nb_cfg_timestamp\u0027, nb_cfg_timestamp)).execute(check_error\u003dTrue) ?","commit_id":"c08ad5ee8754481a1f20fbc2463386dcf677a5e5"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"1bf9ac5068972390b86d52ed3ca6abd0faa6d4d0","unresolved":true,"context_lines":[{"line_number":473,"context_line":"        nb_cfg_timestamp \u003d timestamp * 1000"},{"line_number":474,"context_line":"        self.sb_api.db_set(\u0027Chassis_Private\u0027, self.chassis_name, ("},{"line_number":475,"context_line":"            \u0027nb_cfg_timestamp\u0027, nb_cfg_timestamp)).execute(check_error\u003dTrue)"},{"line_number":476,"context_line":"        # Also increment nb_cfg by 1 to trigger ChassisAgentWriteEvent which"},{"line_number":477,"context_line":"        # is responsible to update AgentCache"},{"line_number":478,"context_line":"        old_nb_cfg \u003d self.sb_api.db_get(\u0027Chassis_Private\u0027, self.chassis_name,"},{"line_number":479,"context_line":"            \u0027nb_cfg\u0027).execute(check_error\u003dTrue)"},{"line_number":480,"context_line":"        self.sb_api.db_set(\u0027Chassis_Private\u0027, self.chassis_name, ("},{"line_number":481,"context_line":"            \u0027nb_cfg\u0027, old_nb_cfg + 1)).execute(check_error\u003dTrue)"},{"line_number":482,"context_line":"        try:"},{"line_number":483,"context_line":"            n_utils.wait_until_true(check_agent_ts, timeout\u003d5)"},{"line_number":484,"context_line":"        except n_utils.WaitTimeout:"}],"source_content_type":"text/x-python","patch_set":1,"id":"83b210ec_46ffab9c","line":481,"range":{"start_line":476,"start_character":8,"end_line":481,"end_character":64},"in_reply_to":"db54bfdf_0debeb0d","updated":"2022-11-23 07:51:27.000000000","message":"Seen this today again https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_fbd/864977/2/gate/neutron-functional-with-uwsgi/fbd62ac/testr_results.html","commit_id":"c08ad5ee8754481a1f20fbc2463386dcf677a5e5"}]}