diff --git a/evaluation/utils/shared.py b/evaluation/utils/shared.py index 2981e55799..db8f976904 100644 --- a/evaluation/utils/shared.py +++ b/evaluation/utils/shared.py @@ -375,18 +375,27 @@ def reset_logger_for_multiprocessing( # Remove all existing handlers from logger for handler in logger.handlers[:]: logger.removeHandler(handler) - # add back the console handler to print ONE line - logger.addHandler(get_console_handler()) + + # add console handler to print ONE line + console_handler = get_console_handler(log_level=logging.INFO) + console_handler.setFormatter( + logging.Formatter( + f'Instance {instance_id} - ' + '%(asctime)s - %(levelname)s - %(message)s' + ) + ) + logger.addHandler(console_handler) logger.info( f'Starting evaluation for instance {instance_id}.\n' f'Hint: run "tail -f {log_file}" to see live logs in a separate shell' ) - # Remove all existing handlers from logger - for handler in logger.handlers[:]: - logger.removeHandler(handler) + # Only log WARNING or higher to console + console_handler.setLevel(logging.WARNING) + + # Log INFO and above to file os.makedirs(os.path.dirname(log_file), exist_ok=True) file_handler = logging.FileHandler(log_file) file_handler.setFormatter( logging.Formatter('%(asctime)s - %(levelname)s - %(message)s') ) + file_handler.setLevel(logging.INFO) logger.addHandler(file_handler) diff --git a/openhands/runtime/remote/runtime.py b/openhands/runtime/remote/runtime.py index 9cc0ebe7ba..3e9a80ad20 100644 --- a/openhands/runtime/remote/runtime.py +++ b/openhands/runtime/remote/runtime.py @@ -61,7 +61,7 @@ class RemoteRuntime(Runtime): self.config = config if self.config.sandbox.api_hostname == 'localhost': self.config.sandbox.api_hostname = 'api.all-hands.dev/v0/runtime' - logger.warning( + logger.info( 'Using localhost as the API hostname is not supported in the RemoteRuntime. Please set a proper hostname.\n' 'Setting it to default value: api.all-hands.dev/v0/runtime' )