Downgrade load_yaml_file log messages to debug

Callers already log at their own preferred level (error or warning)
when load_yaml_file returns None. Having load_yaml_file also log at
error produced double logging for the same failure.

Downgrade the messages in load_yaml_file to debug so that callers
remain in control of the log level. This matters because not all
callers treat a failure the same way: get_cloud_password has a
fallback to secure.yml, so a missing secrets.yml is not an error
there.

An alternative would be to raise specific exceptions from
load_yaml_file instead of returning None so that callers can
distinguish file-not-found, decryption errors, and YAML parse
errors. That would require a larger refactoring of all call sites.

AI-assisted: Claude Code
Signed-off-by: Roger Luethi <luethi@osism.tech>

Author: ideaship

osism/tasks/conductor/utils.py

@@ -94,7 +94,7 @@ def get_vault():
 def load_yaml_file(path):
     """Load a YAML file and only request the vault secret when needed."""
     if not os.path.exists(path):
-        logger.error(f"YAML file not found: {path}")
+        logger.debug(f"YAML file not found: {path}")
         return None
 
     try:
@@ -110,10 +110,10 @@ def load_yaml_file(path):
 
         return yaml.safe_load(decrypted_data)
     except yaml.YAMLError as exc:
-        logger.error(f"Failed to parse YAML file {path}: {exc}")
+        logger.debug(f"Failed to parse YAML file {path}: {exc}")
         return None
     except Exception as exc:
-        logger.error(f"Failed to load YAML file {path}: {exc}")
+        logger.debug(f"Failed to load YAML file {path}: {exc}")
         return None