Uh oh!
There was an error while loading. Please reload this page.
feat(appkit): complete MLflow UC tracing across agent lifecycle - #535
feat(appkit): complete MLflow UC tracing across agent lifecycle#535adamgurary wants to merge 32 commits into
Conversation
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
…tracing-post-supervisor Signed-off-by: Adam Gurary <adam.gurary@databricks.com> # Conflicts: # packages/appkit/src/plugins/agents/agents.ts # packages/appkit/src/plugins/agents/tests/route-handler-errors.test.ts
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
adamgurary
commented
Aug 13, 2026
@MarioCadenas GitHub has queued PR Metadata Verification, Bundle Size, and CI as |
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
Signed-off-by: Adam Gurary <adam.gurary@databricks.com>
adamgurary
commented
Aug 13, 2026
Ran the repo-local /review-pr origin/main command repeatedly against the full committed branch and addressed the substantiated findings. Review fixes now included:
One review recommendation was intentionally not adopted: agents still enable UC tracing by default and missing or invalid UC configuration still fails startup. That is the approved fail-fast requirement for this contribution, so tracing cannot silently disappear from an agent deployment. Verification:
|
Summary
This continues and supersedes the tracing architecture introduced in #477. It keeps one AppKit OpenTelemetry provider and attaches MLflow/Unity Catalog processing to that provider, instead of creating a second
mlflow-tracingprovider. The existing run-linking behavior is preserved through the semanticmlflow.sourceRunroot attribute.Companion rollout across executable templates: databricks/app-templates#260
The change makes MLflow tracing a first-class AppKit agent lifecycle contract:
AGENT,CHAT_MODEL,TOOL,RETRIEVER,MEMORY,CHAIN,PARSER, and nested-agent spans across agent routes, adapters, tools, AI Search, MCP, model serving, caches, parsers, and Supervisor integrations;Architecture relative to #477
#477 introduced plugin-local MLflow tracing backed by a separate
mlflow-tracingprovider. This PR folds that intent into AppKit's existing telemetry system: one provider owns the span lifecycle, while the UC processor/exporter handles MLflow persistence. This avoids split or orphaned trace trees and lets every AppKit agent surface share the same propagation, serialization, usage, cost, and error semantics.Verification
pnpm check: passed (89 existing warnings)Live-environment limitations