Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[kube-prometheus-stack] Add comment about excluded components from kube-prometheus #5113

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ChrisMcD1
Copy link

What this PR does / why we need it

I was confused on initial install of kube-prometheus-stack why I was not successfully getting a Prometheus Adapter set up, and saw there was an open issue #4741 that shared my confusion. This PR just clarifies that going to kube-prometheus and seeing the component list will not be exactly the same behavior as a fresh install of this chart. @jkroepke I saw you on the initial issue thread, so I figured I should ping you!

Which issue this PR fixes

Special notes for your reviewer

Checklist

  • DCO signed
  • Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[kube-prometheus-stack] Clarify chart is not 100% faithful to kube-prometheus
1 participant