grafana / tempo

Grafana Tempo is a high volume, minimal dependency distributed tracing backend.
https://grafana.com/oss/tempo/
GNU Affero General Public License v3.0
3.76k stars 488 forks source link

Documentation feedback: /docs/sources/tempo/configuration/_index.md - Cache allowed roles is missing options #3801

Open jordanbertasso opened 2 weeks ago

jordanbertasso commented 2 weeks ago

The allowed roles listed in the cache documentation differ from the actual allowed roles

image (28)

https://github.com/grafana/tempo/blob/efb11ea97f2dbd0f8e2416a5c64f64d415084823/pkg/cache/cache.go#L11-L21

joe-elliott commented 1 week ago

parquet-column-idx and parquet-offset-idx are supposed to be undocumented. I didn't realize they were in the example.

These cache keys currently do nothing, b/c we don't use offset and column indexes in Parquet atm. We should remove them from the example.

knylander-grafana commented 1 week ago

I'll remove them from the documentation.

knylander-grafana commented 3 days ago

PR https://github.com/grafana/tempo/pull/3826