Skip to content

Move Micrometer instrumentation to this project #501

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
bclozel opened this issue Oct 6, 2022 · 0 comments
Closed

Move Micrometer instrumentation to this project #501

bclozel opened this issue Oct 6, 2022 · 0 comments
Assignees
Labels
in: core Issues related to config and core support type: enhancement A general enhancement
Milestone

Comments

@bclozel
Copy link
Member

bclozel commented Oct 6, 2022

With the new Micrometer 1.10 Observation API, we need to move the existing metrics instrumentation from Spring Boot to this project and use the Observation API instead. This needs to happen before #469.

@bclozel bclozel added type: enhancement A general enhancement in: core Issues related to config and core support labels Oct 6, 2022
@bclozel bclozel added this to the 1.1.0-RC1 milestone Oct 6, 2022
@bclozel bclozel self-assigned this Oct 6, 2022
bclozel added a commit to spring-projects/spring-boot that referenced this issue Oct 19, 2022
This commit migrates the existing metrics support (added in #29140) to
the new `Observation` instrumentation contributed in
spring-projects/spring-graphql#501.

We cannot have a smoother migration path here as the instrumentation
does not use the same context information for metadata extraction.

Closes gh-32794
koenpunt pushed a commit to koenpunt/spring-graphql that referenced this issue Feb 2, 2023
Prior to this commit, metrics instrumentation was provided by Spring
Boot, using Micrometer metrics. The `Instrumentation` would publish two
timers and a counter, but would not support tracing.

This commit replaces the former with a dedicated support in Spring for
GraphQL directly. This uses the new `Observation` API from Micrometer
and publishes two observations:

* a request execution observation, with timing and tracing included
* a data fetching observation with the request execution as a parent
  observation

Closes spring-projectsgh-501
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: core Issues related to config and core support type: enhancement A general enhancement
Projects
None yet
Development

No branches or pull requests

1 participant