Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change to TOC structure and other reorg to individual topics #365

Conversation

achatterjee-grafana
Copy link
Contributor

Change to TOC and general restructuring based on #356.

@CLAassistant
Copy link

CLAassistant commented Nov 24, 2020

CLA assistant check
All committers have signed the CLA.

@joe-elliott joe-elliott merged commit 2e7a500 into master Nov 24, 2020
@joe-elliott joe-elliott deleted the Change-to-TOC-structure-and-other-reorg-to-individual-topics branch November 24, 2020 04:08
Copy link
Contributor

@annanay25 annanay25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @achatterjee-grafana!

One more thing, can we remove the .DS_Store files from the repo as part of a future PR?


### Spin up Tempo backend
> **Note:** The Grafana Cloud Agent is already set up to use Tempo. Refer to the [configuration](https://github.com/grafana/agent/blob/master/docs/configuration-reference.md#tempo_config) and [example](https://github.com/grafana/agent/blob/master/example/docker-compose/agent/config/agent.yaml) for details.
### Step 1: Spin up Tempo backend
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: This needs to be a ## to match the header size with other steps


## Setting up Tempo
Getting started with Tempo is easy. For an application already instrumented for tracing, this guide can help quickly set it up with Tempo. If you're looking for a demo application to play around with Tempo, skip to the [examples with demo app]({{< relref "example-demo-app.md" >}})topic.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: We need a space between the link and the following word here - [examples with demo app]({{< relref "example-demo-app.md" >}})topic

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joe-elliott, did you fix the nits @annanay25 pointed out? LMK, thanks!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@achatterjee-grafana i did not. merged blindly!

feel free to submit a new PR :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants