From 9ecbaf120e4cff66a473e4cf303ac155f9ca5859 Mon Sep 17 00:00:00 2001 From: Yuneng Jiang Date: Fri, 2 Oct 2026 16:03:44 -0700 Subject: [PATCH] test(integration): opt the config pass-through spend-log case into auth The failure spend-log row from #42695 is written for pass-through routes that run as LLM API routes, which a config route only does with auth: true. The test omitted auth and passed only while config wins (#41779) registered config entries through the typed model, where auth defaults to true. #43962 restored the pre-config-wins registration, so the route lost that status and the row was never written. Set auth: true on the route so the test covers the logging it was written for without depending on that side effect (cherry picked from commit 1d9cd9b09c0825ef8359a703e2fc59998752b290) --- .../observability/test_passthrough_upstream_error_visibility.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/integration/observability/test_passthrough_upstream_error_visibility.py b/tests/integration/observability/test_passthrough_upstream_error_visibility.py index bb18add2f2ff..c29ce4f1c624 100644 --- a/tests/integration/observability/test_passthrough_upstream_error_visibility.py +++ b/tests/integration/observability/test_passthrough_upstream_error_visibility.py @@ -238,6 +238,7 @@ def respond(request: Request) -> Reply: "target": f"{wire.url}/upstream?trace=secret-q", "include_subpath": True, "headers": {"Authorization": "Bearer scripted"}, + "auth": True, } ] path.write_text(yaml.safe_dump(config))