canonical / prometheus-k8s-operator

https://charmhub.io/prometheus-k8s
Apache License 2.0
21 stars 35 forks source link

Add scheme to alertmanagers config section #495

Closed sed-i closed 1 year ago

sed-i commented 1 year ago

Issue

The alertmanagers section in prometheus's config is rendered without a scheme and without tls_config, and prometheus uses http by default.

Solution

This PR goes in tandem with https://github.com/canonical/alertmanager-k8s-operator/pull/171.

Reference: https://prometheus.io/docs/prometheus/latest/configuration/configuration/#tls_config

Context

https://github.com/canonical/cos-lite-bundle/issues/75

Testing Instructions

Release Notes

Add scheme to alertmanagers config section.

sed-i commented 1 year ago

Itests passed but not reflected on the PR page here. Merging.