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: Influxdb dashboards #466

Merged
merged 3 commits into from
Sep 9, 2021
Merged

feat: Influxdb dashboards #466

merged 3 commits into from
Sep 9, 2021

Conversation

vgratian
Copy link
Contributor

@vgratian vgratian commented Aug 27, 2021

This PR provides dashboards for the InfluxDB by me and @SamyukthaM. It also fixes #430.

  • At the current state, the dashboards can not be imported with the grafana-tool. This will be fixed with a different PR.
  • PR includes 8 dashboards, they are far from complete or perfect, but it's a start
    • most are based on our Prometheus-dashboards, but also introduce several improvements
    • they are tested only against cdot systems, they should work for 7modes to the same extend as our Prometheus-based dashboards, but since we are likely to split them up, I did not spend time here
  • A few things were necessary to fix:
    • the InfluxDB exporter will check for conflicting metric names and rename them
    • duplicate metrics interfere with some dashboards, that's why I removed update from conf/zapiperf/cdot/9.8.0/system_node.yaml (exact same metric is collected by the Zapi collector)
    • writing 7mode data to the same bucket as cdot is unsafe, this is documented
      exporter and a few minor fixes. Important: these dashboards

@cgrinds
Copy link
Collaborator

cgrinds commented Aug 27, 2021

Thanks @SamyukthaM and @vgratian - these look good!

One comment about 7-mode: Harvest does not support InfluxDB dashboards for 7-mode

@cgrinds cgrinds merged commit da8ac06 into main Sep 9, 2021
@cgrinds cgrinds deleted the influxdb-dashboards branch September 9, 2021 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

InfluxDB exporter: metric names collide with label names
4 participants