From cb01bf2986daa8d932d33525393588da871cde68 Mon Sep 17 00:00:00 2001 From: Bowen Liang Date: Mon, 29 Jul 2024 13:40:18 +0800 Subject: [PATCH] chore: set logging level to debug when reading YAML files and falling back to default value in case of None (#6758) --- api/core/tools/utils/yaml_utils.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/api/core/tools/utils/yaml_utils.py b/api/core/tools/utils/yaml_utils.py index 11486da7da..21155a6960 100644 --- a/api/core/tools/utils/yaml_utils.py +++ b/api/core/tools/utils/yaml_utils.py @@ -12,7 +12,7 @@ def load_yaml_file(file_path: str, ignore_error: bool = True, default_value: Any Safe loading a YAML file :param file_path: the path of the YAML file :param ignore_error: - if True, return default_value if error occurs and the error will be logged in warning level + if True, return default_value if error occurs and the error will be logged in debug level if False, raise error if error occurs :param default_value: the value returned when errors ignored :return: an object of the YAML content @@ -20,12 +20,13 @@ def load_yaml_file(file_path: str, ignore_error: bool = True, default_value: Any try: with open(file_path, encoding='utf-8') as yaml_file: try: - return yaml.safe_load(yaml_file) + yaml_content = yaml.safe_load(yaml_file) + return yaml_content if yaml_content else default_value except Exception as e: raise YAMLError(f'Failed to load YAML file {file_path}: {e}') except Exception as e: if ignore_error: - logger.warning(f'Failed to load YAML file {file_path}: {e}') + logger.debug(f'Failed to load YAML file {file_path}: {e}') return default_value else: raise e