Skip to content

feat(otel): remove span creation for saa operations that do not currently propagate tracing headers.#1492

Open
cconstable wants to merge 3 commits intomainfrom
otel-saa-consistency
Open

feat(otel): remove span creation for saa operations that do not currently propagate tracing headers.#1492
cconstable wants to merge 3 commits intomainfrom
otel-saa-consistency

Conversation

@cconstable
Copy link
Copy Markdown
Contributor

@cconstable cconstable commented Apr 29, 2026 •
edited
Loading

What was changed

  • Removed otel tracing for cancel, terminate, describe, and count activities.
  • Improved existing SAA start activity otel tests to check for span attributes.

Why?

  • We want to be creating consistent otel spans across SDK implementations and removing these spans achieves that.

Checklist

  1. How was this tested: tests

@cconstable cconstable force-pushed the otel-saa-consistency branch from 631f689 to 26bbe38 Compare April 29, 2026 21:03
@cconstable cconstable force-pushed the otel-saa-consistency branch from 26bbe38 to dae220c Compare April 29, 2026 21:09
@cconstable cconstable marked this pull request as ready for review April 29, 2026 21:10
@cconstable cconstable requested a review from a team as a code owner April 29, 2026 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tconley1428 tconley1428 tconley1428 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cconstable @tconley1428