feat: Hide agent-card access log, remove /health

We use the agent-card as health endpoint and this one should not polute the logs.

Author: g3force

adk/agenticlayer/agent_to_a2a.py

@@ -6,6 +6,7 @@
 from a2a.server.apps import A2AStarletteApplication
 from a2a.server.request_handlers import DefaultRequestHandler
 from a2a.server.tasks import InMemoryTaskStore
+from a2a.utils.constants import AGENT_CARD_WELL_KNOWN_PATH
 from google.adk.a2a.executor.a2a_agent_executor import A2aAgentExecutor
 from google.adk.a2a.utils.agent_card_builder import AgentCardBuilder
 from google.adk.agents.base_agent import BaseAgent
@@ -17,13 +18,16 @@
 from google.adk.sessions.in_memory_session_service import InMemorySessionService
 from opentelemetry.instrumentation.starlette import StarletteInstrumentor
 from starlette.applications import Starlette
-from starlette.requests import Request
-from starlette.responses import JSONResponse
-from starlette.routing import Route
 
 from .callback_tracer_plugin import CallbackTracerPlugin
 
 
+class HealthCheckFilter(logging.Filter):
+    def filter(self, record: logging.LogRecord) -> bool:
+        # Check if the log message contains the well known path of the card, which is used for health checks
+        return record.getMessage().find(AGENT_CARD_WELL_KNOWN_PATH) == -1
+
+
 def to_a2a(agent: BaseAgent) -> Starlette:
     """Convert an ADK agent to a A2A Starlette application.
     This is an adaption of google.adk.a2a.utils.agent_to_a2a.
@@ -44,6 +48,10 @@ def to_a2a(agent: BaseAgent) -> Starlette:
     setup_adk_logger(log_level)  # type: ignore
     logger = logging.getLogger(__name__)
 
+    # Filter out health check logs from uvicorn access logger
+    uvicorn_access_logger = logging.getLogger("uvicorn.access")
+    uvicorn_access_logger.addFilter(HealthCheckFilter())
+
     async def create_runner() -> Runner:
         """Create a runner for the agent."""
         return Runner(
@@ -65,10 +73,6 @@ async def create_runner() -> Runner:
 
     request_handler = DefaultRequestHandler(agent_executor=agent_executor, task_store=task_store)
 
-    # Add a simple health check endpoint for readiness/liveness probes
-    def health(_: Request) -> JSONResponse:
-        return JSONResponse(content={"status": "healthy"})
-
     # Get the agent card URL from environment variable *only*
     # At this point, we don't know the applications port and the host is unknown when running in k8s or similar
     # A2A_AGENT_CARD_URL is deprecated but still supported for backwards compatibility
@@ -100,7 +104,7 @@ async def lifespan(app: Starlette) -> AsyncIterator[None]:
         yield
 
     # Create a Starlette app that will be configured during startup
-    starlette_app = Starlette(lifespan=lifespan, routes=[Route("/health", health)])
+    starlette_app = Starlette(lifespan=lifespan)
 
     # Instrument the Starlette app with OpenTelemetry
     StarletteInstrumentor().instrument_app(starlette_app)

adk/tests/test_a2a_starlette.py

@@ -23,13 +23,6 @@ def client(self, starlette_app: Starlette) -> TestClient:
         """Create a test client."""
         return TestClient(starlette_app)
 
-    def test_health_endpoint(self, client: TestClient) -> None:
-        """Test that the health check endpoint works."""
-        response = client.get("/health")
-        assert response.status_code == 200
-        data = response.json()
-        assert data["status"] == "healthy"
-
     def test_agent_card_endpoint(self, starlette_app: Starlette, client: TestClient) -> None:
         """Test that the agent card is available at /.well-known/agent-card.json"""