docs: rethinking metrics approach #273
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
wtrocki
approved these changes
Oct 11, 2021
|
We planning release soon so you would be able to see those in OpenShift instance installed by operator |
wtrocki
reviewed
Oct 12, 2021
| @@ -216,7 +218,7 @@ spec: | |||
| - **Total bytes**: Total bytes is the total incoming and outgoing bytes for all topics or for a selected topic in the Kafka instance. This metric enables you to assess data transfer in and out of your Kafka instance. To modify incoming and outgoing bytes, you can adjust topic message size or other topic properties as needed. | |||
| review: | |||
| instructions: |- | |||
| Are metrics displayed for your Kafka instance and topics? You might need to wait several minutes for the latest metrics to appear. | |||
| Did you note how to navigate to the instance **Dashboard** page to view metrics later? | |||
There was a problem hiding this comment.
Not super clear what user will need to do
There was a problem hiding this comment.
@wtrocki Right, it's just an interim section until we have enough metrics content for a separate story, at which point we won't need to include this in the getting started anymore. So the main objective for the user is literally just to note how to get to this page and no need to do anything yet. Just will need to know that it exists to return so they can view metrics if needed later.
sterobin
changed the title
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.
Corresponds to docs PR #339. Previously added metrics section in getting started for lack of better alternative, but rethinking. Pending discussion and will address here.