canonical / grafana-k8s-operator

https://charmhub.io/grafana-k8s
Apache License 2.0
6 stars 22 forks source link

COS is adding Juju topology filter into a panel instead of the top #320

Open nobuto-m opened 6 months ago

nobuto-m commented 6 months ago

Bug Description

As you can see in the screenshots below, COS is inserting the Juju topology filter (juju_unit, juju_application, juju_model_uuid, juju_model) into an unexpected place in the dashboard.

A: with https://github.com/nobuto-m/ceph/blob/681ca6617b59a04f6115c416c4f52c11d8971f66/monitoring/ceph-mixin/dashboards_out/pool-overview.json

-> The Juju topology filter at the top.

ceph-pools-overview_charmed

B: with https://github.com/nobuto-m/ceph/blob/f940876d6c1ce6293eb3a96ec4a0f3df44d8d0d3/monitoring/ceph-mixin/dashboards_out/pool-overview.json

-> The Juju topology filter is inside the "Pool Overview" panel instead of the top.

ceph-pools-overview_charmed_main

To Reproduce

  1. deploy COS
  2. deploy cos-configuration-k8s and import the dashboard definition of two patterns as above.

Environment

App                            Version  Status  Scale  Charm                         Channel      Rev
alertmanager                   0.27.0   active      1  alertmanager-k8s              latest/edge  107
catalogue                               active      1  catalogue-k8s                 latest/edge   33
cos-configuration-ceph         3.5.0    active      1  cos-configuration-k8s         latest/edge   47
grafana                        9.5.3    active      1  grafana-k8s                   latest/edge  108
loki                           2.9.5    active      1  loki-k8s                      latest/edge  128
prometheus                     2.50.1   active      1  prometheus-k8s                latest/edge  173
prometheus-scrape-config-ceph  n/a      active      1  prometheus-scrape-config-k8s  latest/edge   47
traefik                        v2.11.0  active      1  traefik-k8s                   latest/edge  177

Relevant log output

N/A

Additional context

No response

lucabello commented 2 months ago

This will probably be easier to debug once we have #283.