prometheus-k8s-operator

prometheus-k8s-operator copied to clipboard

If a push fails, prometheus may never be reload even though config changed (theoretical edge-case)

1

If some files were pushed but others failed, when _configure re-enters then `should_reload` may be false without prometheus ever reloading. https://github.com/canonical/prometheus-k8s-operator/blob/b4c0fd631a8bfbeb7d22b9b6f157f8f25cefc6f6/src/charm.py#L292-L304

sed-i

Changing name of an alert creates a new alert rule

1

### Bug Description We have just started to implement COS-lite stack into our production. When we create the charms, we add rules for grafana-agent to integrate onto prometheus, but when...

Omgzilla

The return type of this property is wrong https://github.com/canonical/prometheus-k8s-operator/blob/72224ba22b28b4d75adbf115206ed17904dfa2ef/lib/charms/prometheus_k8s/v0/prometheus_remote_write.py#L700 For instance in, [grafana_agent.py](https://github.com/canonical/grafana-agent-k8s-operator/blob/main/src/grafana_agent.py#L418) we append a `Dict[str, Dict[str, str]]` rather than a `Dict[str, str]` ```python prometheus_endpoints = self._remote_write.endpoints if self._cloud.prometheus_ready:...

Abuelodelanada

### Bug Description In order to ensure high availability when using Prometheus remote write multiple remote write agents (Grafana Agent or Prometheus) need to scrape the same targets. On the...

balbirthomas

Add diagnostics tox env

4

## Issue When doing manual tests, I keep using the same lengthy commands over and over. ## Solution Add a diagnostics script to output all the interesting integration-related content. ##...

sed-i

Provider should raise if multiple jobs are declared but no job suffixes are set

4

### Enhancement Proposal If a charm author tries to declare more than one job while instantiating the `MetricsEndpointProvider`, they need to set a job suffix. If they don't, the library...

simskij

### Enhancement Proposal ## Issue Currently, Prometheus [recording rules](https://prometheus.io/docs/prometheus/latest/configuration/recording_rules/) are implicitly supported; if they are put in the `prometheus_alert_folder`, the library will pick them up and import them correctly. However,...

lucabello

## Issue Fixed #604 ## Solution ca cert is pushed into charm container, so pass ca cert path to trace_charm decorator to use that for tls communication with tempo ##...

michaeldmitry

### Bug Description When trying to integrate prometheus with tempo to send its charm traces over tls, it fails to do with an exception "Exception while exporting Span batch." ###...

michaeldmitry