Skip to content

feat(conntrack): make report interval configurable#2501

Merged
nddq merged 1 commit into
microsoft:mainfrom
mmckeen:configureConntrackReportingInterval
Jul 15, 2026
Merged

feat(conntrack): make report interval configurable#2501
nddq merged 1 commit into
microsoft:mainfrom
mmckeen:configureConntrackReportingInterval

Conversation

@mmckeen

@mmckeen mmckeen commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Description

Allow configuring the conntrack reporting interval as another performance tuning lever.

Related Issue

#1760

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

Tested in a cluster with many connections where periodic reports contribute the majority of events, increasing the interval reports the same metrics with less dropped events and resource usage.


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

@mmckeen
mmckeen force-pushed the configureConntrackReportingInterval branch from 0ab0f6b to 81f8887 Compare July 7, 2026 20:42
@mmckeen
mmckeen marked this pull request as ready for review July 7, 2026 20:43
@mmckeen
mmckeen requested a review from a team as a code owner July 7, 2026 20:43
@mmckeen
mmckeen requested review from QxBytes and nddq July 7, 2026 20:43
@mmckeen
mmckeen force-pushed the configureConntrackReportingInterval branch from 81f8887 to 0bf1d72 Compare July 8, 2026 04:42

@nddq nddq left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Minor requests, otherwise LGTM, please rebase with latest main to get the CI fixes

Comment thread deploy/standard/manifests/controller/helm/retina/values.yaml
Signed-off-by: Matthew McKeen <matthew.mckeen@fastly.com>
@mmckeen
mmckeen force-pushed the configureConntrackReportingInterval branch from 0bf1d72 to 917ffcc Compare July 15, 2026 15:39
@nddq
nddq enabled auto-merge July 15, 2026 15:47
@nddq
nddq added this pull request to the merge queue Jul 15, 2026
Merged via the queue into microsoft:main with commit 1b9b111 Jul 15, 2026
36 checks passed
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