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

database_observability: make collectors configurable #2530

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

fridgepoet
Copy link
Member

@fridgepoet fridgepoet commented Jan 24, 2025

PR Description

This enables users to select which components they want to run in the database observability component, similar to how mysqld_exporter works.

Which issue(s) this PR fixes

Fixes https://github.com/grafana/grafana-dbo11y-app/issues/370

Notes to the Reviewer

PR Checklist

  • CHANGELOG.md updated
  • Documentation added
  • Tests updated
  • Config converters updated

@fridgepoet fridgepoet marked this pull request as ready for review January 28, 2025 17:24
@fridgepoet fridgepoet marked this pull request as draft January 29, 2025 10:48
@fridgepoet fridgepoet marked this pull request as ready for review January 29, 2025 11:08
@fridgepoet fridgepoet marked this pull request as draft January 29, 2025 11:11
Copy link
Contributor

github-actions bot commented Jan 29, 2025

💻 Deploy preview available: https://deploy-preview-alloy-2530-zb444pucvq-vp.a.run.app/docs/alloy/latest/

`forward_to` | `list(LogsReceiver)` | Where to forward log entries after processing. | | yes
`collect_interval` | `duration` | How frequently to collect information from database. | `"10s"` | no
`query_samples_enabled` | `bool` | Whether to enable collection of query samples. | `true` | no
Name | Type | Description | Default | Required
Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice!

@fridgepoet fridgepoet marked this pull request as ready for review January 29, 2025 13:01
@fridgepoet fridgepoet changed the title Database Observability: Make collectors configurable database_observability: make collectors configurable Jan 29, 2025
Copy link
Collaborator

@mattdurham mattdurham left a comment

Choose a reason for hiding this comment

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

Looks good, will merge once @clayton-cornell reviews.

Copy link
Collaborator

@matthewnolf matthewnolf left a comment

Choose a reason for hiding this comment

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

Great work!

Comment on lines +33 to +49

[collectors]: #supported-collectors

## Blocks

The `database_observability.mysql` component doesn't support any blocks. You can configure this component with arguments.

### Supported Collectors

The full list of supported collectors is:

| Name | Description | Enabled by default |
|--------------|-------------------------------------------------------|--------------------|
| query_sample | Collect query samples. | yes |
| schema_table | Collect schemas and tables from `information_schema`. | yes |


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
[collectors]: #supported-collectors
## Blocks
The `database_observability.mysql` component doesn't support any blocks. You can configure this component with arguments.
### Supported Collectors
The full list of supported collectors is:
| Name | Description | Enabled by default |
|--------------|-------------------------------------------------------|--------------------|
| query_sample | Collect query samples. | yes |
| schema_table | Collect schemas and tables from `information_schema`. | yes |
The following collectors are enabled by default:
* `query_sample`: Collect query samples.
* `schema_table`: Collect schemas and tables from `information_schema`.
[collectors]: #supported-collectors
## Blocks
The `database_observability.mysql` component doesn't support any blocks. You can configure this component with arguments.

This info is probably better as a bullet list, and should be below the Args table, not a child of Blocks.

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.

5 participants