Skip to content

Add support for custom GrpcServerObservationConvention#49730

Merged
snicoll merged 2 commits intospring-projects:mainfrom
therepanic:apply-keepalive-settings
Mar 25, 2026
Merged

Add support for custom GrpcServerObservationConvention#49730
snicoll merged 2 commits intospring-projects:mainfrom
therepanic:apply-keepalive-settings

Conversation

@therepanic
Copy link
Contributor

In this commit, we make GrpcServerObservationAutoConfiguration and GrpcClientObservationAutoConfiguration consistent by supporting a custom GrpcClientObservationConvention and adding the @ConditionalOnMissingBean annotation to the
observationGrpcServerInterceptor bean.

Original PR: spring-projects/spring-grpc#390

@therepanic therepanic changed the title Support custom GrpcServerObservationConvention in GrpcServerObservationAutoConfiguration Support custom GrpcServerObservationConvention in GrpcServerObservationAutoConfiguration Mar 24, 2026
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Mar 24, 2026
@snicoll snicoll changed the title Support custom GrpcServerObservationConvention in GrpcServerObservationAutoConfiguration Support custom GrpcServerObservationConvention in GrpcServerObservationAutoConfiguration Mar 25, 2026
@snicoll snicoll self-assigned this Mar 25, 2026
therepanic and others added 2 commits March 25, 2026 10:40
In this commit, we make `GrpcServerObservationAutoConfiguration` and
`GrpcClientObservationAutoConfiguration` consistent by supporting a
custom `GrpcClientObservationConvention` and adding the
`@ConditionalOnMissingBean` annotation to the
`observationGrpcServerInterceptor` bean.

See spring-projectsgh-49730

Signed-off-by: Andrey Litvitski <andrey1010102008@gmail.com>
@snicoll snicoll changed the title Support custom GrpcServerObservationConvention in GrpcServerObservationAutoConfiguration Add support for custom GrpcServerObservationConvention Mar 25, 2026
@snicoll snicoll added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Mar 25, 2026
@snicoll snicoll added this to the 4.1.0-M4 milestone Mar 25, 2026
@snicoll snicoll force-pushed the apply-keepalive-settings branch from 1396450 to 3ebc865 Compare March 25, 2026 09:54
@snicoll snicoll merged commit 3898a88 into spring-projects:main Mar 25, 2026
2 of 3 checks passed
snicoll added a commit to therepanic/spring-boot that referenced this pull request Mar 26, 2026
Such parameters are usually named "customConvention" so this commit
aligns to that naming.

See spring-projectsgh-49730
snicoll added a commit that referenced this pull request Mar 26, 2026
* polish-GrpcClientObservationAutoConfiguration:
  Harmonize ObservationConvention parameters in gRPC support
  Rename GrpcClientObservationConvention parameter to convention

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

Labels

type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants