feat(appkit): complete MLflow UC tracing across agent lifecycle - #535
Open
adamgurary wants to merge 32 commits into
Open
feat(appkit): complete MLflow UC tracing across agent lifecycle#535adamgurary wants to merge 32 commits into
adamgurary wants to merge 32 commits into
Conversation
added 29 commits
August 11, 2026 15:52
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>
Contributor
Author
|
@MarioCadenas GitHub has queued PR Metadata Verification, Bundle Size, and CI as |
added 3 commits
August 13, 2026 12:44
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>
Contributor
Author
|
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:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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