Skip to content

HDDS-13679. Design doc for OpenTelemetry integration#9051

Merged
sumitagrawl merged 17 commits intoapache:masterfrom
sumitagrawl:HDDS-13679
Oct 16, 2025
Merged

HDDS-13679. Design doc for OpenTelemetry integration#9051
sumitagrawl merged 17 commits intoapache:masterfrom
sumitagrawl:HDDS-13679

Conversation

@sumitagrawl
Copy link
Contributor

What changes were proposed in this pull request?

Distributed tracing via Open telemetry and improvement design doc

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-13679

How was this patch tested?

  • NA

@sumitagrawl sumitagrawl changed the title HDDS-13679: distributed tracing open telemetry improvement HDDS-13679. distributed tracing open telemetry improvement Sep 19, 2025
@sumitagrawl sumitagrawl requested a review from Copilot September 22, 2025 15:51
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 introduces a design document for migrating Ozone's distributed tracing from the deprecated OpenTracing with Jaeger to OpenTelemetry. The document outlines the migration strategy, implementation details, and improvements to the tracing hierarchy to provide better end-to-end visibility across Ozone components.

Key changes:

  • Addition of comprehensive OpenTelemetry design documentation
  • Migration strategy from deprecated OpenTracing to OpenTelemetry
  • Enhanced tracing hierarchy design for better flow visibility

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@jojochuang jojochuang left a comment

Choose a reason for hiding this comment

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

Suggest to change the subject to "Design doc for OpenTelemetry integration"

@sumitagrawl sumitagrawl marked this pull request as ready for review October 14, 2025 09:22
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

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


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sumitagrawl sumitagrawl changed the title HDDS-13679. distributed tracing open telemetry improvement HDDS-13679. Design doc for OpenTelemetry integration Oct 14, 2025
@sumitagrawl sumitagrawl requested a review from Copilot October 14, 2025 09:27
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

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


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sumitagrawl sumitagrawl requested a review from Copilot October 14, 2025 09:29
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

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


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@devmadhuu devmadhuu left a comment

Choose a reason for hiding this comment

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

Thanks @sumitagrawl for this comprehensive design doc. LGTM +1

@sumitagrawl sumitagrawl merged commit 5fdf2d2 into apache:master Oct 16, 2025
15 checks passed
chungen0126 pushed a commit to chungen0126/ozone that referenced this pull request Oct 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants