Skip to content

feat(enricher): make ring buffer capacity configurable#2506

Open
mmckeen wants to merge 1 commit into
microsoft:mainfrom
mmckeen:configureEnricherRingCapacity
Open

feat(enricher): make ring buffer capacity configurable#2506
mmckeen wants to merge 1 commit into
microsoft:mainfrom
mmckeen:configureEnricherRingCapacity

Conversation

@mmckeen

@mmckeen mmckeen commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Description

Configuring the enricher ring buffer size can be useful to absorb spiky reports at the expense of some memory, allow tuning it.

Checklist

  • I have read the contributing documentation.
  • I signed and signed-off the commits (git commit -S -s ...). See this documentation on signing commits.
  • I have correctly attributed the author(s) of the code.
  • I have tested the changes locally.
  • I have followed the project's style guidelines.
  • I have updated the documentation, if necessary.
  • I have added tests, if applicable.

Screenshots (if applicable) or Testing Completed

Tuned and noticed enricher drop events going down as this goes up (and spikes can be better tolerated).


Please refer to the CONTRIBUTING.md file for more information on how to contribute to this project.

@mmckeen
mmckeen force-pushed the configureEnricherRingCapacity branch from 8a5a69a to 519b157 Compare July 9, 2026 02:50
@mmckeen
mmckeen marked this pull request as ready for review July 9, 2026 02:51
@mmckeen
mmckeen requested a review from a team as a code owner July 9, 2026 02:51
@mmckeen
mmckeen requested review from agrawaliti and mainred July 9, 2026 02:51
@mmckeen
mmckeen force-pushed the configureEnricherRingCapacity branch 2 times, most recently from 505c6ea to 01444dd Compare July 10, 2026 21:44
Signed-off-by: Matthew McKeen <matthew.mckeen@fastly.com>
@mmckeen
mmckeen force-pushed the configureEnricherRingCapacity branch from 01444dd to 7183fa0 Compare July 15, 2026 15:44
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.

1 participant