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

feat(server): Org rate limit per metric bucket #2836

Merged
merged 7 commits into from
Dec 13, 2023
Merged

feat(server): Org rate limit per metric bucket #2836

merged 7 commits into from
Dec 13, 2023

Conversation

TBS1996
Copy link
Contributor

@TBS1996 TBS1996 commented Dec 11, 2023

part of: #2716

in order to protect our kafka metric consumers, we want to have a way of rate limiting based on the amount of buckets, as that's what decides the load placed on our kafka topics.

We are starting out with just the org throughput limits but will be expanded upon further as outlined in the linked epic.

This is a re-revert of #2758 (reverted in #2821).

@TBS1996 TBS1996 self-assigned this Dec 11, 2023
@TBS1996 TBS1996 marked this pull request as ready for review December 13, 2023 07:20
@TBS1996 TBS1996 requested a review from a team as a code owner December 13, 2023 07:20
@jjbayer jjbayer enabled auto-merge (squash) December 13, 2023 12:00
@jjbayer jjbayer merged commit a3b5def into master Dec 13, 2023
20 checks passed
@jjbayer jjbayer deleted the tor/totalorg branch December 13, 2023 12:05
jan-auer added a commit that referenced this pull request Dec 19, 2023
* master: (35 commits)
  fix(spans): Parse quotes in MySQL (#2846)
  ref(cardinality): Use a Lua script and in-memory cache for the cardinality limiter (#2849)
  fix(spans): Detect hex with fallback scrubber (#2868)
  release: 23.12.0
  Revert "ci: Update upload-artifact and download-artifact actions" (#2866)
  Revert "build: Update axum and http" (#2863)
  feat(spans): Allow resource.img spans (#2855)
  build: Update axum and http (#2844)
  fix(build): Add additional dependencies to the release build (#2858)
  ci: Update upload-artifact and download-artifact actions (#2861)
  feat(spans): Parse timestamps from strings (#2857)
  fix(spans): Scrub integer file extensions (#2856)
  feat(spans): Remove unused transaction tag from resource metrics (#2853)
  ref(cardinality): Recover buckets on cardinality limiter failure (#2852)
  feat(server): Org rate limit per metric bucket (#2836)
  ref(spans): List metric tags explicitly (#2834)
  feat(spans): Resource response sizes as measurements (#2845)
  feat(crons): Add thresholds to monitor config payload (#2842)
  feat(spans): Allow ingestion of metrics summary on spans (#2823)
  ref(crons): Add documentation to CheckInMessageType (#2840)
  ...
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.

3 participants