Use BaseConfigSchema as the default schema in load_config()

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

Author: llucax

RELEASE_NOTES.md

@@ -30,7 +30,7 @@ This release includes a new `ConfigManager` class to simplify managing the confi
 
     * `load_config()`:
 
-         + The `base_schema` argument is now keyword-only.
+         + The `base_schema` argument is now keyword-only and defaults to `BaseConfigSchema`.
          + The arguments forwarded to `marshmallow.Schema.load()` now must be passed explicitly via the `marshmallow_load_kwargs` argument, as a `dict`, to improve the type-checking.
 
     * `ConfigManagingActor`: Raise a `ValueError` if the `config_files` argument an empty sequence.

src/frequenz/sdk/config/_util.py

@@ -9,6 +9,8 @@
 from marshmallow import Schema
 from marshmallow_dataclass import class_schema
 
+from ._base_schema import BaseConfigSchema
+
 
 # This is a hack that relies on identifying dataclasses by looking into an undocumented
 # property of dataclasses[1], so it might break in the future. Nevertheless, it seems to
@@ -33,7 +35,7 @@ def load_config(
     config: Mapping[str, Any],
     /,
     *,
-    base_schema: type[Schema] | None = None,
+    base_schema: type[Schema] | None = BaseConfigSchema,
     marshmallow_load_kwargs: dict[str, Any] | None = None,
 ) -> DataclassT:
     """Load a configuration from a dictionary into an instance of a configuration class.