Remove the LoggingConfig.load method

We now have `frequenz.sdk.config.load_config()` for this.

Signed-off-by: Leandro Lucarella <[email protected]>

Author: llucax

RELEASE_NOTES.md

@@ -11,6 +11,8 @@
     * Renamed to `LoggingConfigUpdatingActor` to follow the actor naming convention.
     * Make all arguments to the constructor keyword-only.
 
+- The `LoggingConfig.load` method was removed. Please use `frequenz.sdk.config.load_config()` instead.
+
 ## New Features
 
 - `LoggingConfigUpdatingActor`

src/frequenz/sdk/config/_logging_actor.py

@@ -6,16 +6,17 @@
 import logging
 from collections.abc import Mapping
 from dataclasses import field
-from typing import Annotated, Any, Self, cast
+from typing import Annotated, Any
 
 import marshmallow
 import marshmallow.validate
 from frequenz.channels import Receiver
-from marshmallow import RAISE
-from marshmallow_dataclass import class_schema, dataclass
+from marshmallow_dataclass import dataclass
 
 from frequenz.sdk.actor import Actor
 
+from ._util import load_config
+
 _logger = logging.getLogger(__name__)
 
 LogLevel = Annotated[
@@ -69,22 +70,6 @@ class LoggingConfig:
     )
     """The list of loggers configurations."""
 
-    @classmethod
-    def load(cls, configs: Mapping[str, Any]) -> Self:  # noqa: DOC502
-        """Load and validate configs from a dictionary.
-
-        Args:
-            configs: The configuration to validate.
-
-        Returns:
-            The configuration if they are valid.
-
-        Raises:
-            ValidationError: if the configuration are invalid.
-        """
-        schema = class_schema(cls)()
-        return cast(Self, schema.load(configs, unknown=RAISE))
-
 
 class LoggingConfigUpdatingActor(Actor):
     """Actor that listens for logging configuration changes and sets them.
@@ -174,7 +159,7 @@ async def _run(self) -> None:
         """Listen for configuration changes and update logging."""
         async for message in self._config_recv:
             try:
-                new_config = LoggingConfig.load(message)
+                new_config = load_config(LoggingConfig, message)
             except marshmallow.ValidationError:
                 _logger.exception(
                     "Invalid logging configuration received. Skipping config update"

tests/config/test_logging_actor.py

@@ -13,7 +13,12 @@
 from marshmallow import ValidationError
 from pytest_mock import MockerFixture
 
-from frequenz.sdk.config import LoggerConfig, LoggingConfig, LoggingConfigUpdatingActor
+from frequenz.sdk.config import (
+    LoggerConfig,
+    LoggingConfig,
+    LoggingConfigUpdatingActor,
+    load_config,
+)
 
 
 def test_logging_config() -> None:
@@ -33,19 +38,19 @@ def test_logging_config() -> None:
         },
     )
 
-    assert LoggingConfig.load(config_raw) == config
+    assert load_config(LoggingConfig, config_raw) == config
 
     config_raw = {}
     config = LoggingConfig()
-    assert LoggingConfig.load(config_raw) == config
+    assert load_config(LoggingConfig, config_raw) == config
 
     config_raw = {"root_logger": {"level": "UNKNOWN"}}
     with pytest.raises(ValidationError):
-        LoggingConfig.load(config_raw)
+        load_config(LoggingConfig, config_raw)
 
     config_raw = {"unknown": {"frequenz.sdk.actor": {"level": "DEBUG"}}}
     with pytest.raises(ValidationError):
-        LoggingConfig.load(config_raw)
+        load_config(LoggingConfig, config_raw)
 
 
 @pytest.fixture