Skip to content

Conversation

@alexliahushau
Copy link
Contributor

@alexliahushau alexliahushau commented Jun 2, 2025

This PR introduces auto-configuration for OpenTelemetry HTTP retry attributes, providing seamless integration and enhanced observability for HTTP client retry behavior in Spring-based applications.

Summary of Changes:

  • Enhanced OpenTelemetry plugin factory to include RetrySpanDecorator into a list of pre-configured decorators when client retry logic is enabled.

Why is this change necessary?

This feature simplifies the process of enabling HTTP retry observability for Spring applications by leveraging Spring Boot's auto-configuration mechanism. Developers using the Spring ecosystem can now gain actionable insights into HTTP retry behavior without requiring manual setup. This contributes to better debugging, performance monitoring, and alignment with OpenTelemetry standards.

Related Issue:

@fatroom
Copy link
Member

fatroom commented Jun 3, 2025

Hi Alex, can you give more details on what is the ISSUE-1660?

@fatroom fatroom added the minor Minor changes, e.g. low risk config updates, changes that do not introduce a new API call. label Jun 3, 2025
@alexliahushau
Copy link
Contributor Author

Hi Alex, can you give more details on what is the ISSUE-1660?
My bad I forgot to add a link to the issue
please check this one ISSUE-1600

@alexliahushau alexliahushau marked this pull request as draft June 4, 2025 10:48
@alexliahushau alexliahushau marked this pull request as ready for review June 4, 2025 10:59
@fatroom
Copy link
Member

fatroom commented Jun 10, 2025

👍

1 similar comment
@alexliahushau
Copy link
Contributor Author

👍

@fatroom fatroom merged commit b4657a6 into zalando:main Jun 10, 2025
4 checks passed
@fatroom fatroom added this to the 4.4.0 milestone Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Minor changes, e.g. low risk config updates, changes that do not introduce a new API call.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants