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

[docs][sc-105732] Update the Fluent Bit monitoring page #1453

Conversation

lockewritesdocs
Copy link
Contributor

@lockewritesdocs lockewritesdocs commented Sep 6, 2024

This PR updates the Fluent Bit monitoring page for style and consistency with Chronosphere guidelines.

@lockewritesdocs lockewritesdocs self-assigned this Sep 6, 2024
@lockewritesdocs lockewritesdocs requested review from a team as code owners September 6, 2024 19:58
Copy link
Contributor

@esmerel esmerel left a comment

Choose a reason for hiding this comment

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

minor comment but this is MUCH cleaner.

Comment on lines 10 to 12
Fluent Bit includes features for monitoring the internals of your pipeline, and for
connecting to Prometheus and Grafana, Health checks, and connectors to use external
services:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Fluent Bit includes features for monitoring the internals of your pipeline, and for
connecting to Prometheus and Grafana, Health checks, and connectors to use external
services:
Fluent Bit includes features for monitoring the internals of your pipeline,
connecting to Prometheus and Grafana, Health checks, and connectors to use external
services:

multiple ands in there feels awkward

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mmm, yeah -- good callout! I'll change to read:

Fluent Bit includes features for monitoring the internals of your pipeline, in
addition to connecting to Prometheus and Grafana, Health checks, and connectors to
use external services:

| /api/v2/reload | Execute hot reloading or get the status of hot reloading. See the [hot-reloading documentation](hot-reload.md). | JSON |

### v1 metrics

Copy link
Contributor

Choose a reason for hiding this comment

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

should be an sentence here between headers but it's not a blocker, same for below, but that can totally be iterated later.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ACK -- will add a sentence there 👍

@lockewritesdocs lockewritesdocs merged commit 9b39bf8 into master Sep 9, 2024
5 checks passed
@lockewritesdocs lockewritesdocs deleted the adamlocke/sc-105732/update-administration-monitoring-md-fluent branch September 9, 2024 11:11
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.

2 participants