Add a built-in cert-manager integration and dashboard#10536
Add a built-in cert-manager integration and dashboard#10536hriszc wants to merge 1 commit intoSigNoz:mainfrom
Conversation
|
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Free Tier Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| "yAxisUnit": "none" | ||
| } | ||
| ] | ||
| } |
There was a problem hiding this comment.
Dot dashboard uses underscore metric names instead of dots
High Severity
The overview_dot.json file is an exact copy of overview.json — all metric names and variable queries still use underscore notation (e.g., certmanager_certificate_expiration_timestamp_seconds) instead of dot notation (e.g., certmanager.certificate.expiration.timestamp.seconds). When dot metrics mode is enabled, builtin.go swaps in the _dot.json dashboard, but since it contains the wrong metric names, every panel and variable query will match zero data and the dashboard will appear empty.


Closes #6023
Summary
cert_managerintegration with setup docs, metadata, and icon assets_dotdashboard definitions so the integration works with either metrics modeValidation
jq empty pkg/query-service/app/integrations/builtin_integrations/cert_manager/integration.json pkg/query-service/app/integrations/builtin_integrations/cert_manager/assets/dashboards/overview.json pkg/query-service/app/integrations/builtin_integrations/cert_manager/assets/dashboards/overview_dot.jsongit diff --checkNotes
go, so I could not run the Go test/build pipeline hereNote
Low Risk
Asset-only change (JSON/Markdown/SVG) that doesn’t touch runtime logic; main risk is dashboard/query correctness and expected metric/label availability.
Overview
Adds a new built-in
cert_managerintegration (metadata, icon, and overview docs) underbuiltin_integrations/.Ships a cert-manager overview dashboard (plus an
_dotvariant) with filters for namespace/issuer/certificate and panels for certificate inventory, readiness, challenge status, and ACME request rate, along with collector scrape/configuration instructions.Written by Cursor Bugbot for commit d4ff11c. This will update automatically on new commits. Configure here.