Allow setting an emptyDir sizeLimit for Prometheus #12281
Allow setting an emptyDir sizeLimit for Prometheus #12281
Conversation
Signed-off-by: Mike Tougeron <tougeron@adobe.com>
Hi @mtougeron. Thanks for your PR. I'm waiting for a helm member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@mgoodness / @gianrubio just wanted to ping you for a rough idea of when you'll be able to review this. Thanks! |
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.
/ok-to-test
/lgtm
I'll resolve the conflicts for this today |
f86c04e
to
fc08c26
Compare
Signed-off-by: Mike Tougeron <tougeron@adobe.com>
fc08c26
to
09f70b9
Compare
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.
/ok-to-test
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gianrubio, mtougeron 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 |
* upstream/master: (4160 commits) [stable/mysql] support loadBalancerIP, configurationFilesPath (helm#13670) Allow setting an emptyDir sizeLimit for Prometheus (helm#12281) [stable/concourse] Add possibility to set strategy for rollingUpdates (web) (helm#13616) [stable/cluster-autoscaler] support dnsPolicy (helm#9029) [stable/rabbitmq-ha] allow setting different topology keys for anti affinity (helm#13635) [stable/airflow] Fix prometheusRule additionalLabels (helm#13683) [stable/postgresql] Set the right env. variable to enable debug (helm#13682) Use KUBERNETES_SERVICE_HOST environment variable (helm#13676) Enable new_k8s flag by default (helm#13618) [stable/prometheus-operator] Bump version dependency (helm#13625) [stable/odoo] Update Odoo dependencies (helm#13657) [stable/airflow] Add support for prometheus rules (helm#13662) [stable/signalfx-agent] Adding deprication notice (helm#13586) [stable/kube-state-metrics] Allows deploying ServiceAccount without RBAC (helm#13495) stable/ambassador - Update to Ambassador 0.61.0 (helm#13661) [stable/redmine] Update Redmine dependencies (helm#13658) [stable/chart] Add deploymentStrategy on HackMD chart (helm#13645) [stable/grafana] Adds imagePullSecrets to test pod (helm#13574) [stable/k8s-dashboard] Clean up cluster-service label (helm#13545) [stable/postgresql] Fix 'mountPath' for password files. (helm#13653) ...
* Allow setting an emptyDir sizeLimit for Prometheus Signed-off-by: Mike Tougeron <tougeron@adobe.com> * Bump the version number Signed-off-by: Mike Tougeron <tougeron@adobe.com>
* Allow setting an emptyDir sizeLimit for Prometheus Signed-off-by: Mike Tougeron <tougeron@adobe.com> * Bump the version number Signed-off-by: Mike Tougeron <tougeron@adobe.com>
Signed-off-by: Mike Tougeron tougeron@adobe.com
What this PR does / why we need it:
This PR allows you to set a
sizeLimit
for Prometheus'emptyDir
volume when not using persistent storage.Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]