Skip to content

Commit 3984945

Browse files
committed
test(lfx): install the otel extra in the dev group
The lfx test job runs `uv sync --dev` in src/lfx, which resolves the lfx member alone and so uninstalls the opentelemetry packages the full langflow install happens to provide. test_serve_app_flow_span's subprocess probe then failed with ModuleNotFoundError, and it passed locally only because langflow was in the venv. Depend on the extra itself rather than copying its six pins, so the two cannot drift.
1 parent 47dbddb commit 3984945

2 files changed

Lines changed: 6 additions & 0 deletions

File tree

src/lfx/pyproject.toml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,10 @@ dev = [
140140
"coverage>=7.9.2",
141141
"hypothesis>=6.136.3",
142142
"langchain-anthropic~=1.4.6",
143+
# The otel extra itself, so the observability tests exercise the real exporters. Bare lfx
144+
# still ships without them: the tests that assert the no-otel path force the condition by
145+
# blocking the import rather than by relying on the extra being absent here.
146+
"lfx[otel]",
143147
"pytest>=9.0.3",
144148
"pytest-asyncio>=0.26.0",
145149
"pytest-cov>=7.0.0",

uv.lock

Lines changed: 2 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)