[HER Hack-Astron # 2] Langfuse integration via OpenTelemetry bridgeFeat/otel langfuse bridge - #1614
[HER Hack-Astron # 2] Langfuse integration via OpenTelemetry bridgeFeat/otel langfuse bridge#1614Karen0758 wants to merge 2 commits into
Conversation
Signed-off-by: yuanli <13564291248@163.com>
Signed-off-by: yuanli <13564291248@163.com>
10c8584 to
0de21f7
Compare
FenjuFu
left a comment
There was a problem hiding this comment.
Thanks for the real Langfuse trace evidence and the OTLP/HTTP integration work. Two blockers remain before approval:
- The current
Check workflowjob is failing because Black would reformatcore/workflow/extensions/otlp/trace/trace.py. Please run the repository formatter and push the result so the CI summary is green. - The added exporter-selection tests cover only
core/common, while this PR also changes the separatecore/workflowexporter and adds node-level observability attributes/span renaming. Please add focused tests for the workflow exporter protocol/endpoint/header selection and for the model/token/branch attributes plus span naming. The real trace proves the integration works, but these tests are needed to protect the two changed runtime paths from regression.
Please update the PR and request re-review after CI passes.
|
感谢你参与 HER Hack-Astron #2 并提交这份 Langfuse 可观测性集成 🙏 本期活动已完成评审。经社区 maintainers 综合评估,Langfuse 集成最终选定 #1607 作为合并实现,因此本 PR 不再单独合入,现予关闭 —— 这不代表你的方案质量不足,只是同一能力我们只保留一条实现路径,避免重复维护。 评选结果、获奖名单与领奖方式请见: 你在 OpenTelemetry 桥接、Trace 语义和隐私边界上的思路对最终方案很有参考价值。欢迎继续关注后续的 HER Hack-Astron 活动,也欢迎在其他 issue 上继续贡献 🚀 Thank you for taking part in HER Hack-Astron #2 and submitting this Langfuse observability integration 🙏 Judging for this round is complete. After review by the community maintainers, #1607 was selected as the implementation to merge for Langfuse integration, so this PR is being closed rather than merged. This is not a reflection on the quality of your work — we're keeping a single implementation path for this capability to avoid duplicated maintenance. Results, the award list, and how to claim your prize are here: Your work on the OpenTelemetry bridge, trace semantics and privacy boundaries fed usefully into the final design. We'd love to see you in the next HER Hack-Astron round, and contributions on other issues are always welcome 🚀 |
AI Assistance Disclosure
This PR was developed with AI assistance.
新春游故宫20260811165759demo workflow used for validation was provided by another team member. @Tomatoes0814Closes #1575
Summary
This PR adds Langfuse-compatible tracing support through Astron's existing OpenTelemetry integration. It keeps OTLP/gRPC as the default behavior and adds OTLP/HTTP support for backends such as Langfuse Cloud, without introducing the Langfuse SDK into Astron's agent runtime.
Changes
OTEL_EXPORTER_OTLP_PROTOCOLOTEL_EXPORTER_OTLP_ENDPOINTOTEL_EXPORTER_OTLP_HEADERSVerification
Verified with a real local Astron workflow:
新春游故宫20260811165759. Screenshots are attached below.a0e518210dae343d9696142d4dd8d705/workflow/v1/debug/chat/completions6.78srun_node:spark-llm:大模型_2:deepseek-v4-flashdeepseek-v4-flash117 prompt -> 124 completion (Σ 241)2.15s