)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"28b88303b1860ce1ce87dab83405a0c3ab7daca7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e33d7604_7470c385","updated":"2026-07-15 19:41:31.000000000","message":"Looks OK to me","commit_id":"0ac130c9be77e1567fca76bce296bb511ea7178e"}],"vif_plug_ovs/ovs.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"816acc6e5fac93f2025fbfe3381878d1cc123e93","unresolved":true,"context_lines":[{"line_number":226,"context_line":""},{"line_number":227,"context_line":"    def _get_mtu(self, vif: _OVSVif) -\u003e int:"},{"line_number":228,"context_line":"        network \u003d self._get_vif_network(vif)"},{"line_number":229,"context_line":"        if \u0027mtu\u0027 in network and network.mtu:"},{"line_number":230,"context_line":"            return network.mtu"},{"line_number":231,"context_line":"        # oslo.config is untyped"},{"line_number":232,"context_line":"        return cast(int, self.config.network_device_mtu)"}],"source_content_type":"text/x-python","patch_set":1,"id":"f5b3ca5e_99fc38c9","line":229,"updated":"2026-07-15 06:56:44.000000000","message":"Aren\u0027t we missing here a check? `if network`\n```\nif network and \u0027mtu\u0027 in network and network.mtu:\n```","commit_id":"0ac130c9be77e1567fca76bce296bb511ea7178e"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6996908c255b8d7a4267570fbd5b5191ed086a52","unresolved":false,"context_lines":[{"line_number":226,"context_line":""},{"line_number":227,"context_line":"    def _get_mtu(self, vif: _OVSVif) -\u003e int:"},{"line_number":228,"context_line":"        network \u003d self._get_vif_network(vif)"},{"line_number":229,"context_line":"        if \u0027mtu\u0027 in network and network.mtu:"},{"line_number":230,"context_line":"            return network.mtu"},{"line_number":231,"context_line":"        # oslo.config is untyped"},{"line_number":232,"context_line":"        return cast(int, self.config.network_device_mtu)"}],"source_content_type":"text/x-python","patch_set":1,"id":"a35f30f5_aa240f68","line":229,"in_reply_to":"f5b3ca5e_99fc38c9","updated":"2026-07-15 06:57:41.000000000","message":"Ok, no, `_get_vif_network` raises an exception in that case.\n```\n    if network is None:\n      raise ValueError(\u0027VIF network is required\u0027)\n```","commit_id":"0ac130c9be77e1567fca76bce296bb511ea7178e"}]}