openfaas / faas-netes

Serverless Functions For Kubernetes
https://www.openfaas.com
MIT License
2.12k stars 473 forks source link

Add storage for Prometheus using a PersistentVolumeClaim #1226

Closed alexellis closed 4 weeks ago

alexellis commented 1 month ago

Description

Add storage for Prometheus using a PersistentVolumeClaim

Why is this needed?

Prior to this change, Prometheus' timeseries data would be lost whenever the chart was upgraded, the Prometheus Pod was rescheduled, or restarted due to a configuration change.

How Has This Been Tested?

Tested with a blank storageClassName with KinD, and the local path provisioner. After creating load with hey, and restarting Prometheus, the invocation metrics remained available.

prom-pvc

Types of changes

Checklist:

I have updated the documentation in the values.yaml file, however this will not be a default, so needs additional documentation in other areas such as the chart README and the Production guide for OpenFaaS Pro.