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(pkg/operator): sort endpoints in reconciler for better perf #5943

Merged
merged 4 commits into from
Jan 12, 2024

Conversation

hainenber
Copy link
Contributor

PR Description

Sort Kubelet endpoints in agent operator to reduce resource cost on API server and endpoint watchers

Which issue(s) this PR fixes

Fixes #5934

Notes to the Reviewer

PR Checklist

  • CHANGELOG.md updated

Copy link
Member

@tpaschalis tpaschalis left a comment

Choose a reason for hiding this comment

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

LGTM; leaving to @captncraig to see if he has any more input so we can close the original issue.

Thank you @hainenber for the continued efforts to improving the Agent 🙌

@tpaschalis tpaschalis enabled auto-merge (squash) January 12, 2024 11:18
@tpaschalis tpaschalis merged commit eae8c03 into grafana:main Jan 12, 2024
9 of 10 checks passed
@hainenber hainenber deleted the sort-kubelet-endpoints branch January 12, 2024 18:10
BarunKGP pushed a commit to BarunKGP/grafana-agent that referenced this pull request Feb 20, 2024
@github-actions github-actions bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Feb 21, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

operator: kubelet endpoints are non-deterministic
2 participants