The DPDKBuildInfo and NodeInfo classes, representing information
gathered in runtime, were erroneously placed in the configuration
package. This moves them in more appropriate modules.
NodeInfo, specifically, is moved to os_session instead of node mostly
as a consequence of circular dependencies. And given os_session is the
top-most module to reference it, it appears to be the most suitable
place outside of node.
Finally NodeInfo, is better renamed to OSSessionInfo as it represents
the information on the target OS session.
Signed-off-by: Luca Vizzarro <luca.vizzarro@arm.com>
Reviewed-by: Paul Szczepanek <paul.szczepanek@arm.com>
Reviewed-by: Nicholas Pratte <npratte@iol.unh.edu>
Reviewed-by: Patrick Robb <probb@iol.unh.edu>
---
dts/framework/config/__init__.py | 31 --------------------
dts/framework/test_result.py | 6 ++--
dts/framework/testbed_model/os_session.py | 23 +++++++++++++--
dts/framework/testbed_model/posix_session.py | 8 ++---
dts/framework/testbed_model/sut_node.py | 22 ++++++++++----
5 files changed, 46 insertions(+), 44 deletions(-)
@@ -318,24 +318,6 @@ class TGNodeConfiguration(NodeConfiguration):
traffic_generator: TrafficGeneratorConfig
-@dataclass(slots=True, frozen=True)
-class NodeInfo:
- """Supplemental node information.
-
- Attributes:
- os_name: The name of the running operating system of
- the :class:`~framework.testbed_model.node.Node`.
- os_version: The version of the running operating system of
- the :class:`~framework.testbed_model.node.Node`.
- kernel_version: The kernel version of the running operating system of
- the :class:`~framework.testbed_model.node.Node`.
- """
-
- os_name: str
- os_version: str
- kernel_version: str
-
-
@dataclass(slots=True, frozen=True)
class DPDKBuildConfiguration:
"""DPDK build configuration.
@@ -493,19 +475,6 @@ def from_dict(cls, d: DPDKConfigurationDict) -> Self:
)
-@dataclass(slots=True, frozen=True)
-class DPDKBuildInfo:
- """Various versions and other information about a DPDK build.
-
- Attributes:
- dpdk_version: The DPDK version that was built.
- compiler_version: The version of the compiler used to build DPDK.
- """
-
- dpdk_version: str | None
- compiler_version: str | None
-
-
@dataclass(slots=True, frozen=True)
class TestSuiteConfig:
"""Test suite configuration.
@@ -30,11 +30,13 @@
from framework.testbed_model.capability import Capability
-from .config import DPDKBuildInfo, NodeInfo, TestRunConfiguration, TestSuiteConfig
+from .config import TestRunConfiguration, TestSuiteConfig
from .exception import DTSError, ErrorSeverity
from .logger import DTSLogger
from .settings import SETTINGS
from .test_suite import TestCase, TestSuite
+from .testbed_model.os_session import OSSessionInfo
+from .testbed_model.sut_node import DPDKBuildInfo
@dataclass(slots=True, frozen=True)
@@ -421,7 +423,7 @@ def test_suites_with_cases(self, test_suites_with_cases: list[TestSuiteWithCases
)
self._test_suites_with_cases = test_suites_with_cases
- def add_sut_info(self, sut_info: NodeInfo) -> None:
+ def add_sut_info(self, sut_info: OSSessionInfo) -> None:
"""Add SUT information gathered at runtime.
Args:
@@ -24,11 +24,12 @@
"""
from abc import ABC, abstractmethod
from collections.abc import Iterable
+from dataclasses import dataclass
from ipaddress import IPv4Interface, IPv6Interface
from pathlib import Path, PurePath, PurePosixPath
from typing import Union
-from framework.config import Architecture, NodeConfiguration, NodeInfo
+from framework.config import Architecture, NodeConfiguration
from framework.logger import DTSLogger
from framework.remote_session import (
InteractiveRemoteSession,
@@ -44,6 +45,24 @@
from .port import Port
+@dataclass(slots=True, frozen=True)
+class OSSessionInfo:
+ """Supplemental OS session information.
+
+ Attributes:
+ os_name: The name of the running operating system of
+ the :class:`~framework.testbed_model.node.Node`.
+ os_version: The version of the running operating system of
+ the :class:`~framework.testbed_model.node.Node`.
+ kernel_version: The kernel version of the running operating system of
+ the :class:`~framework.testbed_model.node.Node`.
+ """
+
+ os_name: str
+ os_version: str
+ kernel_version: str
+
+
class OSSession(ABC):
"""OS-unaware to OS-aware translation API definition.
@@ -482,7 +501,7 @@ def get_compiler_version(self, compiler_name: str) -> str:
"""
@abstractmethod
- def get_node_info(self) -> NodeInfo:
+ def get_node_info(self) -> OSSessionInfo:
"""Collect additional information about the node.
Returns:
@@ -15,7 +15,7 @@
from collections.abc import Iterable
from pathlib import Path, PurePath, PurePosixPath
-from framework.config import Architecture, NodeInfo
+from framework.config import Architecture
from framework.exception import DPDKBuildError, RemoteCommandExecutionError
from framework.settings import SETTINGS
from framework.utils import (
@@ -26,7 +26,7 @@
extract_tarball,
)
-from .os_session import OSSession
+from .os_session import OSSession, OSSessionInfo
class PosixSession(OSSession):
@@ -386,11 +386,11 @@ def get_compiler_version(self, compiler_name: str) -> str:
case _:
raise ValueError(f"Unknown compiler {compiler_name}")
- def get_node_info(self) -> NodeInfo:
+ def get_node_info(self) -> OSSessionInfo:
"""Overrides :meth:`~.os_session.OSSession.get_node_info`."""
os_release_info = self.send_command(
"awk -F= '$1 ~ /^NAME$|^VERSION$/ {print $2}' /etc/os-release",
SETTINGS.timeout,
).stdout.split("\n")
kernel_version = self.send_command("uname -r", SETTINGS.timeout).stdout
- return NodeInfo(os_release_info[0].strip(), os_release_info[1].strip(), kernel_version)
+ return OSSessionInfo(os_release_info[0].strip(), os_release_info[1].strip(), kernel_version)
@@ -14,13 +14,12 @@
import os
import time
+from dataclasses import dataclass
from pathlib import PurePath
from framework.config import (
DPDKBuildConfiguration,
- DPDKBuildInfo,
DPDKLocation,
- NodeInfo,
SutNodeConfiguration,
TestRunConfiguration,
)
@@ -30,10 +29,23 @@
from framework.utils import MesonArgs, TarCompressionFormat
from .node import Node
-from .os_session import OSSession
+from .os_session import OSSession, OSSessionInfo
from .virtual_device import VirtualDevice
+@dataclass(slots=True, frozen=True)
+class DPDKBuildInfo:
+ """Various versions and other information about a DPDK build.
+
+ Attributes:
+ dpdk_version: The DPDK version that was built.
+ compiler_version: The version of the compiler used to build DPDK.
+ """
+
+ dpdk_version: str | None
+ compiler_version: str | None
+
+
class SutNode(Node):
"""The system under test node.
@@ -63,7 +75,7 @@ class SutNode(Node):
_app_compile_timeout: float
_dpdk_kill_session: OSSession | None
_dpdk_version: str | None
- _node_info: NodeInfo | None
+ _node_info: OSSessionInfo | None
_compiler_version: str | None
_path_to_devbind_script: PurePath | None
_ports_bound_to_dpdk: bool
@@ -125,7 +137,7 @@ def dpdk_version(self) -> str | None:
return self._dpdk_version
@property
- def node_info(self) -> NodeInfo:
+ def node_info(self) -> OSSessionInfo:
"""Additional node information."""
if self._node_info is None:
self._node_info = self.main_session.get_node_info()