)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"3442aba5f29e475b9db33f9c37eba91bca9abb1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d8a78752_df384829","updated":"2024-05-16 09:57:39.000000000","message":"Testing in https://review.opendev.org/c/openstack/devstack/+/919788","commit_id":"8cd3af69c7cc9e3a4a2062e48958a5ba67abd3f0"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"4eb7f2fd2f185be7cf48d02f2d58f4bdcc36d7bd","unresolved":true,"context_lines":[{"line_number":1985,"context_line":"            options \u003d self._gen_router_port_options(port, prov_net)"},{"line_number":1986,"context_line":"            # Do not fail for cases where logical router port get deleted"},{"line_number":1987,"context_line":"            commands.append(self._nb_idl.lrp_set_options(lrp_name,"},{"line_number":1988,"context_line":"                                                         if_exists\u003dTrue,"},{"line_number":1989,"context_line":"                                                         **options))"},{"line_number":1990,"context_line":"        self._transaction(commands, txn\u003dtxn)"},{"line_number":1991,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"19bef30a_153dcfdd","line":1988,"updated":"2024-05-17 16:30:20.000000000","message":"is it completely safe? How is `.options` (that will now include `if_exists\u003dTrue`) used by the library?","commit_id":"8cd3af69c7cc9e3a4a2062e48958a5ba67abd3f0"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"b85a10fad4177c5e10bd8ac6188f60e7eff34704","unresolved":true,"context_lines":[{"line_number":1985,"context_line":"            options \u003d self._gen_router_port_options(port, prov_net)"},{"line_number":1986,"context_line":"            # Do not fail for cases where logical router port get deleted"},{"line_number":1987,"context_line":"            commands.append(self._nb_idl.lrp_set_options(lrp_name,"},{"line_number":1988,"context_line":"                                                         if_exists\u003dTrue,"},{"line_number":1989,"context_line":"                                                         **options))"},{"line_number":1990,"context_line":"        self._transaction(commands, txn\u003dtxn)"},{"line_number":1991,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"cde48646_26bc4684","line":1988,"in_reply_to":"19bef30a_153dcfdd","updated":"2024-05-28 08:11:00.000000000","message":"yes not safe without bumping minimal version for ovsdbapp as tested/mentioned in https://review.opendev.org/c/openstack/neutron/+/919699, pushed alternatives https://review.opendev.org/q/I447990509cdea9830228d3bc92a97062cc57a472 to avoid ovsdbapp minimal version update","commit_id":"8cd3af69c7cc9e3a4a2062e48958a5ba67abd3f0"}]}