fix(logs): Limit log length and check to ensure service is running before printing logs

devservices/commands/logs.py

@@ -5,10 +5,12 @@
 from argparse import ArgumentParser
 from argparse import Namespace
 
+from devservices.constants import MAX_LOG_LINES
 from devservices.exceptions import DependencyError
 from devservices.exceptions import DockerComposeError
 from devservices.utils.docker_compose import run_docker_compose_command
 from devservices.utils.services import find_matching_service
+from devservices.utils.state import State
 
 
 def add_parser(subparsers: _SubParsersAction[ArgumentParser]) -> None:
@@ -36,8 +38,16 @@ def logs(args: Namespace) -> None:
     mode_to_use = "default"
     mode_dependencies = modes[mode_to_use]
 
+    state = State()
+    running_services = state.get_started_services()
+    if service_name not in running_services:
+        print(f"Service {service_name} is not running")
+        return
+
     try:
-        logs_output = run_docker_compose_command(service, "logs", mode_dependencies)
+        logs_output = run_docker_compose_command(
+            service, "logs", mode_dependencies, options=["-n", MAX_LOG_LINES]
+        )
     except DependencyError as de:
         print(str(de))
         exit(1)

devservices/constants.py

@@ -23,3 +23,4 @@
 DOCKER_COMPOSE_DOWNLOAD_URL = "https://github.com/docker/compose/releases/download"
 DEVSERVICES_DOWNLOAD_URL = "https://github.com/getsentry/devservices/releases/download"
 BINARY_PERMISSIONS = 0o755
+MAX_LOG_LINES = "100"