-
Notifications
You must be signed in to change notification settings - Fork 451
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
[GEP-19] Integrate long-term Prometheus deployment into Garden
controller
#9606
Conversation
83ba80c
to
f02f0bc
Compare
/assign |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for this great contribution. It highlights nicely how easy it is now with prometheus operator to add another prometheus to the gardener ecosystem.
f02f0bc
to
e855f34
Compare
LGTM label has been added. Git tree hash: c0d74d679a7bdb8889c5452316403ae2f3a94339
|
The longterm Prometheus uses this for caching, see docs (first commit)
Also adapt Plutono dashboard datasources This Prometheus has a cortex sidecar which serves as frontend, so the port must be 9091 (cortex container port)
e855f34
to
6bcdd63
Compare
New changes are detected. LGTM label has been removed. |
Already |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rfranzke The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
(from gardener#9606, released with `v1.94.0`)
(from gardener#9606, released with `v1.94.0`)
* Remove deprecated fields from `OperatingSystemConfig` (from #9477, released with `v1.92.0`) * Remove cleanup of old `kube-apiserver` `Ingress` resource (from #9300, released with `v1.91.0`) * Remove Istio zone migration code (from #9304 and #9457, released with `v1.91.0` and `v1.92.0`) * Increase removal period of `<name>.ca-cluster` `Secret` To give users more time to adapt * Remove PVC migration for `garden` Prometheus (from #9543, released with `v1.93.0`) * Remove PVC migration for `longterm` Prometheus (from #9606, released with `v1.94.0`) * Drop migration code in `skaffold.yaml` for `core.gardener.cloud/v1` API (from #9771, released with `v1.96.0`) * Remove migration code for e2e upgrade tests after `provider-local` VPN fix (from #9752, released with `v1.96.0`) * Remove cleanup of old `vali` `VerticalPodAutoscaler`s (from #9681, released with `v1.94.0`) * Remove cleanuop code after making `Secret`s of `ManagedResource`s immutable (from #8116, released with `v1.77.0`) * Remove cleanup code of resources of legacy `cloud-config-downloader` (from #8847, released with `v1.85.0`) * Revert "Remove Istio zone migration code" This reverts commit 8850346. * Increase removal period of Istio zone migration code
How to categorize this PR?
/area dev-productivity monitoring
/kind enhancement
What this PR does / why we need it:
gardener-operator
now deploys two Prometheus replicas into thegarden
namespace for long-term retention of some metrics (e.g., availability/SLI or metering data).This also features a cortex frontend for caching purposes.
Which issue(s) this PR fixes:
Part of #9065
Special notes for your reviewer:
/cc @ScheererJ
FYI @istvanballok @vicwicker
Release note:
`gardener-operator` now deploys two more Prometheus replicas into the `garden` namespace for storing long-term metrics. Read more about it [here](https://github.com/gardener/gardener/blob/master/docs/concepts/operator.md#long-term-prometheus).