)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"7cda8813fa3e19eccf0c36a994401a33fb4dc657","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"44a2abf5_9cd9ef18","updated":"2024-05-21 14:35:12.000000000","message":"I\u0027m ok with moving forward with it; though I think we could wait for DELETE only if we\u0027d use del_fake_chassis. I will post a follow-up to see if it works.","commit_id":"20b9893e34dda0b448ac75c795867cb46de5e127"}],"neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/test_mech_driver.py":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"c78827a2f9bba8c6a2988c201f954c35ad5a72c7","unresolved":true,"context_lines":[{"line_number":1237,"context_line":"                                     events\u003d(event.RowEvent.ROW_UPDATE,"},{"line_number":1238,"context_line":"                                             event.RowEvent.ROW_DELETE,))"},{"line_number":1239,"context_line":"        self.sb_api.idl.notify_handler.watch_event(agent_event)"},{"line_number":1240,"context_line":"        self.sb_api.chassis_del(self.chassis).execute(check_error\u003dTrue)"},{"line_number":1241,"context_line":"        self.assertTrue(agent_event.wait())"},{"line_number":1242,"context_line":"        agent_ids \u003d [a[\u0027id\u0027] for a in self.plugin.get_agents("},{"line_number":1243,"context_line":"            self.context, filters\u003d{\u0027host\u0027: self.host})]"}],"source_content_type":"text/x-python","patch_set":1,"id":"ebe826ed_4d6de110","line":1240,"updated":"2024-05-21 14:01:38.000000000","message":"I think you could call `del_fake_chassis` here and then remove `ROW_UPDATE` from the list of events expected - and make the test case checking the same thing for all ovsdbapp versions. Thoughts?","commit_id":"e443c9426da076377904c7de1d94ed7bc612fae4"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"cdc5db2819e168d839419e5dac178483843ee573","unresolved":true,"context_lines":[{"line_number":1237,"context_line":"                                     events\u003d(event.RowEvent.ROW_UPDATE,"},{"line_number":1238,"context_line":"                                             event.RowEvent.ROW_DELETE,))"},{"line_number":1239,"context_line":"        self.sb_api.idl.notify_handler.watch_event(agent_event)"},{"line_number":1240,"context_line":"        self.sb_api.chassis_del(self.chassis).execute(check_error\u003dTrue)"},{"line_number":1241,"context_line":"        self.assertTrue(agent_event.wait())"},{"line_number":1242,"context_line":"        agent_ids \u003d [a[\u0027id\u0027] for a in self.plugin.get_agents("},{"line_number":1243,"context_line":"            self.context, filters\u003d{\u0027host\u0027: self.host})]"}],"source_content_type":"text/x-python","patch_set":1,"id":"9b191549_dda453a8","line":1240,"in_reply_to":"51ccad68_f00b2960","updated":"2024-05-21 15:40:05.000000000","message":"thx Ihar","commit_id":"e443c9426da076377904c7de1d94ed7bc612fae4"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"7cda8813fa3e19eccf0c36a994401a33fb4dc657","unresolved":true,"context_lines":[{"line_number":1237,"context_line":"                                     events\u003d(event.RowEvent.ROW_UPDATE,"},{"line_number":1238,"context_line":"                                             event.RowEvent.ROW_DELETE,))"},{"line_number":1239,"context_line":"        self.sb_api.idl.notify_handler.watch_event(agent_event)"},{"line_number":1240,"context_line":"        self.sb_api.chassis_del(self.chassis).execute(check_error\u003dTrue)"},{"line_number":1241,"context_line":"        self.assertTrue(agent_event.wait())"},{"line_number":1242,"context_line":"        agent_ids \u003d [a[\u0027id\u0027] for a in self.plugin.get_agents("},{"line_number":1243,"context_line":"            self.context, filters\u003d{\u0027host\u0027: self.host})]"}],"source_content_type":"text/x-python","patch_set":1,"id":"a3b62e00_6f36da00","line":1240,"in_reply_to":"85d85e88_d579a103","updated":"2024-05-21 14:35:12.000000000","message":"you would list events\u003d as ROW_DELETE only. Then your del_fake_chassis would take care of issuing delete for Private in case ovsdbapp doesn\u0027t already. So either way, you\u0027d get the event and the `wait()` would proceed.","commit_id":"e443c9426da076377904c7de1d94ed7bc612fae4"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"7b287cd7d43624259500d42abf0b5d79a7eff9b6","unresolved":false,"context_lines":[{"line_number":1237,"context_line":"                                     events\u003d(event.RowEvent.ROW_UPDATE,"},{"line_number":1238,"context_line":"                                             event.RowEvent.ROW_DELETE,))"},{"line_number":1239,"context_line":"        self.sb_api.idl.notify_handler.watch_event(agent_event)"},{"line_number":1240,"context_line":"        self.sb_api.chassis_del(self.chassis).execute(check_error\u003dTrue)"},{"line_number":1241,"context_line":"        self.assertTrue(agent_event.wait())"},{"line_number":1242,"context_line":"        agent_ids \u003d [a[\u0027id\u0027] for a in self.plugin.get_agents("},{"line_number":1243,"context_line":"            self.context, filters\u003d{\u0027host\u0027: self.host})]"}],"source_content_type":"text/x-python","patch_set":1,"id":"83ce9fee_78b2afaa","line":1240,"in_reply_to":"9b191549_dda453a8","updated":"2024-05-21 15:40:33.000000000","message":"Done","commit_id":"e443c9426da076377904c7de1d94ed7bc612fae4"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"631e10dca7de9ab39d480f40cef054fabc22d40d","unresolved":true,"context_lines":[{"line_number":1237,"context_line":"                                     events\u003d(event.RowEvent.ROW_UPDATE,"},{"line_number":1238,"context_line":"                                             event.RowEvent.ROW_DELETE,))"},{"line_number":1239,"context_line":"        self.sb_api.idl.notify_handler.watch_event(agent_event)"},{"line_number":1240,"context_line":"        self.sb_api.chassis_del(self.chassis).execute(check_error\u003dTrue)"},{"line_number":1241,"context_line":"        self.assertTrue(agent_event.wait())"},{"line_number":1242,"context_line":"        agent_ids \u003d [a[\u0027id\u0027] for a in self.plugin.get_agents("},{"line_number":1243,"context_line":"            self.context, filters\u003d{\u0027host\u0027: self.host})]"}],"source_content_type":"text/x-python","patch_set":1,"id":"51ccad68_f00b2960","line":1240,"in_reply_to":"a3b62e00_6f36da00","updated":"2024-05-21 14:39:08.000000000","message":"trying here: https://review.opendev.org/c/openstack/neutron/+/920103","commit_id":"e443c9426da076377904c7de1d94ed7bc612fae4"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"8fa52a972da9cec8f4dd10ee5bada16c507aba72","unresolved":true,"context_lines":[{"line_number":1237,"context_line":"                                     events\u003d(event.RowEvent.ROW_UPDATE,"},{"line_number":1238,"context_line":"                                             event.RowEvent.ROW_DELETE,))"},{"line_number":1239,"context_line":"        self.sb_api.idl.notify_handler.watch_event(agent_event)"},{"line_number":1240,"context_line":"        self.sb_api.chassis_del(self.chassis).execute(check_error\u003dTrue)"},{"line_number":1241,"context_line":"        self.assertTrue(agent_event.wait())"},{"line_number":1242,"context_line":"        agent_ids \u003d [a[\u0027id\u0027] for a in self.plugin.get_agents("},{"line_number":1243,"context_line":"            self.context, filters\u003d{\u0027host\u0027: self.host})]"}],"source_content_type":"text/x-python","patch_set":1,"id":"85d85e88_d579a103","line":1240,"in_reply_to":"ebe826ed_4d6de110","updated":"2024-05-21 14:31:10.000000000","message":"But with empty events it will default to CREATE event and that doesn\u0027t seems to be triggered.\nConsidering this temporary may be can go with this only. will remove depends-on for now and get it ready, as couldn\u0027t get to other changes till tomorrow.","commit_id":"e443c9426da076377904c7de1d94ed7bc612fae4"}]}