Skip to content
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

Adding cortex_ingester_max_inflight_query_requests metric #5798

Merged
merged 4 commits into from
Mar 5, 2024

Conversation

alanprot
Copy link
Member

@alanprot alanprot commented Mar 5, 2024

What this PR does:
Create a new cortex_ingester_inflight_query_requests metric to track how many request are inflight on each ingester.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

…mber of query being executed on ingesters

Signed-off-by: Alan Protasio <alanprot@gmail.com>
@alanprot alanprot marked this pull request as ready for review March 5, 2024 05:40
pkg/ingester/metrics.go Outdated Show resolved Hide resolved
pkg/ingester/metrics.go Outdated Show resolved Hide resolved
…x_ingester_max_inflight_query_requests

Signed-off-by: Alan Protasio <alanprot@gmail.com>
@alanprot
Copy link
Member Author

alanprot commented Mar 5, 2024

@yeya24 PTAL?

pkg/ingester/metrics.go Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
alanprot and others added 2 commits March 5, 2024 10:25
Signed-off-by: Alan Protasio <alanprot@gmail.com>
Signed-off-by: Alan Protasio <approtas@amazon.com>
@yeya24 yeya24 changed the title Adding cortex_ingester_inflight_query_requests metric Adding cortex_ingester_max_inflight_query_requests metric Mar 5, 2024
@yeya24 yeya24 merged commit 3cf9380 into cortexproject:master Mar 5, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants