canonical / tempo-k8s-operator

This charmed operator automates the operational procedures of running Grafana Tempo, an open-source tracing backend.
https://charmhub.io/tempo-k8s
Apache License 2.0
3 stars 3 forks source link

Clean up tracing databag in v2 #108

Closed michaeldmitry closed 1 month ago

michaeldmitry commented 1 month ago

Issue

Fixes #105

Solution

Testing instructions

with tempo charm repo root as CWD:

juju add-model tempo-clite-test
juju deploy cos-lite --channel edge --trust

git fetch
git checkout OPENG-2399
jhack sync "*" --touch ./lib/charms/tempo_k8s/v2/tracing.py

charmcraft pack
jhack deploy 

jhack imatrix fill

now go to the grafana dashboard and ensure you can see charm traces for all apps (and grafana workload)

add tls:

juju deploy self-signed-certificates ssc
jhack imatrix fill

verify you can still see the new charm traces for all apps (and grafana workload)

Issues: traefik gets stuck because we removed the deprecated api that traefik was still relying on. see https://github.com/canonical/traefik-k8s-operator/pull/364

Same for grafana: https://github.com/canonical/grafana-k8s-operator/pull/333

Catalogue has broken tls charm tracing https://github.com/canonical/catalogue-k8s-operator/pull/102