Skip to content

[feat] PIP-223: Add metrics for all Rest Endpoints. Part1#18836

Closed
tjiuming wants to merge 7 commits intoapache:masterfrom
tjiuming:dev/restapi_metrics
Closed

[feat] PIP-223: Add metrics for all Rest Endpoints. Part1#18836
tjiuming wants to merge 7 commits intoapache:masterfrom
tjiuming:dev/restapi_metrics

Conversation

@tjiuming
Copy link
Contributor

@tjiuming tjiuming commented Dec 9, 2022

Master Issue: #18560

Motivation

Add per rest endpoint metrics.

Modifications

  1. Add configuration exposePerRestEndpointMetricsInPrometheus
  2. Add class RestEndpointMetricsFilter
  3. Add test class BrokerRestEndpointMetricsTest

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository: tjiuming#13

@github-actions github-actions bot added the doc-required Your PR changes impact docs and you will update later. label Dec 9, 2022
@tjiuming tjiuming changed the title [broker][monitoring] PIP-223: Add metrics for all Rest Endpoints. Part1 [monitoring] PIP-223: Add metrics for all Rest Endpoints. Part1 Dec 9, 2022
@tjiuming tjiuming changed the title [monitoring] PIP-223: Add metrics for all Rest Endpoints. Part1 [feat] PIP-223: Add metrics for all Rest Endpoints. Part1 Dec 9, 2022
@github-actions
Copy link

github-actions bot commented Jan 9, 2023

The pr had no activity for 30 days, mark with Stale label.

@dao-jun
Copy link
Member

dao-jun commented Dec 20, 2023

Moved to #21772, close this as inactive

@dao-jun dao-jun closed this Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-required Your PR changes impact docs and you will update later. ready-to-test Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants