)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"15467cb68e88fd86550a997a20bf9484066fa3a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"44a7daaf_91359636","updated":"2025-04-07 14:33:14.000000000","message":"this is alpha version","commit_id":"907f40ff80ee28cb0946f93991a4688947e4bab8"}],"networking_generic_switch/devices/ovs/vtep.py":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"15467cb68e88fd86550a997a20bf9484066fa3a6","unresolved":true,"context_lines":[{"line_number":117,"context_line":""},{"line_number":118,"context_line":"        :param trunk_details: trunk information if port is a part of trunk"},{"line_number":119,"context_line":"        \"\"\""},{"line_number":120,"context_line":"        self.vtep.unbind_ls(self.ps_name, port_id, 0).execute(check_error\u003dTrue)"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    def add_subports_on_trunk(self, binding_profile, port_id, subports):"},{"line_number":123,"context_line":"        \"\"\"Allow subports on trunk"}],"source_content_type":"text/x-python","patch_set":1,"id":"9785328f_4bc31afe","line":120,"range":{"start_line":120,"start_character":18,"end_line":120,"end_character":27},"updated":"2025-04-07 14:33:14.000000000","message":"add handling of subports","commit_id":"907f40ff80ee28cb0946f93991a4688947e4bab8"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"cd52703b15bb0bef10be1442ebb565ed55f46212","unresolved":true,"context_lines":[{"line_number":48,"context_line":"            _idl \u003d connection.OvsdbIdl.from_server(self.ovsdb_connection,"},{"line_number":49,"context_line":"                                                   \u0027hardware_vtep\u0027)"},{"line_number":50,"context_line":"            _connection \u003d connection.Connection(_idl, 5)"},{"line_number":51,"context_line":"            self._vtep \u003d impl_idl.HwVtepApiIdlImpl(_connection)"},{"line_number":52,"context_line":"        return self._vtep"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    @property"}],"source_content_type":"text/x-python","patch_set":3,"id":"4e6a2185_91ce9f6c","line":51,"range":{"start_line":51,"start_character":25,"end_line":51,"end_character":63},"updated":"2025-04-14 19:31:53.000000000","message":"this will not work as connection is stored in class attr","commit_id":"1731a6b785d947071efdb112fb8f703020413fec"}],"networking_generic_switch/generic_switch_mech.py":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"7d158c35da7d0b3e23aba2cc1f39aa2cd31469dc","unresolved":true,"context_lines":[{"line_number":386,"context_line":"                          {\u0027switch_port\u0027: port_id, \u0027switch_info\u0027: switch_info,"},{"line_number":387,"context_line":"                           \u0027segmentation_id\u0027: segmentation_id})"},{"line_number":388,"context_line":"                trunk_details \u003d port.get(\u0027trunk_details\u0027, {})"},{"line_number":389,"context_line":"                plug_kwargs \u003d {\"network_id\": port[\"network_id\"]}"},{"line_number":390,"context_line":"                if trunk_details:"},{"line_number":391,"context_line":"                    plug_kwargs[\"trunk_details\"] \u003d trunk_details"},{"line_number":392,"context_line":"                # Move port to network"}],"source_content_type":"text/x-python","patch_set":3,"id":"195e0680_59ad4172","line":389,"range":{"start_line":389,"start_character":30,"end_line":389,"end_character":64},"updated":"2025-04-09 10:02:40.000000000","message":"this will break out of tree drivers, not sure if we want to support it.\n\n@jay@jvf.cc, @dtantsur@protonmail.com, @juliaashleykreger@gmail.com plese share your thoughts on this, we have 3 options:\n\n1. Prohibit out of tree drivers, will give us full freedom on changes as all drivers will be internal\n2. Introduce driver API versionsing possible implementation https://review.opendev.org/c/openstack/networking-generic-switch/+/946639\n3. Just assume we do not allow out of tree drivers and just break APIs without any notification like its done in this patch, more relaxed than option1 will give ability to have out of tree drivers but without any commitments on API stability.\n\nI personally prefer either 1st or 3rd option, but 2nd is also good enough.","commit_id":"1731a6b785d947071efdb112fb8f703020413fec"}]}