[Task] Default loki-mixin dashboards to TSDB

[trevorwhitney]

Given that TSDB is now the preferred index format for Loki, we should update the loki-mixin to reflect that. This probably includes:

• making it easier to include TSDB dashboards / rows in the mixin via a configuration variable in the jsonnet
• defaulting to TSDB when compiling the mixin

[pooladkhay]

Hi, I'd like to start working on this issue. Can I get a bit of help?

[trevorwhitney]

@pooladkhay sure thing!

So I think the best way to accomplish this is by adding a config section for setting the schema or enabling tsdb, like we have for our scalable deployment (aka ssd)

Then, in the relevant dashboard libsonnet, we can hide/show rows or panels, like here

It may require adding new rows to the raw dashboard json file, and then toggling their visibility in the libsonnet based on the config values set.

Once that is ready, you can change the defaults in mixin.libsonnet to set the appropriate configs, sort of like what we do in mixin-ssd.libsonnet.

Hope that helps!

[trevorwhitney]

@pooladkhay are you picking this up? Don't want to step on toes if you are, but we have some free cycles over here and were thinking if picking it up if you're not.

[pooladkhay]

@pooladkhay are you picking this up? Don't want to step on toes if you are, but we have some free cycles over here and were thinking if picking it up if you're not.

@trevorwhitney Please go ahead, I'm a bit busy unfortunately

[hainenber]

hi @trevorwhitney, has you or your team picking up this yet? If not , I can volunteer to work on. Thanks!

[hainenber]

Hi Trevor, it appears that currently there are no metrics to monitor TSDB shipper (see #9719), so my best effort for this task is to add placeholder settings

[Mahii25]

is this issue still open? I would love to work on it

[AravindAkuthota]

Hey @trevorwhitney i would like to contribute to the issue and work on it .please let me know anyone else working on it or not .

[trevorwhitney]

No, I don't think anyone is working on this at the moment. I'm not sure what the current state of the issue is either. #11087 added a tsdb boolean in the mixin config, which is used to govern the display of the structured metadata row. Are there other rows we need to put behind the same condition?