Skip to content

restclient observability - #169

Merged
henrybley merged 1 commit into
mainfrom
tom/#155-restclient-observability
May 29, 2024
Merged

henrybley merged 1 commit into
mainfrom
tom/#155-restclient-observability

Conversation

@tomellm

@tomellm tomellm commented May 29, 2024

Copy link
Copy Markdown
Contributor

#155 All RestClients now expose some metrics on the actuator endpoint. The exposed metrics values have not been modified in any way from the default. The only thing that changed is that the uri path tag has been modified, removing the path variables to decrease the cardinality of the metrics.

All RestClients now expose some metrics on the actuator endpoint. The exposed metrics
values have not been modified in any way from the default. The only thing that changed
is that the uri path tag has been modified, removing the path variables to decrease the
cardinality of the metrics.
@tomellm
tomellm force-pushed the tom/#155-restclient-observability branch from 1f9ff88 to 039ad2e Compare May 29, 2024 07:54
@sonarqubecloud

Copy link
Copy Markdown

@henrybley henrybley left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@henrybley
henrybley merged commit dfcdd3c into main May 29, 2024
@henrybley
henrybley deleted the tom/#155-restclient-observability branch May 29, 2024 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants