tempo icon indicating copy to clipboard operation
tempo copied to clipboard

[Doc] Update or create doc for Tempo config using Helm

Open knylander-grafana opened this issue 2 years ago • 2 comments

Update the HELM deployment instructions for Tempo. Right now we have a pointer to a Helm chart that goes to these places:

  • GET: https://github.com/suitupalex/helm-charts/tree/enterprise-traces/charts/enterprise-traces
  • Tempo: https://github.com/grafana/tempo/tree/main/example/helm

We don't have any instructions for Tempo for Helm configuration. Our instructions for Tempo point to the above repo link.

  • Community has a Helm chart.
  • Is the Helm chart for GET linked above provided by Grafana?

Possible SME: @zalegrala is researching Helm charts -- See https://github.com/grafana/helm-charts/pull/1701

Example Mimir docs:

  • Doc in repo: https://github.com/grafana/mimir/blob/main/docs/sources/operators-guide/deploy-grafana-mimir/getting-started-helm-charts/_index.md
  • Helm chart get started guide in Mimir: https://grafana.com/docs/mimir/latest/operators-guide/deploy-grafana-mimir/getting-started-helm-charts/

knylander-grafana avatar Aug 09 '22 21:08 knylander-grafana

I've started some modifications on the Helm chart above. I think I should be able to get it working today.

zalegrala avatar Aug 10 '22 13:08 zalegrala

See Mimir doc for Helm: https://github.com/grafana/mimir/pull/2299/files

Also, several members of the Doc Squad (Ursula, me, Chris, Fiona) are working with Helm chart people (Zach Leslie, Krajo) to work on how Helm chart documentation is presented to our customers. See (internal to Grafana Labs) Notes - Helm char docs Google file for notes.

knylander-grafana avatar Aug 24 '22 14:08 knylander-grafana

@knylander-grafana - has this content been built out? and @osg-grafana - Will this be part of the new Helm docs approach?

GrafanaWriter avatar Dec 20 '22 21:12 GrafanaWriter

@GrafanaWriter I'm going to schedule some time with @hedss to write the Tempo and GET doc for the helm chart for the Tempo 2.0 release. @osg-grafana should be a reviewer on this project.

knylander-grafana avatar Jan 03 '23 16:01 knylander-grafana

See Loki blog post about Helm charts - https://grafana.com/blog/2022/12/19/the-only-helm-chart-you-need-for-grafana-loki-is-here/

New Helm portal is supposed to go live on Jan 9th. https://github.com/grafana/website/issues/10900#issuecomment-1355099192

knylander-grafana avatar Jan 04 '23 21:01 knylander-grafana

Moving to in progress based on the comments above 🎉

GrafanaWriter avatar Jan 10 '23 22:01 GrafanaWriter

@hedss and I met last week and started documenting how to use the tempo-distributed Helm chart. Heds found two issues with the Helm charts. He created a PR to address these issues ( https://github.com/grafana/helm-charts/issues/2111, https://github.com/grafana/helm-charts/issues/2112).

Unfortunately, we can't proceed with the doc until the above issue is resolved.

This issue also affects https://github.com/grafana/enterprise-traces/issues/403.

knylander-grafana avatar Jan 18 '23 00:01 knylander-grafana

@joe-elliott has been working on the Tempo helm-charts repo to address some of the issues (not related to the issue Heds found).

I'm going to meet with Zach today to see what we can do to create a basic Helm cart doc for Tempo.

knylander-grafana avatar Feb 01 '23 18:02 knylander-grafana

@zalegrala and I have created a draft Helm procedure for Tempo. I've been able to walk through the procedure and install the Helm chart itself and have the nodes start in the K8s cluster on my laptop. However, I have to work on storage configuration to get things working correctly.

knylander-grafana avatar Feb 10 '23 20:02 knylander-grafana

@osg-grafana: @jdbaldry has a PR in the Tempo repo (https://github.com/grafana/tempo/pull/2096) to move the doc files so they mirror the Mimir repo's doc structure. This will let us have the Tempo Helm doc appear in the Helm doc portal.

@GrafanaWriter So far, we only have a single Helm page. The focus for now is to get the content to the site and the iterate and add more content.

knylander-grafana avatar Feb 10 '23 20:02 knylander-grafana

Last Friday, Zach and I spent two hours reviewing the doc and updating. The PR was shared with the internal tempo and Helm channels for review.

knylander-grafana avatar Feb 28 '23 21:02 knylander-grafana

Should we use the values.yaml or the custom.yaml file? After reading through the values.yaml file for the tempo-distributed helm chart, I updated the Helm doc PR with suggestions for using values.yaml. These changes are not merged, however the procedure itself works. I can test things, but I don't know how to verify that the Helm values are set properly.

Zach has also created a PR with Helm chart examples, including custom.yaml: https://github.com/grafana/tempo/pull/2124

This makes me think that the better option is to use the custom.yaml file. The values.yaml file is pretty complicated and would be nice to have as simple as possible.

The questions I have are:

  • What is the best practice?
  • If values are not set in customer.yaml (like the username and passwords), are those values defaulted to the ones present in values.yaml?

I am going to meet with @zalegrala this week to hopefully answer these questions.

knylander-grafana avatar Mar 07 '23 15:03 knylander-grafana