UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Mattermost grafana dashboard

Merged brandt keller requested to merge mattermost_grafana_dashboard into main
1 unresolved thread

Description

Adding grafana dashboard json file and configmap to package when monitoring.enabled: true and enterprise.enebled: true (enterprise required for metrics scraping).

ServiceMonitor was already present and healthy in prometheus.

Testing notes

The dashboard does not default to server pointing at the mattermost instance. Rather this has to be selected manually (to which data is being accurately reported).

Issues

Relates to https://repo1.dso.mil/platform-one/big-bang/bigbang/-/issues/1214

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 1 {{- if and .Values.monitoring.enabled .Values.enterprise.enabled }}
    2 apiVersion: v1
    3 kind: ConfigMap
    4 metadata:
    5 name: mattermost-grafana-dashboards
    6 namespace: monitoring
    7 labels:
    8 grafana_dashboard: "1"
    9 data:
    10 mattermost-performance-monitoring-v2_rev2.json: |
    11 {{ .Files.Get "dashboards/mattermost-performance-monitoring-v2_rev2.json" | nindent 4 }}
    12
    13 mattermost-performance-kpi-metrics_rev2.json: |
    14 {{ .Files.Get "dashboards/mattermost-performance-kpi-metrics_rev2.json" | nindent 4 }}
  • Otherwise LGTM - deployed and the performance dashboard loaded and had metrics.

  • brandt keller added 1 commit

    added 1 commit

    • e508d831 - Fix: remove unecessary dashboard

    Compare with previous version

  • Micah Nagel approved this merge request

    approved this merge request

  • Micah Nagel enabled an automatic merge when the pipeline for e508d831 succeeds

    enabled an automatic merge when the pipeline for e508d831 succeeds

  • Micah Nagel mentioned in commit 5fde831a

    mentioned in commit 5fde831a

  • merged

  • Please register or sign in to reply
    Loading