Skip to content

Commit

Permalink
✏️ fix typo
Browse files Browse the repository at this point in the history
  • Loading branch information
davidB committed Jan 9, 2024
1 parent c9ebc8e commit efd2bd8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,9 @@ A set of components to provide a view of which version of services are deployed

## Architecture Overview

[![](doc/images/Drawing%202023-12-27%2016.04.47-cdviz-architecture.excalidraw.svg)](doc/images/Drawing%202023-12-27%2016.04.47-cdviz-architecture.excalidraw.svg)
![cdviz architecture](doc/images/Drawing%202023-12-27%2016.04.47-cdviz-architecture.excalidraw.svg)

**Why dashboard tool (like Grafana) have a read-only access to the DB (postgresql), and go through an API (micro)service (like cdviz-collector)?**
**Why do dashboard tools, like Grafana, have access (read-only) to the DB (PostgreSQL), and go through an API (micro)service (like `cdviz-collector`)?**

- access to the data is the value, not the service
- allow dashboards to use the full query power of SQL to query data, and to plug any analytics tools
Expand Down
1 change: 0 additions & 1 deletion charts/cdviz-collector/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -100,4 +100,3 @@ dbMigration:
# see Chart of cdviz for more details (list of keys,...) or templates/deployment.yaml
secrets:
name: cdviz # name of the secret (also used by cdviz-collector, dbMigration, postgresql, grafana,...)

0 comments on commit efd2bd8

Please sign in to comment.