)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2bb5ff39217f0c1f8306b8249d5e06a7cf1537b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"65285735_7af0df6d","updated":"2025-02-19 13:53:36.000000000","message":"recheck the failures were caused by timeouts","commit_id":"bf05561aae0e061a44b185c2b633fb4062b03454"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"e492f773dde9a99706a2998f15b13affbcccd210","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"144ea402_4d0bd23d","updated":"2025-02-19 22:10:29.000000000","message":"There are a couple of other places I think can be changed:\n\nneutron/tests/fullstack/resources/config.py\nneutron/tests/functional/agent/l2/base.py\nneutron/tests/functional/agent/linux/test_ip_lib.py","commit_id":"fcde0c9930f012df9891ec9f6a136f2060f99418"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2a183efc2b98ee12f94aabd54a8003fbca93bcd7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4f4c874d_d652ec4c","updated":"2025-02-19 16:51:44.000000000","message":"recheck test_check_metadata_started","commit_id":"fcde0c9930f012df9891ec9f6a136f2060f99418"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"39d3c8ebd95a27f5a4f7d061974eae64b3653dfe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d2b69d09_abe61092","in_reply_to":"144ea402_4d0bd23d","updated":"2025-02-20 12:31:12.000000000","message":"Done","commit_id":"fcde0c9930f012df9891ec9f6a136f2060f99418"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"57c34472977e381dfcc6a9255f5a0caa54acce17","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d25a6c18_8d9d612b","updated":"2025-02-20 22:09:57.000000000","message":"Thanks for working on this, my recommendations seem to have found a failure.","commit_id":"67f9f4aa2cd8209e67c12a535911396c03b0a629"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5481568fc084b7c7aa453c2f12b4cd23dffcfaf9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9b8ba48a_10a17173","updated":"2025-02-28 19:18:38.000000000","message":"recheck different neutron-functional failure","commit_id":"28fb9e8d68bdc880780c4d0563a7622bd649d534"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"20952db04cbf65d4ad563e330a781de3c9de7f18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"3ae7610e_14184148","updated":"2025-03-02 03:42:41.000000000","message":"recheck","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"953b68e86b3cc9795399b3d3791d8eea37af0ef2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"cc032494_fbf680f0","updated":"2025-03-05 15:06:13.000000000","message":"recheck ovn-jammy failure","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"}],"doc/source/admin/deploy-ovs-ha-vrrp.rst":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"8cbfbe576a14fa1a96de091432b2387027d7b88e","unresolved":true,"context_lines":[{"line_number":113,"context_line":"   Replace ``OVERLAY_INTERFACE_IP_ADDRESS`` with the IP address of the"},{"line_number":114,"context_line":"   interface that handles VXLAN overlays for self-service networks."},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"#. In the ``l3_agent.ini`` file, configure the layer-3 agent."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"   .. code-block:: ini"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"      [DEFAULT]"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"#. Start the following services:"},{"line_number":123,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"f6609830_5df760dd","line":120,"range":{"start_line":116,"start_character":0,"end_line":120,"end_character":15},"updated":"2025-02-19 14:02:38.000000000","message":"This section is not needed","commit_id":"40438df0948f427bdd6d0273154d3c74b463f963"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"39d3c8ebd95a27f5a4f7d061974eae64b3653dfe","unresolved":false,"context_lines":[{"line_number":113,"context_line":"   Replace ``OVERLAY_INTERFACE_IP_ADDRESS`` with the IP address of the"},{"line_number":114,"context_line":"   interface that handles VXLAN overlays for self-service networks."},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"#. In the ``l3_agent.ini`` file, configure the layer-3 agent."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"   .. code-block:: ini"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"      [DEFAULT]"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"#. Start the following services:"},{"line_number":123,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"a2734a9d_9ba67997","line":120,"range":{"start_line":116,"start_character":0,"end_line":120,"end_character":15},"in_reply_to":"f6609830_5df760dd","updated":"2025-02-20 12:31:12.000000000","message":"Done","commit_id":"40438df0948f427bdd6d0273154d3c74b463f963"}],"doc/source/admin/deploy-ovs-selfservice.rst":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"8cbfbe576a14fa1a96de091432b2387027d7b88e","unresolved":true,"context_lines":[{"line_number":143,"context_line":"   Replace ``OVERLAY_INTERFACE_IP_ADDRESS`` with the IP address of the"},{"line_number":144,"context_line":"   interface that handles VXLAN overlays for self-service networks."},{"line_number":145,"context_line":""},{"line_number":146,"context_line":"#. In the ``l3_agent.ini`` file, configure the layer-3 agent."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"   .. code-block:: ini"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"      [DEFAULT]"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"#. Start the following services:"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"b88d52f3_0ed878a0","line":150,"range":{"start_line":146,"start_character":0,"end_line":150,"end_character":15},"updated":"2025-02-19 14:02:38.000000000","message":"This section is not needed","commit_id":"40438df0948f427bdd6d0273154d3c74b463f963"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"39d3c8ebd95a27f5a4f7d061974eae64b3653dfe","unresolved":false,"context_lines":[{"line_number":143,"context_line":"   Replace ``OVERLAY_INTERFACE_IP_ADDRESS`` with the IP address of the"},{"line_number":144,"context_line":"   interface that handles VXLAN overlays for self-service networks."},{"line_number":145,"context_line":""},{"line_number":146,"context_line":"#. In the ``l3_agent.ini`` file, configure the layer-3 agent."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"   .. code-block:: ini"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"      [DEFAULT]"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"#. Start the following services:"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"8b25d78c_04f8879f","line":150,"range":{"start_line":146,"start_character":0,"end_line":150,"end_character":15},"in_reply_to":"b88d52f3_0ed878a0","updated":"2025-02-20 12:31:12.000000000","message":"Done","commit_id":"40438df0948f427bdd6d0273154d3c74b463f963"}],"doc/source/install/controller-install-option2-obs.rst":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"8cbfbe576a14fa1a96de091432b2387027d7b88e","unresolved":true,"context_lines":[{"line_number":312,"context_line":"    module needs to be loaded. Check your operating system\u0027s documentation for"},{"line_number":313,"context_line":"    additional details on enabling this module."},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"Configure the layer-3 agent"},{"line_number":316,"context_line":"---------------------------"},{"line_number":317,"context_line":""},{"line_number":318,"context_line":"The Layer-3 (L3) agent provides routing and NAT services for"},{"line_number":319,"context_line":"self-service virtual networks."},{"line_number":320,"context_line":""},{"line_number":321,"context_line":"* Edit the ``/etc/neutron/l3_agent.ini`` file and complete the following"},{"line_number":322,"context_line":"  actions:"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":"  * In the ``[DEFAULT]`` section, configure the Open vSwitch interface driver:"},{"line_number":325,"context_line":""},{"line_number":326,"context_line":"    .. path /etc/neutron/l3_agent.ini"},{"line_number":327,"context_line":"    .. code-block:: ini"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"       [DEFAULT]"},{"line_number":330,"context_line":"       # ..."},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"    .. end"},{"line_number":333,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"96b156f0_b5291a44","line":330,"range":{"start_line":315,"start_character":0,"end_line":330,"end_character":12},"updated":"2025-02-19 14:02:38.000000000","message":"This section is not needed","commit_id":"40438df0948f427bdd6d0273154d3c74b463f963"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"39d3c8ebd95a27f5a4f7d061974eae64b3653dfe","unresolved":false,"context_lines":[{"line_number":312,"context_line":"    module needs to be loaded. Check your operating system\u0027s documentation for"},{"line_number":313,"context_line":"    additional details on enabling this module."},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"Configure the layer-3 agent"},{"line_number":316,"context_line":"---------------------------"},{"line_number":317,"context_line":""},{"line_number":318,"context_line":"The Layer-3 (L3) agent provides routing and NAT services for"},{"line_number":319,"context_line":"self-service virtual networks."},{"line_number":320,"context_line":""},{"line_number":321,"context_line":"* Edit the ``/etc/neutron/l3_agent.ini`` file and complete the following"},{"line_number":322,"context_line":"  actions:"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":"  * In the ``[DEFAULT]`` section, configure the Open vSwitch interface driver:"},{"line_number":325,"context_line":""},{"line_number":326,"context_line":"    .. path /etc/neutron/l3_agent.ini"},{"line_number":327,"context_line":"    .. code-block:: ini"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"       [DEFAULT]"},{"line_number":330,"context_line":"       # ..."},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"    .. end"},{"line_number":333,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"bde0a00f_67926ac3","line":330,"range":{"start_line":315,"start_character":0,"end_line":330,"end_character":12},"in_reply_to":"96b156f0_b5291a44","updated":"2025-02-20 12:31:12.000000000","message":"Done","commit_id":"40438df0948f427bdd6d0273154d3c74b463f963"}],"doc/source/install/controller-install-option2-rdo.rst":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"8cbfbe576a14fa1a96de091432b2387027d7b88e","unresolved":true,"context_lines":[{"line_number":323,"context_line":"* Edit the ``/etc/neutron/l3_agent.ini`` file and complete the following"},{"line_number":324,"context_line":"  actions:"},{"line_number":325,"context_line":""},{"line_number":326,"context_line":"  * In the ``[DEFAULT]`` section, configure the Open vSwitch interface driver:"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    .. path /etc/neutron/l3_agent.ini"},{"line_number":329,"context_line":"    .. code-block:: ini"},{"line_number":330,"context_line":""},{"line_number":331,"context_line":"       [DEFAULT]"},{"line_number":332,"context_line":"       # ..."},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    .. end"}],"source_content_type":"text/x-rst","patch_set":2,"id":"edd39ee4_8dd434af","line":331,"range":{"start_line":326,"start_character":1,"end_line":331,"end_character":16},"updated":"2025-02-19 14:02:38.000000000","message":"ditto","commit_id":"40438df0948f427bdd6d0273154d3c74b463f963"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"39d3c8ebd95a27f5a4f7d061974eae64b3653dfe","unresolved":false,"context_lines":[{"line_number":323,"context_line":"* Edit the ``/etc/neutron/l3_agent.ini`` file and complete the following"},{"line_number":324,"context_line":"  actions:"},{"line_number":325,"context_line":""},{"line_number":326,"context_line":"  * In the ``[DEFAULT]`` section, configure the Open vSwitch interface driver:"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    .. path /etc/neutron/l3_agent.ini"},{"line_number":329,"context_line":"    .. code-block:: ini"},{"line_number":330,"context_line":""},{"line_number":331,"context_line":"       [DEFAULT]"},{"line_number":332,"context_line":"       # ..."},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    .. end"}],"source_content_type":"text/x-rst","patch_set":2,"id":"cfe5b4ae_c0dd13b6","line":331,"range":{"start_line":326,"start_character":1,"end_line":331,"end_character":16},"in_reply_to":"edd39ee4_8dd434af","updated":"2025-02-20 12:31:12.000000000","message":"Done","commit_id":"40438df0948f427bdd6d0273154d3c74b463f963"}],"doc/source/install/controller-install-option2-ubuntu.rst":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"8cbfbe576a14fa1a96de091432b2387027d7b88e","unresolved":true,"context_lines":[{"line_number":331,"context_line":""},{"line_number":332,"context_line":"       [DEFAULT]"},{"line_number":333,"context_line":"       # ..."},{"line_number":334,"context_line":"       interface_driver \u003d openvswitch"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":"    .. end"},{"line_number":337,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"e0036a85_3822c394","side":"PARENT","line":334,"updated":"2025-02-19 14:02:38.000000000","message":"ditto","commit_id":"3454723a97a563a04726a1b3e2557261bac0b475"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"39d3c8ebd95a27f5a4f7d061974eae64b3653dfe","unresolved":false,"context_lines":[{"line_number":331,"context_line":""},{"line_number":332,"context_line":"       [DEFAULT]"},{"line_number":333,"context_line":"       # ..."},{"line_number":334,"context_line":"       interface_driver \u003d openvswitch"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":"    .. end"},{"line_number":337,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"b8357026_17077dbc","side":"PARENT","line":334,"in_reply_to":"e0036a85_3822c394","updated":"2025-02-20 12:31:12.000000000","message":"Done","commit_id":"3454723a97a563a04726a1b3e2557261bac0b475"}],"neutron/tests/fullstack/resources/config.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"57c34472977e381dfcc6a9255f5a0caa54acce17","unresolved":true,"context_lines":[{"line_number":386,"context_line":"        self.config.update({"},{"line_number":387,"context_line":"            \u0027DEFAULT\u0027: {"},{"line_number":388,"context_line":"                \u0027interface_driver\u0027: (\u0027neutron.agent.linux.interface.\u0027"},{"line_number":389,"context_line":"                                     \u0027OVSInterfaceDriver\u0027),"},{"line_number":390,"context_line":"            },"},{"line_number":391,"context_line":"            \u0027OVS\u0027: {"},{"line_number":392,"context_line":"                \u0027integration_bridge\u0027: integration_bridge,"}],"source_content_type":"text/x-python","patch_set":4,"id":"0c24d16a_5dfc59db","side":"PARENT","line":389,"updated":"2025-02-20 22:09:57.000000000","message":"From the fullstack failure looks like we need to leave the DEFAULT key here just with a {}, as on L366 we are going to update it.","commit_id":"3454723a97a563a04726a1b3e2557261bac0b475"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"e5e74103010adfc089d7b15c32f4723353407030","unresolved":false,"context_lines":[{"line_number":386,"context_line":"        self.config.update({"},{"line_number":387,"context_line":"            \u0027DEFAULT\u0027: {"},{"line_number":388,"context_line":"                \u0027interface_driver\u0027: (\u0027neutron.agent.linux.interface.\u0027"},{"line_number":389,"context_line":"                                     \u0027OVSInterfaceDriver\u0027),"},{"line_number":390,"context_line":"            },"},{"line_number":391,"context_line":"            \u0027OVS\u0027: {"},{"line_number":392,"context_line":"                \u0027integration_bridge\u0027: integration_bridge,"}],"source_content_type":"text/x-python","patch_set":4,"id":"d864db0d_30c514e7","side":"PARENT","line":389,"in_reply_to":"0c24d16a_5dfc59db","updated":"2025-02-21 15:32:18.000000000","message":"Done","commit_id":"3454723a97a563a04726a1b3e2557261bac0b475"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"561c89e1f4f99489318e7509af24f83c8742c6a3","unresolved":true,"context_lines":[{"line_number":384,"context_line":""},{"line_number":385,"context_line":"    def _prepare_config_with_ovs_agent(self, integration_bridge):"},{"line_number":386,"context_line":"        self.config.update({"},{"line_number":387,"context_line":"            \u0027DEFAULT\u0027: {"},{"line_number":388,"context_line":"            },"},{"line_number":389,"context_line":"            \u0027OVS\u0027: {"},{"line_number":390,"context_line":"                \u0027integration_bridge\u0027: integration_bridge,"},{"line_number":391,"context_line":"            }"}],"source_content_type":"text/x-python","patch_set":8,"id":"e947ba8c_0f159e99","line":388,"range":{"start_line":387,"start_character":0,"end_line":388,"end_character":14},"updated":"2025-03-03 09:48:39.000000000","message":"nit: maybe this section can be deleted entirely.","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"953b68e86b3cc9795399b3d3791d8eea37af0ef2","unresolved":false,"context_lines":[{"line_number":384,"context_line":""},{"line_number":385,"context_line":"    def _prepare_config_with_ovs_agent(self, integration_bridge):"},{"line_number":386,"context_line":"        self.config.update({"},{"line_number":387,"context_line":"            \u0027DEFAULT\u0027: {"},{"line_number":388,"context_line":"            },"},{"line_number":389,"context_line":"            \u0027OVS\u0027: {"},{"line_number":390,"context_line":"                \u0027integration_bridge\u0027: integration_bridge,"},{"line_number":391,"context_line":"            }"}],"source_content_type":"text/x-python","patch_set":8,"id":"eb9c7bb4_bcb8eafa","line":388,"range":{"start_line":387,"start_character":0,"end_line":388,"end_character":14},"in_reply_to":"76deae10_44b3b85a","updated":"2025-03-05 15:06:13.000000000","message":"Correct, we removed in an earlier PS and it caused a breakage.","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"085c07c3f90a504d4852af589e23088cabb031bb","unresolved":true,"context_lines":[{"line_number":384,"context_line":""},{"line_number":385,"context_line":"    def _prepare_config_with_ovs_agent(self, integration_bridge):"},{"line_number":386,"context_line":"        self.config.update({"},{"line_number":387,"context_line":"            \u0027DEFAULT\u0027: {"},{"line_number":388,"context_line":"            },"},{"line_number":389,"context_line":"            \u0027OVS\u0027: {"},{"line_number":390,"context_line":"                \u0027integration_bridge\u0027: integration_bridge,"},{"line_number":391,"context_line":"            }"}],"source_content_type":"text/x-python","patch_set":8,"id":"76deae10_44b3b85a","line":388,"range":{"start_line":387,"start_character":0,"end_line":388,"end_character":14},"in_reply_to":"e947ba8c_0f159e99","updated":"2025-03-05 14:47:16.000000000","message":"No this is required because the existing logics (especially ones in the chil classes) assumes the DEFAULT key exists when these inject additional options. See L403 for example.\n\nWe can fix it but I tend to leave it now to minimize the change.","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"561c89e1f4f99489318e7509af24f83c8742c6a3","unresolved":true,"context_lines":[{"line_number":419,"context_line":""},{"line_number":420,"context_line":"    def _prepare_config_with_ovs_agent(self, integration_bridge):"},{"line_number":421,"context_line":"        self.config.update({"},{"line_number":422,"context_line":"            \u0027DEFAULT\u0027: {"},{"line_number":423,"context_line":"            },"},{"line_number":424,"context_line":"            \u0027OVS\u0027: {"},{"line_number":425,"context_line":"                \u0027integration_bridge\u0027: integration_bridge,"},{"line_number":426,"context_line":"            }"}],"source_content_type":"text/x-python","patch_set":8,"id":"952689d4_42f69caa","line":423,"range":{"start_line":422,"start_character":0,"end_line":423,"end_character":14},"updated":"2025-03-03 09:48:39.000000000","message":"ditto","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0a141de9e35c8a9a97a3b509f1afe16b7ead0bd3","unresolved":false,"context_lines":[{"line_number":419,"context_line":""},{"line_number":420,"context_line":"    def _prepare_config_with_ovs_agent(self, integration_bridge):"},{"line_number":421,"context_line":"        self.config.update({"},{"line_number":422,"context_line":"            \u0027DEFAULT\u0027: {"},{"line_number":423,"context_line":"            },"},{"line_number":424,"context_line":"            \u0027OVS\u0027: {"},{"line_number":425,"context_line":"                \u0027integration_bridge\u0027: integration_bridge,"},{"line_number":426,"context_line":"            }"}],"source_content_type":"text/x-python","patch_set":8,"id":"984dcdab_4aab181f","line":423,"range":{"start_line":422,"start_character":0,"end_line":423,"end_character":14},"in_reply_to":"4da0c58c_c83597fd","updated":"2025-03-11 18:11:36.000000000","message":"Done","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"085c07c3f90a504d4852af589e23088cabb031bb","unresolved":true,"context_lines":[{"line_number":419,"context_line":""},{"line_number":420,"context_line":"    def _prepare_config_with_ovs_agent(self, integration_bridge):"},{"line_number":421,"context_line":"        self.config.update({"},{"line_number":422,"context_line":"            \u0027DEFAULT\u0027: {"},{"line_number":423,"context_line":"            },"},{"line_number":424,"context_line":"            \u0027OVS\u0027: {"},{"line_number":425,"context_line":"                \u0027integration_bridge\u0027: integration_bridge,"},{"line_number":426,"context_line":"            }"}],"source_content_type":"text/x-python","patch_set":8,"id":"4da0c58c_c83597fd","line":423,"range":{"start_line":422,"start_character":0,"end_line":423,"end_character":14},"in_reply_to":"952689d4_42f69caa","updated":"2025-03-05 14:47:16.000000000","message":"ditto","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"}],"neutron/tests/functional/agent/linux/test_ip_lib.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"57c34472977e381dfcc6a9255f5a0caa54acce17","unresolved":true,"context_lines":[{"line_number":61,"context_line":"        self._configure()"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def _configure(self):"},{"line_number":64,"context_line":"        config.register_interface_driver_opts_helper(cfg.CONF)"},{"line_number":65,"context_line":"        config.register_interface_opts()"},{"line_number":66,"context_line":"        self.driver \u003d importutils.import_object(cfg.CONF.interface_driver,"},{"line_number":67,"context_line":"                                                cfg.CONF)"}],"source_content_type":"text/x-python","patch_set":4,"id":"48160518_b182f275","line":64,"updated":"2025-02-20 22:09:57.000000000","message":"There is some kind of config registration missing here based on the failure. The one think I see two of the other tests do is:\n\nconf \u003d cfg.ConfigOpts()\nconfig.register_interface_driver_opts_helper(conf)\n\nMaybe that will help I just haven\u0027t tried. Otherwise the call on L66 is failing.","commit_id":"67f9f4aa2cd8209e67c12a535911396c03b0a629"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"e5e74103010adfc089d7b15c32f4723353407030","unresolved":true,"context_lines":[{"line_number":61,"context_line":"        self._configure()"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def _configure(self):"},{"line_number":64,"context_line":"        config.register_interface_driver_opts_helper(cfg.CONF)"},{"line_number":65,"context_line":"        config.register_interface_opts()"},{"line_number":66,"context_line":"        self.driver \u003d importutils.import_object(cfg.CONF.interface_driver,"},{"line_number":67,"context_line":"                                                cfg.CONF)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7798ca19_ad8785aa","line":64,"in_reply_to":"48160518_b182f275","updated":"2025-02-21 15:32:18.000000000","message":"Hmm I\u0027m not too sure why this does not work with cfg.CONF. I updated a bit suspicious name conflict in the conf module and I wonder it could make any difference.","commit_id":"67f9f4aa2cd8209e67c12a535911396c03b0a629"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"57c34472977e381dfcc6a9255f5a0caa54acce17","unresolved":true,"context_lines":[{"line_number":64,"context_line":"        config.register_interface_driver_opts_helper(cfg.CONF)"},{"line_number":65,"context_line":"        config.register_interface_opts()"},{"line_number":66,"context_line":"        self.driver \u003d importutils.import_object(cfg.CONF.interface_driver,"},{"line_number":67,"context_line":"                                                cfg.CONF)"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    def generate_device_details(self, name\u003dNone, ip_cidrs\u003dNone,"},{"line_number":70,"context_line":"                                mac_address\u003dNone, namespace\u003dNone):"}],"source_content_type":"text/x-python","patch_set":4,"id":"42efca5b_d5ccea2d","line":67,"updated":"2025-02-20 22:09:57.000000000","message":"These two arguments would need to change accordingly.","commit_id":"67f9f4aa2cd8209e67c12a535911396c03b0a629"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"1b68731fddbfc200b74dbb5c84faacb85ab24c9f","unresolved":true,"context_lines":[{"line_number":64,"context_line":"        config.register_interface_driver_opts_helper(cfg.CONF)"},{"line_number":65,"context_line":"        config.register_interface_opts()"},{"line_number":66,"context_line":"        self.driver \u003d importutils.import_object(cfg.CONF.interface_driver,"},{"line_number":67,"context_line":"                                                cfg.CONF)"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    def generate_device_details(self, name\u003dNone, ip_cidrs\u003dNone,"},{"line_number":70,"context_line":"                                mac_address\u003dNone, namespace\u003dNone):"}],"source_content_type":"text/x-python","patch_set":4,"id":"3190420e_092ea7c4","line":67,"in_reply_to":"42efca5b_d5ccea2d","updated":"2025-02-24 21:24:38.000000000","message":"This is still the line that fails 😞 and this seems to be the only functional test calling import_object()","commit_id":"67f9f4aa2cd8209e67c12a535911396c03b0a629"}],"neutron/tests/unit/agent/dhcp/test_agent.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"e5e74103010adfc089d7b15c32f4723353407030","unresolved":false,"context_lines":[{"line_number":829,"context_line":"        config.register_interface_driver_opts_helper(cfg.CONF)"},{"line_number":830,"context_line":"        cfg.CONF.set_override(\u0027interface_driver\u0027,"},{"line_number":831,"context_line":"                              \u0027neutron.agent.linux.interface.NullDriver\u0027)"},{"line_number":832,"context_line":"        entry.register_options(cfg.CONF)  # register all dhcp cfg options"},{"line_number":833,"context_line":""},{"line_number":834,"context_line":"        self.plugin_p \u003d mock.patch(DHCP_PLUGIN)"},{"line_number":835,"context_line":"        plugin_cls \u003d self.plugin_p.start()"}],"source_content_type":"text/x-python","patch_set":5,"id":"f2bd7c27_ac20ce36","side":"PARENT","line":832,"range":{"start_line":832,"start_character":14,"end_line":832,"end_character":30},"updated":"2025-02-21 15:32:18.000000000","message":"entry.register_options calls register_interface_driver_opts_helper internally so this may cause duplicate options. I don\u0027t know why this does not raise error but anyway we can drop the redundant call.","commit_id":"3454723a97a563a04726a1b3e2557261bac0b475"}],"releasenotes/notes/interface_driver-use-ovs-by-default-53938648662a590b.yaml":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"57c34472977e381dfcc6a9255f5a0caa54acce17","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    The ``[DEFAULT] interface_driver`` option now defaults to ``openvswitch``,"},{"line_number":5,"context_line":"    and is not required if the openvswitch mechanism driver or the ovn"},{"line_number":6,"context_line":"    mechanism driver is used."},{"line_number":7,"context_line":"    "}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ef88bc84_1d2c4f7c","line":7,"updated":"2025-02-20 22:09:57.000000000","message":"nit: blank spaces","commit_id":"67f9f4aa2cd8209e67c12a535911396c03b0a629"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"e5e74103010adfc089d7b15c32f4723353407030","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    The ``[DEFAULT] interface_driver`` option now defaults to ``openvswitch``,"},{"line_number":5,"context_line":"    and is not required if the openvswitch mechanism driver or the ovn"},{"line_number":6,"context_line":"    mechanism driver is used."},{"line_number":7,"context_line":"    "}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5578aebf_06cc9b37","line":7,"in_reply_to":"ef88bc84_1d2c4f7c","updated":"2025-02-21 15:32:18.000000000","message":"Done","commit_id":"67f9f4aa2cd8209e67c12a535911396c03b0a629"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"561c89e1f4f99489318e7509af24f83c8742c6a3","unresolved":true,"context_lines":[{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The ``[DEFAULT] interface_driver`` option now defaults to ``openvswitch``,"},{"line_number":5,"context_line":"    and is not required if the openvswitch mechanism driver or the ovn"},{"line_number":6,"context_line":"    mechanism driver is used."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"deab89ae_ac3340b8","line":6,"range":{"start_line":5,"start_character":60,"end_line":6,"end_character":29},"updated":"2025-03-03 09:48:39.000000000","message":"IMO, this is not related to OVN mechanism at all, because there is no python neutron-*-agent running with such config option for it.","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"953b68e86b3cc9795399b3d3791d8eea37af0ef2","unresolved":true,"context_lines":[{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The ``[DEFAULT] interface_driver`` option now defaults to ``openvswitch``,"},{"line_number":5,"context_line":"    and is not required if the openvswitch mechanism driver or the ovn"},{"line_number":6,"context_line":"    mechanism driver is used."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"4c33a3fa_337203ba","line":6,"range":{"start_line":5,"start_character":60,"end_line":6,"end_character":29},"in_reply_to":"4bee6bd6_38724edf","updated":"2025-03-05 15:06:13.000000000","message":"I think the wording is ok here, we\u0027re just trying to say if you\u0027re not using OVS or OVN then you need to set it to some other value.","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0a141de9e35c8a9a97a3b509f1afe16b7ead0bd3","unresolved":false,"context_lines":[{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The ``[DEFAULT] interface_driver`` option now defaults to ``openvswitch``,"},{"line_number":5,"context_line":"    and is not required if the openvswitch mechanism driver or the ovn"},{"line_number":6,"context_line":"    mechanism driver is used."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"4e57ff09_4d1f89ca","line":6,"range":{"start_line":5,"start_character":60,"end_line":6,"end_character":29},"in_reply_to":"4c33a3fa_337203ba","updated":"2025-03-11 18:11:36.000000000","message":"Done","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"085c07c3f90a504d4852af589e23088cabb031bb","unresolved":true,"context_lines":[{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The ``[DEFAULT] interface_driver`` option now defaults to ``openvswitch``,"},{"line_number":5,"context_line":"    and is not required if the openvswitch mechanism driver or the ovn"},{"line_number":6,"context_line":"    mechanism driver is used."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"4bee6bd6_38724edf","line":6,"range":{"start_line":5,"start_character":60,"end_line":6,"end_character":29},"in_reply_to":"deab89ae_ac3340b8","updated":"2025-03-05 14:47:16.000000000","message":"AFAIK we can run technically neutron-dhcp-agent in ml2-ovn. What I meant to insist here is that this option is no longer required as long as built-in driver is used (it may be still required if a third party plugin is used). Probably describing that is a better option here ?","commit_id":"f92c677b5691d5ec89c04c0dd66835308a85e428"}]}