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

r311: Cherry-pick memberlist debounce PR from dskit #9715

Merged
merged 3 commits into from
Oct 23, 2024

Conversation

seizethedave
Copy link
Contributor

@seizethedave seizethedave commented Oct 22, 2024

What this PR does

This is a manual quasi-backport to obtain grafana/dskit#592 which is cherry-picked into dskit's r311 branch. It is turned off unless explicitly configured.

Checklist

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

@seizethedave seizethedave changed the title r311: Set dskit to v0.0.0-20241022223531-b743e70423b6 r311: Cherry-pick memberlist debounce PR from dskit Oct 22, 2024
@seizethedave seizethedave marked this pull request as ready for review October 22, 2024 23:24
@@ -2959,6 +2959,11 @@ The `memberlist` block configures the Gossip memberlist.
# CLI flag: -memberlist.compression-enabled
[compression_enabled: <boolean> | default = true]

# (advanced) How frequently to notify watchers when a key changes. Can reduce
# CPU activity in large memberlist deployments. 0 to notify without delay.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# CPU activity in large memberlist deployments. 0 to notify without delay.
# CPU activity in large member list deployments. Set to `0` to notify without delay.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you- I’ll make these changes on the main branch. If I make them here no one will ever see them. 😺

@seizethedave seizethedave merged commit 6393cfc into r311 Oct 23, 2024
29 checks passed
@seizethedave seizethedave deleted the davidgrant/dskit-592-backport branch October 23, 2024 15:54
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