)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"465c74b31fde444c08b9ddf59e14e10b65077f80","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"84c2ff21_a8469c57","updated":"2026-01-13 14:46:34.000000000","message":"Another approach i see, is to remove the unbound filter from [1], and somehow check there if the port is either unbound or already virtual.\n\nBut since the update_port_precommit already kind of prevents a virtual port to be updated/bound i figured this approach was more consistent.","commit_id":"b1b60d7ddbc55fb944e8db25b9496e43caf311d5"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"135f17e2bd31dcce88ea71691600302321bf6533","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fdd4a209_63d1b546","updated":"2026-01-13 14:48:42.000000000","message":"Please let me know which approach has the preference of the neutron team, so i can focus my efforts for unit testing on that one and abandon the approach which is least likely.","commit_id":"b1b60d7ddbc55fb944e8db25b9496e43caf311d5"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"585ac9c85c9f0cbb3a99994feabe69092a408b06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6739dac5_4d754c6e","updated":"2026-01-16 08:51:53.000000000","message":"So this is the final solution and https://review.opendev.org/c/openstack/neutron/+/973115 can be abandoned?","commit_id":"b1b60d7ddbc55fb944e8db25b9496e43caf311d5"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"08863786a4a0489d814aab938d21844e5a666532","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"72a7478d_442ee69a","in_reply_to":"6739dac5_4d754c6e","updated":"2026-01-16 12:57:33.000000000","message":"Well that is the question, yes. \n\nI think this change is more in line with the rest of the code, but both would do the same thing..","commit_id":"b1b60d7ddbc55fb944e8db25b9496e43caf311d5"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py":[{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"2970ab506b04d12e7142e54de27224ee9e40326d","unresolved":false,"context_lines":[{"line_number":1135,"context_line":""},{"line_number":1136,"context_line":"            # As this code might not be called via the binding precommit"},{"line_number":1137,"context_line":"            # hook, we need to validate the binding profile here as well."},{"line_number":1138,"context_line":"            ovn_utils.validate_port_binding_and_virtual_port("},{"line_number":1139,"context_line":"                context, self._plugin, port, original_port)"},{"line_number":1140,"context_line":"        except n_exc.BadRequest as e:"},{"line_number":1141,"context_line":"            # This means the port is type virtual and it should stay unbound"}],"source_content_type":"text/x-python","patch_set":1,"id":"a70398bf_192d43c8","line":1138,"updated":"2026-01-13 14:43:49.000000000","message":"Same method is used in update_port_precommit (in this file), but since we are already in bind_port i needed to call the set_binding to prevent the port to go to binding_failed.","commit_id":"b1b60d7ddbc55fb944e8db25b9496e43caf311d5"}]}