Skip to content

Conversation

@B-Step62
Copy link

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

  • I have read the CONTRIBUTING document
  • My changes follow the project's documentation style
  • I have tested the documentation locally using mkdocs serve
  • Links in the documentation are valid and working
  • Images/diagrams are properly sized and formatted
  • All new and existing tests pass

Additional Notes

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Signed-off-by: B-Step62 <yuki.watanabe@databricks.com>
yonib05
yonib05 previously approved these changes Dec 1, 2025
@yonib05 yonib05 enabled auto-merge (squash) December 1, 2025 03:39
Copy link
Member

@cagataycali cagataycali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

📝 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 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants