Add MLflow to tracing options#343
Conversation
Signed-off-by: B-Step62 <yuki.watanabe@databricks.com>
cagataycali
left a comment
cagataycali
left a comment
There was a problem hiding this comment.
Review Summary
This PR adds MLflow to the list of tracing options in the observability documentation. MLflow is a first-party MLOps service in SageMaker AI, making this a valuable addition.
✅ What's Good
- Minimal, focused change - Only 5 additions and 4 deletions, very low risk
- Accurate documentation - Links to official MLflow docs for Strands integration: https://mlflow.org/docs/latest/genai/tracing/integrations/listing/strands/
- AWS ecosystem alignment - MLflow integration with SageMaker AI makes this particularly relevant
📝 Notes
- This PR has been open since November 2025 - it's a straightforward addition that could be merged to improve documentation completeness
- CI is passing and there are no conflicts
Status
- CI: ✅ Passing
- Mergeable: ✅ Yes
- Conflicts: None
This looks ready to merge. The change is minimal, accurate, and adds value for users who want MLflow tracing.
Review by strands-coder autonomous agent 🤖
|
Hi @yonib05 @cagataycali, is there any change I should make on this PR? Otherwise it would be great if you can merge this, thank you so much! |
B-Step62
had a problem deploying
to
manual-approval
GitHub Actions
Failure
B-Step62
temporarily deployed
to
manual-approval
GitHub Actions
Inactive
|
Hi @zastrowm, @JackYPCOnline, I resolved the conflict. Could you help merging this PR? |
Documentation Preview ReadyYour documentation preview has been successfully deployed! Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-cms-343/docs/user-guide/quickstart/overview/ Updated at: 2026-04-09T22:01:04.794Z |
… inline links Per zastrowm's review: Langfuse stays at strands-agents#2 (used internally), MLflow moves to strands-agents#3, and inline links removed for consistency with other list items.
ryanycoleman
temporarily deployed
to
manual-approval
GitHub Actions
Inactive
ryanycoleman
had a problem deploying
to
manual-approval
GitHub Actions
Failure
ryanycoleman
dismissed
zastrowm’s stale review
Addressed feedback in my latest commit
ryanycoleman
left a comment
ryanycoleman
left a comment
There was a problem hiding this comment.
Addressed feedback to get this merge-ready. Thank you for the contribution!
|
@ryanycoleman Thank you so much for making the change to address commits! It seems CI is blocked, can you help unblocking it and merging? |
|
This repository has been merged into the strands-agents/harness-sdk monorepo and will be archived shortly. All new development happens there. If this PR is still relevant, please recreate it against the monorepo. The code now lives under Apologies for the disruption, and thank you for contributing! |
6 participants
Description
MLflow support tracing integration for Strands Agents: https://mlflow.org/docs/latest/genai/tracing/integrations/listing/strands/
This PR adds it to the list of tracing options.
Type of Change
Content update/revision
Motivation and Context
MLflow is first-partiy MLOps service in SageMaker AI: https://aws.amazon.com/jp/sagemaker/ai/experiments/.
Areas Affected
Tracing
Screenshots
na
Checklist
mkdocs serveAdditional Notes
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.