Port 4484 change ocean sail default log level (#69)

* changed default log level and fixed config bug

changelog/1.bugfix.md

@@ -0,0 +1 @@
+Fixed an issue with loading the configuration from the environment variables if the config is a dictionary

changelog/1.improvement.md

@@ -0,0 +1 @@
+Changed default log level to INFO in the cli

port_ocean/cli/commands/sail.py

@@ -13,11 +13,11 @@
     "--log-level",
     "log_level",
     type=click.Choice(["DEBUG", "INFO", "WARNING", "ERROR", "CRITICAL"]),
-    default="DEBUG",
+    default="INFO",
     help="""Set the logging level for the integration.
             Supported levels are DEBUG, INFO, WARNING, ERROR,
             and CRITICAL. If not specified, the default level
-            is DEBUG.""",
+            is INFO.""",
 )
 @click.option(
     "-p",

port_ocean/config/base.py

@@ -53,15 +53,27 @@ def load_from_config_provider(config_provider: str) -> Any:
 
 
 def parse_providers(
+    settings_model: BaseModel | ModelMetaclass,
     config: dict[str, Any],
     existing_data: dict[str, Any],
 ) -> dict[str, Any]:
     """
     Normalizing the config yaml file to work with snake_case and getting only the data that is missing for the settings
     """
     for key, value in config.items():
-        if isinstance(value, dict):
-            existing_data[key] = parse_providers(value, existing_data.get(key, {}))
+        if isinstance(value, dict) and settings_model is not None:
+            # If the value is of type ModelMetaClass then its a nested model, and we need to parse it
+            # If the value is of type primitive dict then we need to decamelize the keys and not recurse into the values because its no longer part of the model
+            _type = settings_model.__annotations__[key]
+            is_primitive_dict_type = _type is dict or (
+                isinstance(_type, GenericAlias) and _type.__origin__ is dict
+            )
+
+            if is_primitive_dict_type:
+                _type = None
+            existing_data[key] = parse_providers(
+                _type, value, existing_data.get(key, {})
+            )
 
         elif isinstance(value, str):
             # If the value is a provider, we try to load it from the provider
@@ -114,7 +126,7 @@ def load_providers(
     yaml_content = read_yaml_config_settings_source(settings, base_path)
     data = yaml.safe_load(yaml_content)
     snake_case_config = decamelize_config(settings, data)
-    return parse_providers(snake_case_config, existing_values)
+    return parse_providers(settings, snake_case_config, existing_values)
 
 
 class BaseOceanSettings(BaseSettings):

port_ocean/run.py

@@ -59,7 +59,7 @@ def _create_default_app(path: str | None = None) -> Ocean:
 
 def run(
     path: str = ".",
-    log_level: LogLevelType = "DEBUG",
+    log_level: LogLevelType = "INFO",
     port: int = 8000,
     initialize_port_resources: bool | None = None,
 ) -> None: