)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"bb6b7095b933fda9759e07a61c1afbef9f564edf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"12ff5ac3_f6dddc19","updated":"2026-04-17 01:00:08.000000000","message":"recheck ironic devstack plugin fixed","commit_id":"10f0f49d4b071d85a9c2d300450da7df024c60cd"},{"author":{"_account_id":25437,"name":"Allain Legacy","email":"alegacy@redhat.com","username":"alegacy"},"change_message_id":"c57cfb73ca9dce1aad8609230c0508491fe9809d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9c164cac_32a61a3a","updated":"2026-06-02 11:31:37.000000000","message":"Thanks for adjusting the default mtu handling.  LGTM","commit_id":"e97a1d60f4ef1ae9daa584615f65222dfbd85d99"}],"networking_generic_switch/devices/netmiko_devices/__init__.py":[{"author":{"_account_id":25437,"name":"Allain Legacy","email":"alegacy@redhat.com","username":"alegacy"},"change_message_id":"fb05a530a2c9cc0cb7d5cde8a985cefce9401de7","unresolved":true,"context_lines":[{"line_number":631,"context_line":"                port\u003dport,"},{"line_number":632,"context_line":"                segmentation_id\u003dsegmentation_id)"},{"line_number":633,"context_line":""},{"line_number":634,"context_line":"        if self.SET_PORT_MTU and mtu:"},{"line_number":635,"context_line":"            cmds +\u003d self._format_commands("},{"line_number":636,"context_line":"                self.SET_PORT_MTU, port\u003dport, mtu\u003dmtu)"},{"line_number":637,"context_line":""},{"line_number":638,"context_line":"        return self.send_commands_to_device(cmds)"},{"line_number":639,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"18d1930c_96066710","line":636,"range":{"start_line":634,"start_character":8,"end_line":636,"end_character":54},"updated":"2026-04-20 15:48:06.000000000","message":"For the sake of the standalone case, I\u0027d like to see this done the same way that the default_vlan was done (see line 613) so that it can be overridden by the caller where necessary, but in other cases falls back to the config default. \n\nSomething like or similar to:\n\n```suggestion\n        port_mtu \u003d mtu or self._get_port_default_mtu()\n        if self.SET_PORT_MTU and port_mtu:\n           cmds +\u003d self._format_commands(\n                self.SET_PORT_MTU, port\u003dport, mtu\u003dport_mtu) \n```","commit_id":"10f0f49d4b071d85a9c2d300450da7df024c60cd"},{"author":{"_account_id":25437,"name":"Allain Legacy","email":"alegacy@redhat.com","username":"alegacy"},"change_message_id":"c57cfb73ca9dce1aad8609230c0508491fe9809d","unresolved":false,"context_lines":[{"line_number":631,"context_line":"                port\u003dport,"},{"line_number":632,"context_line":"                segmentation_id\u003dsegmentation_id)"},{"line_number":633,"context_line":""},{"line_number":634,"context_line":"        if self.SET_PORT_MTU and mtu:"},{"line_number":635,"context_line":"            cmds +\u003d self._format_commands("},{"line_number":636,"context_line":"                self.SET_PORT_MTU, port\u003dport, mtu\u003dmtu)"},{"line_number":637,"context_line":""},{"line_number":638,"context_line":"        return self.send_commands_to_device(cmds)"},{"line_number":639,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"adf353b7_2a9f3d9f","line":636,"range":{"start_line":634,"start_character":8,"end_line":636,"end_character":54},"in_reply_to":"18d1930c_96066710","updated":"2026-06-02 11:31:37.000000000","message":"Done","commit_id":"10f0f49d4b071d85a9c2d300450da7df024c60cd"},{"author":{"_account_id":25437,"name":"Allain Legacy","email":"alegacy@redhat.com","username":"alegacy"},"change_message_id":"fb05a530a2c9cc0cb7d5cde8a985cefce9401de7","unresolved":true,"context_lines":[{"line_number":668,"context_line":"                port\u003dport,"},{"line_number":669,"context_line":"                segmentation_id\u003dport_default_vlan)"},{"line_number":670,"context_line":""},{"line_number":671,"context_line":"        default_mtu \u003d self._get_port_default_mtu()"},{"line_number":672,"context_line":"        if self.SET_PORT_MTU and default_mtu:"},{"line_number":673,"context_line":"            cmds +\u003d self._format_commands("},{"line_number":674,"context_line":"                self.SET_PORT_MTU, port\u003dport, mtu\u003ddefault_mtu)"},{"line_number":675,"context_line":""},{"line_number":676,"context_line":"        if self._disable_inactive_ports() and self.DISABLE_PORT:"},{"line_number":677,"context_line":"            cmds +\u003d self._format_commands(self.DISABLE_PORT, port\u003dport)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ddd5071e_8ac1fb38","line":674,"range":{"start_line":671,"start_character":0,"end_line":674,"end_character":62},"updated":"2026-04-20 15:48:06.000000000","message":"Similar to the comment above in plug_port_to_network, I\u0027d like to see this handle an override in the function args so that here similar to how it was done on line 646. \n\nSomething like:\n\n\n```suggestion\n        default_mtu \u003d mtu or self._get_port_default_mtu()\n        if self.SET_PORT_MTU and default_mtu:\n            cmds +\u003d self._format_commands(\n                self.SET_PORT_MTU, port\u003dport, mtu\u003ddefault_mtu)\n```","commit_id":"10f0f49d4b071d85a9c2d300450da7df024c60cd"},{"author":{"_account_id":25437,"name":"Allain Legacy","email":"alegacy@redhat.com","username":"alegacy"},"change_message_id":"c57cfb73ca9dce1aad8609230c0508491fe9809d","unresolved":false,"context_lines":[{"line_number":668,"context_line":"                port\u003dport,"},{"line_number":669,"context_line":"                segmentation_id\u003dport_default_vlan)"},{"line_number":670,"context_line":""},{"line_number":671,"context_line":"        default_mtu \u003d self._get_port_default_mtu()"},{"line_number":672,"context_line":"        if self.SET_PORT_MTU and default_mtu:"},{"line_number":673,"context_line":"            cmds +\u003d self._format_commands("},{"line_number":674,"context_line":"                self.SET_PORT_MTU, port\u003dport, mtu\u003ddefault_mtu)"},{"line_number":675,"context_line":""},{"line_number":676,"context_line":"        if self._disable_inactive_ports() and self.DISABLE_PORT:"},{"line_number":677,"context_line":"            cmds +\u003d self._format_commands(self.DISABLE_PORT, port\u003dport)"}],"source_content_type":"text/x-python","patch_set":1,"id":"e5712787_16f8a356","line":674,"range":{"start_line":671,"start_character":0,"end_line":674,"end_character":62},"in_reply_to":"ddd5071e_8ac1fb38","updated":"2026-06-02 11:31:37.000000000","message":"Done","commit_id":"10f0f49d4b071d85a9c2d300450da7df024c60cd"},{"author":{"_account_id":25437,"name":"Allain Legacy","email":"alegacy@redhat.com","username":"alegacy"},"change_message_id":"fb05a530a2c9cc0cb7d5cde8a985cefce9401de7","unresolved":true,"context_lines":[{"line_number":711,"context_line":"                bond\u003dbond,"},{"line_number":712,"context_line":"                segmentation_id\u003dsegmentation_id)"},{"line_number":713,"context_line":""},{"line_number":714,"context_line":"        if self.SET_BOND_MTU and mtu:"},{"line_number":715,"context_line":"            cmds +\u003d self._format_commands("},{"line_number":716,"context_line":"                self.SET_BOND_MTU, bond\u003dbond, mtu\u003dmtu)"},{"line_number":717,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"cbea9028_ee0addc7","line":714,"updated":"2026-04-20 15:48:06.000000000","message":"equivalent comment here to what I mentioned in the plug_port_to_network","commit_id":"10f0f49d4b071d85a9c2d300450da7df024c60cd"},{"author":{"_account_id":25437,"name":"Allain Legacy","email":"alegacy@redhat.com","username":"alegacy"},"change_message_id":"c57cfb73ca9dce1aad8609230c0508491fe9809d","unresolved":false,"context_lines":[{"line_number":711,"context_line":"                bond\u003dbond,"},{"line_number":712,"context_line":"                segmentation_id\u003dsegmentation_id)"},{"line_number":713,"context_line":""},{"line_number":714,"context_line":"        if self.SET_BOND_MTU and mtu:"},{"line_number":715,"context_line":"            cmds +\u003d self._format_commands("},{"line_number":716,"context_line":"                self.SET_BOND_MTU, bond\u003dbond, mtu\u003dmtu)"},{"line_number":717,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"6365f21f_2986c4fd","line":714,"in_reply_to":"cbea9028_ee0addc7","updated":"2026-06-02 11:31:37.000000000","message":"Done","commit_id":"10f0f49d4b071d85a9c2d300450da7df024c60cd"},{"author":{"_account_id":25437,"name":"Allain Legacy","email":"alegacy@redhat.com","username":"alegacy"},"change_message_id":"fb05a530a2c9cc0cb7d5cde8a985cefce9401de7","unresolved":true,"context_lines":[{"line_number":753,"context_line":"                bond\u003dbond,"},{"line_number":754,"context_line":"                segmentation_id\u003dport_default_vlan)"},{"line_number":755,"context_line":""},{"line_number":756,"context_line":"        default_mtu \u003d self._get_port_default_mtu()"},{"line_number":757,"context_line":"        if self.SET_BOND_MTU and default_mtu:"},{"line_number":758,"context_line":"            cmds +\u003d self._format_commands("},{"line_number":759,"context_line":"                self.SET_BOND_MTU, bond\u003dbond, mtu\u003ddefault_mtu)"}],"source_content_type":"text/x-python","patch_set":1,"id":"c24403d9_2b5be934","line":756,"range":{"start_line":756,"start_character":0,"end_line":756,"end_character":50},"updated":"2026-04-20 15:48:06.000000000","message":"equivalent comment here to what I mentioned in the delete_port","commit_id":"10f0f49d4b071d85a9c2d300450da7df024c60cd"},{"author":{"_account_id":25437,"name":"Allain Legacy","email":"alegacy@redhat.com","username":"alegacy"},"change_message_id":"c57cfb73ca9dce1aad8609230c0508491fe9809d","unresolved":false,"context_lines":[{"line_number":753,"context_line":"                bond\u003dbond,"},{"line_number":754,"context_line":"                segmentation_id\u003dport_default_vlan)"},{"line_number":755,"context_line":""},{"line_number":756,"context_line":"        default_mtu \u003d self._get_port_default_mtu()"},{"line_number":757,"context_line":"        if self.SET_BOND_MTU and default_mtu:"},{"line_number":758,"context_line":"            cmds +\u003d self._format_commands("},{"line_number":759,"context_line":"                self.SET_BOND_MTU, bond\u003dbond, mtu\u003ddefault_mtu)"}],"source_content_type":"text/x-python","patch_set":1,"id":"89b646bb_ce095ec3","line":756,"range":{"start_line":756,"start_character":0,"end_line":756,"end_character":50},"in_reply_to":"c24403d9_2b5be934","updated":"2026-06-02 11:31:37.000000000","message":"Done","commit_id":"10f0f49d4b071d85a9c2d300450da7df024c60cd"}],"releasenotes/notes/add-mtu-management-e41fefe85b21.yaml":[{"author":{"_account_id":25437,"name":"Allain Legacy","email":"alegacy@redhat.com","username":"alegacy"},"change_message_id":"fb05a530a2c9cc0cb7d5cde8a985cefce9401de7","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    control the behavior. When configured, port MTU is set from the Neutron"},{"line_number":7,"context_line":"    network\u0027s MTU attribute during binding and reset to the default on unbind."},{"line_number":8,"context_line":"    Trunk port MTU is set when VLANs are created, and serves as an upper bound"},{"line_number":9,"context_line":"    for access port MTU validation. MTU commands are supported on Arista EOS,"},{"line_number":10,"context_line":"    ArubaOS-CX, Cisco NX-OS, Cumulus NCLU/NVUE, Dell OS10, Juniper Junos,"},{"line_number":11,"context_line":"    Ruijie, and SONiC. Devices without MTU command templates silently skip"},{"line_number":12,"context_line":"    MTU configuration, preserving backward compatibility."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1263c53a_290651e0","line":9,"range":{"start_line":9,"start_character":53,"end_line":9,"end_character":63},"updated":"2026-04-20 15:48:06.000000000","message":"For BiFrost testing support (and maybe other cases), it would be nice to have OVS supported added.  Something like:\n\n```\n    SET_MTU_PORT \u003d (\n        \u0027ovs-vsctl set interface {port} mtu_request\u003d{mtu}\u0027,\n    )\n```","commit_id":"10f0f49d4b071d85a9c2d300450da7df024c60cd"},{"author":{"_account_id":25437,"name":"Allain Legacy","email":"alegacy@redhat.com","username":"alegacy"},"change_message_id":"c57cfb73ca9dce1aad8609230c0508491fe9809d","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    control the behavior. When configured, port MTU is set from the Neutron"},{"line_number":7,"context_line":"    network\u0027s MTU attribute during binding and reset to the default on unbind."},{"line_number":8,"context_line":"    Trunk port MTU is set when VLANs are created, and serves as an upper bound"},{"line_number":9,"context_line":"    for access port MTU validation. MTU commands are supported on Arista EOS,"},{"line_number":10,"context_line":"    ArubaOS-CX, Cisco NX-OS, Cumulus NCLU/NVUE, Dell OS10, Juniper Junos,"},{"line_number":11,"context_line":"    Ruijie, and SONiC. Devices without MTU command templates silently skip"},{"line_number":12,"context_line":"    MTU configuration, preserving backward compatibility."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ef1901ef_56dedd8e","line":9,"range":{"start_line":9,"start_character":53,"end_line":9,"end_character":63},"in_reply_to":"1263c53a_290651e0","updated":"2026-06-02 11:31:37.000000000","message":"Done","commit_id":"10f0f49d4b071d85a9c2d300450da7df024c60cd"}]}
