Skip to content

Commit

Permalink
Fix/remove duplicate datasource (#17)
Browse files Browse the repository at this point in the history
* Fix/questions (#15)

* fix: no namespace ALL are allowed in k8s-sidecar search

* fix: no required namespaces in a single namespace installation

* fix: pause feature only if it's enabled

* bump chart version

* fix: restore question, sub without effect

* fix: retentionSize

* chore: minor fixes in docs, corrected questions.yaml (#16)

* bump 1.0.22

* fix: removed the unnecessary second grafana datasource

* chore: bumped chart version

* debug: remove datasource mgmt via sidecar

* chore: finalized fix

---------

Co-authored-by: Bruno Bressi <[email protected]>
Co-authored-by: Bruno Bressi <[email protected]>
  • Loading branch information
3 people authored Sep 21, 2023
1 parent 6719dec commit 58997f6
Show file tree
Hide file tree
Showing 5 changed files with 23,245 additions and 1,427 deletions.
3 changes: 2 additions & 1 deletion app-readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,8 @@ rules:
kube-prometheus-stack will installed in a project namespace with Project-Owner permissions + the additional permissions above.
With default settings at least install namespace are covered with metrics, permitted by the ServiceAccountToken in the Prometheus Pod. If the install namespace is in a project, all other namespaces are also covered by the token permission.
For additonal Prometheus targets with ServiceMonitor in other project namespaces or AlertmanagerConfigs this namespaces must by listed in `caas.projectNamespaces` Prometheus discovery can be limited, e.g.
To monitor additional Prometheus targets with `ServiceMonitors` and/or `AlertmanagerConfigs` in other project namespaces, those namespaces **must** by listed in `caas.projectNamespaces`.

<details>
<summary>code snippet</summary>
Expand Down
Loading

0 comments on commit 58997f6

Please sign in to comment.