From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 9E50945DAF; Tue, 26 Nov 2024 23:18:51 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8B8D1402A7; Tue, 26 Nov 2024 23:18:51 +0100 (CET) Received: from mail-pg1-f179.google.com (mail-pg1-f179.google.com [209.85.215.179]) by mails.dpdk.org (Postfix) with ESMTP id 721F6402A7 for ; Tue, 26 Nov 2024 23:18:50 +0100 (CET) Received: by mail-pg1-f179.google.com with SMTP id 41be03b00d2f7-7fbce800ee5so4224078a12.2 for ; Tue, 26 Nov 2024 14:18:50 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1732659529; x=1733264329; darn=dpdk.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=S7dANRVSrTb+rHb2PYA9BFJEd4AYeCM2/q3aW/SBkjs=; b=cxzFBVIkDJdGRQuZtE2tXcOh09eQSI/CCKm2VngV1ElutWiBFqJyc8RWOmszgakVfU WIBJZM0FEzELuLuwpDurbTnkEcSG4ldQ+/V6P7VK3M+gxujrV7jEYtWmg+7GAiTMjAde JWXB5g7iX6m/mJPOCMvjpm/hIFQn7/OhYasgQ= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732659529; x=1733264329; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=S7dANRVSrTb+rHb2PYA9BFJEd4AYeCM2/q3aW/SBkjs=; b=dhYd/xypsqzdML8lfOeuqiOUQ0woLE+p6jkJO6VibWp7QrrGpKWhJUoJUrTDM7XEZx 3hUOKoz+lEXFLg4lsWX7zUGLwI89dSkBjYyFQSwe5LsPnsIsumzlNFXgk6EKzqOyK8pD Xl60X8J7lMDAjTPUi3kfG1F9q5O8ohOi2FKD6Ly6xPVLH2lFpX0BneHFAvEG9oMqjOjd a5PSnK4XJt2jBlOq40Kpfr6BCNnEZaDSh54SF2AE90CWZkse2DrlpOUH4YP//pjCjwHj yNrpYy1De3R4LY5ewG6eEo/CIzhCx7Sq5zr3pPGTq/Zeyyax7JzzqDnazONe81ShXlu3 eXjw== X-Gm-Message-State: AOJu0YyeqRhiayHSmggd4PUeICwxh6v2zAN5tKhQQXW5h+meedw/Qt9/ gjhTkxCWIvc3dSVYV8CyQfS5VdKVEUeepAtrf2uUuQJzA1TE768z0p9pEJcYpBm1exVystA4xQe pd8byBkAFZKZSi/xgjsDYK0cuFDUwLYfa5XvEqw== X-Gm-Gg: ASbGncv7nmY/H38efwIMorhPDd0suk9ywL9n30f8lOvJovEZsNN85ZEkcPgjZdYulpG OpZXntLS+oCWgq6OOGevptphpBVSv1s61o2zM4iS5X0COPhVyh/wc16ES+mmy2cTl X-Google-Smtp-Source: AGHT+IFOrnJuPGKrMZUUBRV7LlKRjT+DXEqThIbz8v5u5zGETBzv5ko1BiOaVgROTCh5sNlo/ezG3j/yw29ILKTwTT4= X-Received: by 2002:a17:90b:35c9:b0:2ea:4a6b:79d1 with SMTP id 98e67ed59e1d1-2ee08eb2bdfmr1142485a91.11.1732659529638; Tue, 26 Nov 2024 14:18:49 -0800 (PST) MIME-Version: 1.0 References: <20241126150929.2843601-1-luca.vizzarro@arm.com> In-Reply-To: <20241126150929.2843601-1-luca.vizzarro@arm.com> From: Patrick Robb Date: Tue, 26 Nov 2024 17:16:42 -0500 Message-ID: Subject: Re: [PATCH] dts: remove leftover Node methods To: Luca Vizzarro Cc: dev@dpdk.org, Paul Szczepanek Content-Type: multipart/alternative; boundary="000000000000c805520627d83b88" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org --000000000000c805520627d83b88 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Applied to next-dts, thanks. On Tue, Nov 26, 2024 at 10:09=E2=80=AFAM Luca Vizzarro wrote: > The "remove redundant test suite" removed an unused test suite and some > dead code with it. Some dead code which references now-removed symbols, > remained though. This removes this code, therefore fixing the related > mypy errors. > > Fixes: e3ab9dd5cd5d ("dts: remove redundant test suite") > > Signed-off-by: Luca Vizzarro > Reviewed-by: Paul Szczepanek > --- > dts/framework/testbed_model/node.py | 26 -------------------------- > 1 file changed, 26 deletions(-) > > diff --git a/dts/framework/testbed_model/node.py > b/dts/framework/testbed_model/node.py > index 85144f6f4e..c1844ecd5d 100644 > --- a/dts/framework/testbed_model/node.py > +++ b/dts/framework/testbed_model/node.py > @@ -14,8 +14,6 @@ > """ > > from abc import ABC > -from ipaddress import IPv4Interface, IPv6Interface > -from typing import Union > > from framework.config import ( > OS, > @@ -192,30 +190,6 @@ def _setup_hugepages(self) -> None: > self.config.hugepages.force_first_numa, > ) > > - def configure_port_state(self, port: Port, enable: bool =3D True) -> > None: > - """Enable/disable `port`. > - > - Args: > - port: The port to enable/disable. > - enable: :data:`True` to enable, :data:`False` to disable. > - """ > - self.main_session.configure_port_state(port, enable) > - > - def configure_port_ip_address( > - self, > - address: Union[IPv4Interface, IPv6Interface], > - port: Port, > - delete: bool =3D False, > - ) -> None: > - """Add an IP address to `port` on this node. > - > - Args: > - address: The IP address with mask in CIDR format. Can be > either IPv4 or IPv6. > - port: The port to which to add the address. > - delete: If :data:`True`, will delete the address from the > port instead of adding it. > - """ > - self.main_session.configure_port_ip_address(address, port, delet= e) > - > def close(self) -> None: > """Close all connections and free other resources.""" > if self.main_session: > -- > 2.43.0 > > --000000000000c805520627d83b88 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Applied to next-dts, thanks.

On Tue, Nov 26, 2024 at 10:09= =E2=80=AFAM Luca Vizzarro <luca= .vizzarro@arm.com> wrote:
The "remove redundant test suite" removed an unu= sed test suite and some
dead code with it. Some dead code which references now-removed symbols,
remained though. This removes this code, therefore fixing the related
mypy errors.

Fixes: e3ab9dd5cd5d ("dts: remove redundant test suite")

Signed-off-by: Luca Vizzarro <luca.vizzarro@arm.com>
Reviewed-by: Paul Szczepanek <paul.szczepanek@arm.com>
---
=C2=A0dts/framework/testbed_model/node.py | 26 -------------------------- =C2=A01 file changed, 26 deletions(-)

diff --git a/dts/framework/testbed_model/node.py b/dts/framework/testbed_mo= del/node.py
index 85144f6f4e..c1844ecd5d 100644
--- a/dts/framework/testbed_model/node.py
+++ b/dts/framework/testbed_model/node.py
@@ -14,8 +14,6 @@
=C2=A0"""

=C2=A0from abc import ABC
-from ipaddress import IPv4Interface, IPv6Interface
-from typing import Union

=C2=A0from framework.config import (
=C2=A0 =C2=A0 =C2=A0OS,
@@ -192,30 +190,6 @@ def _setup_hugepages(self) -> None:
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0self.config.h= ugepages.force_first_numa,
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0)

-=C2=A0 =C2=A0 def configure_port_state(self, port: Port, enable: bool =3D = True) -> None:
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 """Enable/disable `port`.
-
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 Args:
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 port: The port to enable/disable= .
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 enable: :data:`True` to enable, = :data:`False` to disable.
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 """
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 self.main_session.configure_port_state(port, e= nable)
-
-=C2=A0 =C2=A0 def configure_port_ip_address(
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 self,
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 address: Union[IPv4Interface, IPv6Interface],<= br> -=C2=A0 =C2=A0 =C2=A0 =C2=A0 port: Port,
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 delete: bool =3D False,
-=C2=A0 =C2=A0 ) -> None:
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 """Add an IP address to `port` = on this node.
-
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 Args:
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 address: The IP address with mas= k in CIDR format. Can be either IPv4 or IPv6.
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 port: The port to which to add t= he address.
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 delete: If :data:`True`, will de= lete the address from the port instead of adding it.
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 """
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 self.main_session.configure_port_ip_address(ad= dress, port, delete)
-
=C2=A0 =C2=A0 =C2=A0def close(self) -> None:
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0"""Close all connections a= nd free other resources."""
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0if self.main_session:
--
2.43.0

--000000000000c805520627d83b88--