feat: add health endpoints on separate port

- Health endpoints now run on port 8001 (main port + 1)
- Added separate FastAPI server for health checks alongside FastMCP
- Updated documentation and Makefile to reflect correct ports
- Health endpoints: /health, /health/ready, /health/live, /metrics
- Added uvicorn dependency for health server

Endpoints available:
- Main MCP server: http://localhost:8000
- Health checks: http://localhost:8001/health
- Metrics: http://localhost:8001/metrics

Author: T-rav

Makefile

@@ -130,15 +130,15 @@ prometheus-open:
 
 health-check:
 	@echo "🏥 Checking service health..."
-	@curl -s http://localhost:8000/health | jq . || echo "Service not responding"
+	@curl -s http://localhost:8001/health | jq . || echo "Service not responding"
 
 metrics-check:
 	@echo "📊 Checking metrics..."
-	@curl -s http://localhost:8000/metrics | head -20
+	@curl -s http://localhost:8001/metrics | head -20
 
 observability-status:
 	@echo "📊 Observability Stack Status:"
-	@echo "Weather MCP: $$(curl -s -o /dev/null -w "%%{http_code}" http://localhost:8000/health || echo "DOWN")"
+	@echo "Weather MCP: $$(curl -s -o /dev/null -w "%%{http_code}" http://localhost:8001/health || echo "DOWN")"
 	@echo "Prometheus: $$(curl -s -o /dev/null -w "%%{http_code}" http://localhost:9091/-/healthy || echo "DOWN")"
 	@echo "Grafana: $$(curl -s -o /dev/null -w "%%{http_code}" http://localhost:3000/api/health || echo "DOWN")"
 	@echo "Loki: $$(curl -s -o /dev/null -w "%%{http_code}" http://localhost:3100/ready || echo "DOWN")"

OBSERVABILITY.md

@@ -32,8 +32,8 @@ make observability-up
 - **Grafana Dashboard**: http://localhost:3000 (admin/admin)
 - **Prometheus**: http://localhost:9091
 - **AlertManager**: http://localhost:9093
-- **Health Check**: http://localhost:8000/health
-- **Metrics Endpoint**: http://localhost:8000/metrics
+- **Health Check**: http://localhost:8001/health (runs on port +1)
+- **Metrics Endpoint**: http://localhost:8001/metrics (runs on port +1)
 
 ### 3. Quick Health Check
 

pyproject.toml

@@ -41,6 +41,7 @@ dependencies = [
     "cachetools>=5.3.0",
     "loguru>=0.7.0",
     "typer>=0.9.0",
+    "uvicorn>=0.30.0",
     # Observability dependencies
     "prometheus-client>=0.19.0",
     "opentelemetry-api>=1.21.0",

weather_mcp/cli.py

@@ -53,9 +53,12 @@ def run(
     host: str = typer.Option("0.0.0.0", help="Server host address"),
     port: int = typer.Option(8000, help="Server port"),
 ):
-    """Run weather server with SSE endpoints"""
+    """Run weather server with SSE endpoints and health checks"""
+    import threading
+
     from fastmcp import FastMCP
 
+    from .health import create_health_app
     from .nws import NationalWeatherServiceClient
 
     async def run_server():
@@ -75,10 +78,25 @@ async def run_server():
 
         return mcp_server
 
-    # Initialize and run server
+    def run_health_server():
+        """Run health check server on a separate port"""
+        import uvicorn
+
+        health_app = create_health_app()
+        logger.info(f"Starting health server on port {port + 1}")
+        uvicorn.run(health_app, host=host, port=port + 1, log_level="warning")
+
+    # Start health server in background thread
+    health_thread = threading.Thread(target=run_health_server, daemon=True)
+    health_thread.start()
+
+    # Initialize and run main MCP server
     mcp_server = asyncio.run(run_server())
 
     # Run FastMCP server with SSE transport
+    logger.info(f"Weather MCP Server: http://{host}:{port}")
+    logger.info(f"Health endpoints: http://{host}:{port + 1}/health")
+    logger.info(f"Metrics endpoint: http://{host}:{port + 1}/metrics")
     mcp_server.run(transport="sse", host=host, port=port)