Skip to content

fix: prevent duplicate metric collection#488

Merged
AndrejsPon00 merged 1 commit into
k0rdent:mainfrom
AndrejsPon00:fix-metric-dups
Aug 27, 2025
Merged

fix: prevent duplicate metric collection#488
AndrejsPon00 merged 1 commit into
k0rdent:mainfrom
AndrejsPon00:fix-metric-dups

Conversation

@AndrejsPon00
Copy link
Copy Markdown
Contributor

@AndrejsPon00 AndrejsPon00 commented Aug 21, 2025

Closes #411

  • Removed the node-exporter job from the daemon collector scrape config

  • Added node and nodename labels to the prometheus-node-exporter ServiceMonitor so Grafana dashboards continue to render correctly.

  • Dashboards related to Node Exporter metrics rendered correctly (tested on a regional cluster):

    • Node Exporter Full
    • KPS / Kubernetes / Compute Resources / Cluster
    • KPS / Kubernetes / Compute Resources / Multi-Cluster
    • KPS / Node Exporter / USE Method / Cluster
    • KPS / Node Exporter / USE Method / Node
    • KPS / Node Exporter / AIX
    • KPS / Node Exporter / Nodes
  • Dropped scraping of kube-proxy and kube-dns pods in the collector daemon scrape config (kubernetes-pods job) to prevent duplicate metrics on regional and child clusters.

  • Dashboards related to Kube-dns and Kube-proxy metrics rendered correctly (tested on a regional cluster):

    • KPS / Kubernetes / Proxy
    • KPS / CoreDNS

@gmlexx
Copy link
Copy Markdown
Collaborator

gmlexx commented Aug 21, 2025

@aglarendil please have a look

@AndrejsPon00 AndrejsPon00 marked this pull request as draft August 22, 2025 15:51
@AndrejsPon00 AndrejsPon00 marked this pull request as ready for review August 25, 2025 11:16
@AndrejsPon00 AndrejsPon00 force-pushed the fix-metric-dups branch 2 times, most recently from 2aefc26 to 5dd93a4 Compare August 25, 2025 13:32
@AndrejsPon00 AndrejsPon00 added this pull request to the merge queue Aug 27, 2025
Merged via the queue into k0rdent:main with commit 9107ce6 Aug 27, 2025
8 of 9 checks passed
@github-project-automation github-project-automation Bot moved this to Done in k0rdent Aug 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

dedup some metrics collection done simulateneously by ta-daemon and daemon

4 participants