teslamotors / helm-charts

10 stars 7 forks source link

Prometheus Monitoring Port Required #31

Open joelbixbyrevel opened 9 months ago

joelbixbyrevel commented 9 months ago

Issue

Configuration monitoring.prometheus_metrics_port must be set or the container will not run the metric endpoint service. As a result, the deployment liveness health checks fail.

References

Expectation

For those not using prometheus, liveness health checks fall back to an endpoint that is enabled.

Workaround

Set the port to an arbitrary value and let the metrics data fail to deliver

joelbixbyrevel commented 9 months ago

I may have some time to help contribute, but would prefer to get a desired direction from repository owners before putting in the time.