Upgrade Grafana to the latest version. #47
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 tasks
jhesketh
approved these changes
Nov 4, 2022
| 1. From the panel drop-down, click Explore. | ||
| 1. In the Explore view, go to the data source drop-down and select Loki. | ||
| 1. Choose a logline with a traceID of Tempo. | ||
| 2. Zoom in on a section of a panel with failed requests. |
There was a problem hiding this comment.
(nit) The repeated 1.'s here will actually render as a correct numbered list in markdown. This is helpful to keep as you can easily add an item in an arbitrary position and not have to renumber everything. (But I see that other lists in this doc don't follow this convention so /shrug).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Grafana 8.2.2 is over a year old, so this PR brings us to a more recent version.
I've tested this manually by running through the steps in the readme and everything seems to be working, but a second check before merging would probably be a good idea.