Skip to content

Commit

Permalink
Merge pull request #2404 from yuvipanda/comment-prom
Browse files Browse the repository at this point in the history
Add comment about why we are using prometheus helm chart
  • Loading branch information
yuvipanda authored Mar 22, 2023
2 parents 7404f63 + 9575443 commit 545930d
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions helm-charts/support/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,11 @@ description: Cluster wide depdencies for deployed hubs
dependencies:
# Prometheus for collection of metrics.
# https://github.com/prometheus-community/helm-charts/tree/main/charts/prometheus
# We have deliberately chosen to use the helm chart rather than the prometheus
# operator, as we will never have more than one prometheus installation per
# cluster. The operator thus adds functionality we will not use, at a complexity
# cost. If we ever have multiple prometheii on the same cluster, we can
# reconsider using the operator.
- name: prometheus
version: 19.7.2
repository: https://prometheus-community.github.io/helm-charts
Expand Down

0 comments on commit 545930d

Please sign in to comment.