canonical / tempo-k8s-operator

https://charmhub.io/tempo-k8s
Apache License 2.0
5 stars 3 forks source link

force-restart tempo service if there is a change in the enabled receivers #91

Closed PietroPasotti closed 5 months ago

PietroPasotti commented 5 months ago

Issue

Tempo was not being restarted properly. We called a pebble replan but pebble didn't notice the change in config.

Solution

force-restart if manually comparing old config and new config shows there are differences

Added an integration test for grpc traces using the tracegen script.