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

Discovery improvements #1120

Merged
merged 5 commits into from
Jan 16, 2025
Merged

Discovery improvements #1120

merged 5 commits into from
Jan 16, 2025

Conversation

petewall
Copy link
Collaborator

@petewall petewall commented Jan 15, 2025

This change uses a new version of alloy modules that improves discovery for kube-state-metrics, node-exporter, and windows-exporter. This make it more efficient on most clusters and it will also make it functional on openshift clusters.

@petewall petewall self-assigned this Jan 15, 2025
@petewall petewall linked an issue Jan 15, 2025 that may be closed by this pull request
@petewall petewall changed the title Get openshift working Discovery improvements Jan 15, 2025
@petewall petewall requested review from a team, skl, nevermind89x, jewbetcha and sleepyfoodie and removed request for a team January 15, 2025 20:07
@petewall petewall marked this pull request as ready for review January 15, 2025 20:07
@petewall petewall merged commit 242434d into main Jan 16, 2025
32 checks passed
@petewall petewall deleted the fix/v2-openshift branch January 16, 2025 16:03
petewall added a commit that referenced this pull request Jan 22, 2025
* Use new alloy modules to enable the right change to get openshift working

Signed-off-by: Pete Wall <[email protected]>

* Further improvements to discovery and openshift

Signed-off-by: Pete Wall <[email protected]>

* Update charts/k8s-monitoring/charts/feature-cluster-metrics/README.md

Co-authored-by: Stephen Lang <[email protected]>

* Also fix values.yaml

Signed-off-by: Pete Wall <[email protected]>

* More readme updates

Signed-off-by: Pete Wall <[email protected]>

---------

Signed-off-by: Pete Wall <[email protected]>
Co-authored-by: Stephen Lang <[email protected]>
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.

v2: Openshift broken due to missing KSM and NE discovery options
2 participants