openstack-k8s-operators / edpm-ansible

External Dataplane Management Ansible Playbooks
https://openstack-k8s-operators.github.io/edpm-ansible/
Apache License 2.0
9 stars 66 forks source link

Disable unneeded Node exporter collectors #606

Closed vyzigold closed 7 months ago

vyzigold commented 7 months ago

Disable Node exporter collectors which we don't need. The collectors, which stayed enabled are the ones, which have something to do with storage, memory, cpu or networking.

This sheet shows an example of metrics being collected before this PR, after this PR and it lists the collectors, which are still enabled: https://docs.google.com/spreadsheets/d/1URZdgTARLxoijSlV6RpEKhF8ANHTIx9kr06XWSSw2wM/edit#gid=0

softwarefactory-project-zuul[bot] commented 7 months ago

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. Warning: Error merging github.com/openstack-k8s-operators/edpm-ansible for 606,c1067d6f34777ef7eabff37eb53b0c6d2bb572b3

fao89 commented 7 months ago

/approve

openshift-ci[bot] commented 7 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fao89, vyzigold

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openstack-k8s-operators/edpm-ansible/blob/main/OWNERS)~~ [fao89] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
jlarriba commented 7 months ago

/lgtm