Skip to content

Python: Remove duplicated workflow observability sample#2357

Merged
TaoChenOSU merged 2 commits intomicrosoft:mainfrom
TaoChenOSU:taochen/python-remove-duplicate-workflow-observability-sample
Nov 20, 2025
Merged

Python: Remove duplicated workflow observability sample#2357
TaoChenOSU merged 2 commits intomicrosoft:mainfrom
TaoChenOSU:taochen/python-remove-duplicate-workflow-observability-sample

Conversation

@TaoChenOSU
Copy link
Contributor

@TaoChenOSU TaoChenOSU commented Nov 20, 2025

Motivation and Context

Closes #2184

Description

  1. Remove the workflow observability sample folder under getting_started/workflows ad point to the one under getting_started/observability

Documentation will be updated in the docs repo.

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

@TaoChenOSU TaoChenOSU self-assigned this Nov 20, 2025
Copilot AI review requested due to automatic review settings November 20, 2025 19:30
@TaoChenOSU TaoChenOSU added the samples Issue relates to the samples label Nov 20, 2025
@github-actions github-actions bot changed the title Remove duplicated workflow observability sample Python: Remove duplicated workflow observability sample Nov 20, 2025
@markwallace-microsoft markwallace-microsoft added the documentation Improvements or additions to documentation label Nov 20, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR consolidates duplicate observability samples by removing the workflow-specific observability sample and directing users to the centralized observability examples directory.

  • Removes python/samples/getting_started/workflows/observability/tracing_basics.py
  • Updates workflows README to reference the main observability samples directory
  • Enhances documentation in workflow_observability.py with detailed workflow description and telemetry span information

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
python/samples/getting_started/workflows/observability/tracing_basics.py Deleted duplicate observability sample previously located in workflows directory
python/samples/getting_started/workflows/README.md Updated observability section to redirect users to the main observability samples directory with proper links
python/samples/getting_started/observability/workflow_observability.py Enhanced documentation with detailed workflow description, telemetry span categories, and prerequisites

@TaoChenOSU TaoChenOSU added this pull request to the merge queue Nov 20, 2025
Merged via the queue into microsoft:main with commit 02af2bc Nov 20, 2025
23 checks passed
arisng pushed a commit to arisng/agent-framework that referenced this pull request Feb 2, 2026
* Remove duplicated workflow observability sample

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

Labels

documentation Improvements or additions to documentation python samples Issue relates to the samples

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Python: Centralize Observability Documentation and Code Samples

5 participants