-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Description
Summary
While reviewing the codebase, it appears that the Grafana provider may also serve as a source of topology data, similar to FluxCD and other integrations.
Specifically, the provider code imports TopologyServiceInDto, suggesting that it contributes or has the potential to contribute data to the Service Topology feature.
However, the official documentation does not mention Grafana among topology-capable providers.
🧩 Observations
-
The Grafana provider code imports:
from keep.api.models.topology import TopologyServiceInDto
This implies potential participation in service topology discovery.
-
The documentation at
👉 https://docs.keephq.dev/overview/servicetopology
lists only Datadog, PagerDuty, ArgoCD, Cilium, and ServiceNow — Grafana is not mentioned.
💡 Expected behavior
If the Grafana provider indeed supports (or partially supports) contributing topology information, it should be:
- Explicitly listed on the Service Topology overview page.
- Documented in the Grafana provider page with a description of what topology data it supplies (e.g. dashboards, service mappings, metrics sources).
If not yet implemented — this should be clarified in the documentation for transparency.
✅ Proposed fix
- Confirm whether Grafana provider participates in service topology.
- If yes → add it to the list of topology-capable providers on
/overview/servicetopology. - Update
grafana-providerdocumentation page with relevant details. - If partial or planned — mark it as “coming soon” to avoid confusion.
🧠 Rationale
- Grafana is widely used for observability and service visualization, so it’s natural for users to expect it to appear in topology-related features.
- Clarifying its role (or absence) improves user understanding of available data sources.
Labels: type:enhancement area:documentation priority:medium