)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"13c904db9e1588645534bade6525b3c9c3895449","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bb7d630a_a33d3e0c","updated":"2025-04-08 17:10:45.000000000","message":"Feedback from review jam at PTG:\n\nJay+Julia: This change makes sense, but we need documentation of motivation and overall goal in an RFE or spec. Especially since this is establishing a contract. It\u0027s also OK if you have an out of tree driver doing crazy stuff to ... just bring it in tree :).\n\nBasically, what\u0027s the driver for this ask? \n\nThanks!!","commit_id":"7167cc55082f240fa9812e3c75b3cf4c3b6dc042"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"e7ce570081861cb93462ae991e563a158b0a2b02","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"65268bbf_2b0106e6","updated":"2025-04-08 17:23:50.000000000","message":"thanks for feedback Jay","commit_id":"7167cc55082f240fa9812e3c75b3cf4c3b6dc042"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"e7ce570081861cb93462ae991e563a158b0a2b02","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fa503ae0_a53b93bd","in_reply_to":"bb7d630a_a33d3e0c","updated":"2025-04-08 17:23:50.000000000","message":"If we think we do not support out of tree drivers, I would prefer to do not introduce it. The motivation for this is to avoid having something like https://github.com/openstack/networking-generic-switch/blob/master/networking_generic_switch/generic_switch_mech.py#L403\n\nWe may need to add additional kwargs for switch driver methods like https://review.opendev.org/c/openstack/networking-generic-switch/+/946558/2/networking_generic_switch/generic_switch_mech.py#L390. But if we do not allow out of tree drivers I can just update the code of all internal drivers.","commit_id":"7167cc55082f240fa9812e3c75b3cf4c3b6dc042"}]}