use simplified default logger

examples/airflow-torch-training/callables.py

@@ -1,11 +1,10 @@
-import logging
-
 import runhouse as rh
+from runhouse.logger import get_logger
 
 from torch_example_for_airflow import DownloadData, SimpleTrainer
 
-# Configure logging
-logger = logging.getLogger(__name__)
+
+logger = get_logger(name=__name__)
 
 
 def bring_up_cluster_callable(**kwargs):

runhouse/logger.py

@@ -1,148 +1,25 @@
 import logging
-import logging.config
-import re
-from datetime import datetime, timezone
-from typing import List, Union
+import os
 
-from runhouse.constants import DEFAULT_LOG_LEVEL
 
+def get_logger(name: str = __name__):
+    logger = logging.getLogger(name)
 
-class ColoredFormatter:
-    COLORS = {
-        "black": "\u001b[30m",
-        "red": "\u001b[31m",
-        "green": "\u001b[32m",
-        "yellow": "\u001b[33m",
-        "blue": "\u001b[34m",
-        "magenta": "\u001b[35m",
-        "cyan": "\u001b[36m",
-        "white": "\u001b[37m",
-        "reset": "\u001b[0m",
-    }
+    level = os.getenv("RH_LOG_LEVEL")
+    if level:
+        # Set the logging level
+        logger.setLevel(level.upper())
 
-    @classmethod
-    def get_color(cls, color: str):
-        return cls.COLORS.get(color, "")
-
-    # TODO: This method is a temp solution, until we'll update logging architecture. Remove once logging is cleaned up.
-    @classmethod
-    def format_log(cls, text):
-        ansi_escape = re.compile(r"(?:\x1B[@-_][0-?]*[ -/]*[@-~])")
-        return ansi_escape.sub("", text)
-
-
-class ClusterLogsFormatter:
-    def __init__(self, system):
-        self.system = system
-        self._display_title = False
-
-    def format(self, output_type):
-        from runhouse import Resource
-        from runhouse.servers.http.http_utils import OutputType
-
-        system_color = ColoredFormatter.get_color("cyan")
-        reset_color = ColoredFormatter.get_color("reset")
-
-        prettify_logs = output_type in [
-            OutputType.STDOUT,
-            OutputType.EXCEPTION,
-            OutputType.STDERR,
-        ]
-
-        if (
-            isinstance(self.system, Resource)
-            and prettify_logs
-            and not self._display_title
-        ):
-            # Display the system name before subsequent logs only once
-            system_name = self.system.name
-            dotted_line = "-" * len(system_name)
-            print(dotted_line)
-            print(f"{system_color}{system_name}{reset_color}")
-            print(dotted_line)
-
-            # Only display the system name once
-            self._display_title = True
-
-        return system_color, reset_color
-
-
-class FunctionLogHandler(logging.Handler):
-    def __init__(self):
-        super().__init__()
-        self.log_records = []
-
-    def emit(self, record):
-        self.log_records.append(record)
-
-    @staticmethod
-    def log_records_to_stdout(log_records: List[logging.LogRecord]) -> str:
-        """Convert the log records to a string repr of the stdout output"""
-        captured_logs = [
-            f"{log_record.levelname} | {log_record.asctime} | {log_record.msg}"
-            for log_record in log_records
-        ]
-        return "\n".join(captured_logs)
-
-
-class UTCFormatter(logging.Formatter):
-    """Ensure logs are always in UTC time"""
-
-    @staticmethod
-    def converter(timestamp):
-        return datetime.fromtimestamp(timestamp, tz=timezone.utc)
+    # Apply a custom formatter
+    formatter = logging.Formatter(
+        fmt="%(levelname)s | %(asctime)s | %(message)s", datefmt="%Y-%m-%d %H:%M:%S"
+    )
 
-    def formatTime(self, record, datefmt=None):
-        dt = self.converter(record.created)
-        if datefmt:
-            return dt.strftime(datefmt)
-        else:
-            return dt.isoformat(timespec="milliseconds")
+    # Apply the formatter to each handler
+    for handler in logger.handlers:
+        handler.setFormatter(formatter)
 
+    # Prevent the logger from propagating to the root logger
+    logger.propagate = False
 
-def get_logger(name: str = __name__, log_level: Union[int, str] = logging.INFO):
-    level_name = (
-        logging.getLevelName(log_level) if isinstance(log_level, int) else log_level
-    )
-    LOGGING_CONFIG = {
-        "version": 1,
-        "disable_existing_loggers": True,
-        "formatters": {
-            "utc_formatter": {
-                "()": UTCFormatter,
-                "format": "%(levelname)s | %(asctime)s | %(message)s",
-                "datefmt": "%Y-%m-%d %H:%M:%S.%f",
-            },
-        },
-        "handlers": {
-            "default": {
-                "level": level_name,
-                "formatter": "utc_formatter",
-                "class": "logging.StreamHandler",
-                "stream": "ext://sys.stderr",  # Default is stderr
-            },
-        },
-        "loggers": {
-            "": {  # root logger
-                "handlers": ["default"],
-                "level": level_name,
-                "propagate": False,
-            },
-            "my.packg": {
-                "handlers": ["default"],
-                "level": level_name,
-                "propagate": False,
-            },
-            "__main__": {  # if __name__ == '__main__'
-                "handlers": ["default"],
-                "level": level_name,
-                "propagate": False,
-            },
-        },
-    }
-    logging.config.dictConfig(LOGGING_CONFIG)
-    logger = logging.getLogger(name=name)
     return logger
-
-
-logger = get_logger(name=__name__, log_level=DEFAULT_LOG_LEVEL)

runhouse/main.py

@@ -34,18 +34,21 @@
     START_SCREEN_CMD,
 )
 from runhouse.globals import obj_store, rns_client
-from runhouse.logger import logger
+from runhouse.logger import get_logger
 from runhouse.resources.hardware.ray_utils import (
     check_for_existing_ray_instance,
     kill_actors,
 )
 
+
 # create an explicit Typer application
 app = typer.Typer(add_completion=False)
 
 # For printing with typer
 console = Console()
 
+logger = get_logger(name=__name__)
+
 
 @app.command()
 def login(

runhouse/resources/envs/conda_env.py

@@ -7,14 +7,15 @@
 
 from runhouse.constants import ENVS_DIR
 from runhouse.globals import obj_store
+from runhouse.logger import get_logger
 
-from runhouse.logger import logger
 from runhouse.resources.envs.utils import install_conda, run_setup_command
-
 from runhouse.resources.packages import Package
 
 from .env import Env
 
+logger = get_logger(name=__name__)
+
 
 class CondaEnv(Env):
     RESOURCE_TYPE = "env"

runhouse/resources/envs/env.py

@@ -1,19 +1,20 @@
 import copy
-import logging
 import os
 import shlex
 from pathlib import Path
 from typing import Dict, List, Optional, Union
 
 from runhouse.globals import obj_store
+from runhouse.logger import get_logger
 
-from runhouse.logger import logger
 from runhouse.resources.envs.utils import _process_env_vars, run_setup_command
 from runhouse.resources.hardware import _get_cluster_from, Cluster
 from runhouse.resources.packages import InstallTarget, Package
 from runhouse.resources.resource import Resource
 from runhouse.utils import run_with_logs
 
+logger = get_logger(name=__name__)
+
 
 class Env(Resource):
     RESOURCE_TYPE = "env"

runhouse/resources/folders/folder.py

@@ -7,13 +7,15 @@
 from typing import List, Optional, Union
 
 from runhouse.globals import rns_client
+from runhouse.logger import get_logger
 
-from runhouse.logger import logger
 from runhouse.resources.hardware import _current_cluster, _get_cluster_from, Cluster
 from runhouse.resources.resource import Resource
 from runhouse.rns.utils.api import generate_uuid, relative_file_path
 from runhouse.utils import locate_working_dir
 
+logger = get_logger(name=__name__)
+
 
 class Folder(Resource):
     RESOURCE_TYPE = "folder"

runhouse/resources/folders/folder_factory.py

@@ -1,11 +1,13 @@
 from pathlib import Path
 from typing import Optional, Union
 
-from runhouse.logger import logger
+from runhouse.logger import get_logger
 
 from runhouse.resources.folders.folder import Folder
 from runhouse.resources.hardware.utils import _get_cluster_from
 
+logger = get_logger(name=__name__)
+
 
 def folder(
     name: Optional[str] = None,

runhouse/resources/folders/gcs_folder.py

@@ -4,10 +4,12 @@
 from pathlib import Path
 from typing import List, Optional
 
-from runhouse.logger import logger
+from runhouse.logger import get_logger
 
 from .folder import Folder
 
+logger = get_logger(name=__name__)
+
 
 class GCSFolder(Folder):
     RESOURCE_TYPE = "folder"

runhouse/resources/folders/s3_folder.py

@@ -5,14 +5,16 @@
 from pathlib import Path
 from typing import List, Optional
 
-from runhouse.logger import logger
+from runhouse.logger import get_logger
 
 from .folder import Folder
 
 MAX_POLLS = 120000
 POLL_INTERVAL = 1
 TIMEOUT_SECONDS = 3600
 
+logger = get_logger(name=__name__)
+
 
 class S3Folder(Folder):
     RESOURCE_TYPE = "folder"

runhouse/resources/functions/aws_lambda.py

@@ -17,16 +17,16 @@
     pass
 
 from runhouse.globals import rns_client
-
-from runhouse.logger import logger
+from runhouse.logger import get_logger
 from runhouse.resources.envs import _get_env_from, Env
-
 from runhouse.resources.functions.function import Function
 
 
 CRED_PATH = f"{Path.home()}/.aws/credentials"
 LOG_GROUP_PREFIX = "/aws/lambda/"
 
+logger = get_logger(name=__name__)
+
 
 class LambdaFunction(Function):
     RESOURCE_TYPE = "lambda_function"

runhouse/resources/functions/function.py

@@ -1,17 +1,18 @@
 import inspect
-import logging
 from pathlib import Path
 from typing import Any, List, Optional, Tuple, Union
 
 from runhouse import globals
+from runhouse.logger import get_logger
 
-from runhouse.logger import logger
 from runhouse.resources.envs import Env
 from runhouse.resources.hardware import Cluster
 from runhouse.resources.module import Module
 
 from runhouse.resources.resource import Resource
 
+logger = get_logger(name=__name__)
+
 
 class Function(Module):
     RESOURCE_TYPE = "function"

runhouse/resources/functions/function_factory.py

@@ -2,12 +2,14 @@
 from pathlib import Path
 from typing import Callable, List, Optional, Union
 
-from runhouse.logger import logger
+from runhouse.logger import get_logger
 
 from runhouse.resources.envs import _get_env_from, Env
 from runhouse.resources.functions.function import Function
 from runhouse.resources.packages import git_package
 
+logger = get_logger(name=__name__)
+
 
 def function(
     fn: Optional[Union[str, Callable]] = None,

runhouse/resources/hardware/cluster.py

@@ -43,8 +43,8 @@
     RESERVED_SYSTEM_NAMES,
 )
 from runhouse.globals import configs, obj_store, rns_client
+from runhouse.logger import get_logger
 
-from runhouse.logger import logger
 from runhouse.resources.envs.utils import _get_env_from
 from runhouse.resources.hardware.utils import (
     _current_cluster,
@@ -56,6 +56,8 @@
 
 from runhouse.servers.http import HTTPClient
 
+logger = get_logger(name=__name__)
+
 
 class Cluster(Resource):
     RESOURCE_TYPE = "cluster"