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

First version on how to bring your custom kube-prometheus-stack #206

Merged
merged 30 commits into from
Nov 2, 2022
Merged

First version on how to bring your custom kube-prometheus-stack #206

merged 30 commits into from
Nov 2, 2022

Conversation

a-thaler
Copy link
Contributor

@a-thaler a-thaler commented Sep 9, 2022

Description

Changes proposed in this pull request:

First version for kyma-project/kyma#15914
More needs to be done to solve the ticket, however the current status provides already a lot of value.

  • new tutorial on how to add a custom prometheus stack
  • assures that it can run in parallel to a kyma stack
  • brings k8s and istio setup
  • supports custom metrics with mTLS
  • enables generic scraping for workloads having prometheus annotation present

Related issue(s)

kyma-project/kyma#14379
kyma-project/kyma#15914

monitoring-custom-metrics/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/values.yaml Outdated Show resolved Hide resolved
prometheus/values.yaml Outdated Show resolved Hide resolved
prometheus/values.yaml Outdated Show resolved Hide resolved
prometheus/values.yaml Outdated Show resolved Hide resolved
prometheus/values.yaml Outdated Show resolved Hide resolved
prometheus/values.yaml Outdated Show resolved Hide resolved
prometheus/values.yaml Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
@kyma-bot kyma-bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Sep 23, 2022
@NHingerl NHingerl added the area/documentation Issues or PRs related to documentation label Sep 27, 2022
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
@a-thaler
Copy link
Contributor Author

a-thaler commented Sep 30, 2022

Further feedback to incorporate:

  • Have a warning for the grafana license
  • Bundle eventing and function dashboards
  • Maybe have a federation scrape on kyma prometheus described
  • kubernetes/statefulset dashboard is missing
  • have an alerting example referenced
  • linked example is based on kyma sytack prometheus and might confuse

@skhalash skhalash changed the title added tutorial on how to bring your custom kube-prometheus-stack Tutorial on how to bring your custom kube-prometheus-stack Oct 14, 2022
@a-thaler
Copy link
Contributor Author

I created a proper issue and moved the open points there so that we can continue merging the current state and adress the open points afterwards in a dedicated PR. kyma-project/kyma#15914

@a-thaler a-thaler changed the title Tutorial on how to bring your custom kube-prometheus-stack First version on how to bring your custom kube-prometheus-stack Oct 26, 2022
@a-thaler a-thaler added the area/monitoring Issues or PRs related to the monitoring module (deprecated) label Oct 26, 2022
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
Co-authored-by: Rakesh Garimella <[email protected]>
2. Deploy the service:
2. Ensure that your Namespace has Istio sidecar injection enabled. This example assumes that the metrics are exposed in a strict mTLS mode:

````
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm surprised that this is rendered correctly; shouldn't the code block end with a couple of backticks in line 28?

prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
@kyma-bot kyma-bot added lgtm Looks good to me! labels Nov 2, 2022
NHingerl
NHingerl previously approved these changes Nov 2, 2022
@kyma-bot kyma-bot added lgtm Looks good to me! labels Nov 2, 2022
@NHingerl
Copy link
Contributor

NHingerl commented Nov 2, 2022

/hold

@kyma-bot kyma-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 2, 2022
prometheus/README.md Outdated Show resolved Hide resolved
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Nov 2, 2022
@NHingerl NHingerl removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 2, 2022
@kyma-bot kyma-bot added lgtm Looks good to me! labels Nov 2, 2022
@kyma-bot kyma-bot merged commit 55e214d into kyma-project:main Nov 2, 2022
@a-thaler a-thaler deleted the prometheus branch November 8, 2022 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Issues or PRs related to documentation area/monitoring Issues or PRs related to the monitoring module (deprecated) lgtm Looks good to me! size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants