Skip to content

Infrastructure Grafana dashboards missing in default variant monitoring #2349

Closed

Description

@myasnikovdaniil

Description

PR #2197 scoped infrastructure dashboards to tenant-root namespace only, but the "default" variant deploys monitoring to the cozy-monitoring namespace. As a result, infrastructure dashboards (VictoriaMetrics, Flux, Hubble, LINSTOR, control-plane, etc.) are no longer rendered in the default variant.

Root cause

In packages/system/monitoring/templates/dashboards.yaml, the infra dashboards are conditionally rendered only when Release.Namespace == "tenant-root":

{{- if eq .Release.Namespace "tenant-root" }}
  {{- range (split "\n" (.Files.Get "dashboards-infra.list")) }}
  ...
  {{- end }}
{{- end }}

However, the default variant installs monitoring in the cozy-monitoring namespace (defined in packages/core/platform/sources/monitoring.yaml):

install:
  privileged: true
  namespace: cozy-monitoring

Since cozy-monitoring != tenant-root, the condition is false and all infra dashboards from dashboards-infra.list are skipped.

Expected behavior

Infrastructure dashboards should be available in the platform-level monitoring deployment regardless of variant. The condition should account for both tenant-root and cozy-monitoring namespaces (or more generally, distinguish platform-level monitoring from per-tenant monitoring).

Affected dashboards (dashboards-infra.list)

VictoriaMetrics, Flux, Hubble, LINSTOR, control-plane, main dashboard, and other infrastructure-level dashboards.

Affected files

  • packages/system/monitoring/templates/dashboards.yaml — namespace condition too restrictive
  • packages/system/monitoring/dashboards-infra.list — infra dashboard list
  • packages/core/platform/sources/monitoring.yaml — defines cozy-monitoring namespace for default variant

Activity

  1. added
    kind/bugCategorizes issue or PR as related to a bug
    on Apr 6, 2026
  2. added a commit that references this issue on Apr 9, 2026
    b8d48ad
  3. mattia-eleuteri commented on Apr 9, 2026

    @mattia-eleuteri
    Collaborator

    This is indeed a bug introduced by #2197. The infra dashboards condition only checks for tenant-root, but the default variant deploys the monitoring chart to the cozy-monitoring namespace (as defined in packages/core/platform/sources/monitoring.yaml).

    Fix PR: #2365 — adds cozy-monitoring to the namespace condition, consistent with the existing pattern in vmagent.yaml.

  4. added a commit that references this issue on Apr 9, 2026
    2c6e0fe
  5. added a commit that references this issue on Apr 9, 2026
    3b2d6a6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    kind/bugCategorizes issue or PR as related to a bug

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions