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
requested a deployment
to
manual-approval
GitHub Actions
Waiting
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? |
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.