Ship Grafana dashboards with the chart #1035

Open
opened 2025-01-09 19:27:46 +00:00 by hucksy · 3 comments

I would like to be able to enable dashboards in the Helm chart and get a ConfigMap with the Forgejo dashboards from the forgejo/contrib/gitea-monitoring-mixin, similar to how other Helm charts ship dashboards (ie Kyverno).

I would like to be able to enable dashboards in the Helm chart and get a `ConfigMap` with the Forgejo dashboards from the [forgejo/contrib/gitea-monitoring-mixin](https://codeberg.org/forgejo/forgejo/src/branch/forgejo/contrib/gitea-monitoring-mixin), similar to how other Helm charts ship dashboards (ie [Kyverno](https://github.com/kyverno/kyverno/blob/main/charts/kyverno/charts/grafana/README.md)).

I belive with focus on the grafanaOperator we should support both:

  • ConfigMap with configurable label and annotations
  • GrafanaDashboard with configurable instanceSelector

my suggestion is to render the json beside the helm templates and use .Files.Get to put in in ConfigMap and/or GrafanaDashboard.

I belive with focus on the grafanaOperator we should support both: - ConfigMap with configurable label and annotations - [GrafanaDashboard](https://grafana.github.io/grafana-operator/docs/dashboards/#json) with configurable instanceSelector my suggestion is to render the json beside the helm templates and use `.Files.Get` to put in in ConfigMap and/or GrafanaDashboard.
Owner

I'm not sure if this is in scope of this chart or if it should be another new chart like the runner chart which is coming soon.

@pat-s opinions?

I'm not sure if this is in scope of this chart or if it should be another new chart like the runner chart which is coming soon. @pat-s opinions?
Member

A separate chart would surely be cleaner.
I wouldn't add it as a sub-chart anyhow, if at all then as an optional configMap and a distinct values section.

However, users might prefer to update these resources independently from Forgejo itself, and this would be much easier with a dedicated chart.

A separate chart would surely be cleaner. I wouldn't add it as a sub-chart anyhow, if at all then as an optional `configMap` and a distinct values section. However, users might prefer to update these resources independently from Forgejo itself, and this would be much easier with a dedicated chart.
Sign in to join this conversation.
No milestone
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
forgejo-helm/forgejo-helm#1035
No description provided.