Skip to content

feat: Database_observability.mysql: support excluding schemas in all collectors#5350

Merged
cristiangreco merged 1 commit intomainfrom
cristian/dbo11y-exclude-schemas-all-collectors
Jan 27, 2026
Merged

feat: Database_observability.mysql: support excluding schemas in all collectors#5350
cristiangreco merged 1 commit intomainfrom
cristian/dbo11y-exclude-schemas-all-collectors

Conversation

@cristiangreco
Copy link
Contributor

@cristiangreco cristiangreco commented Jan 26, 2026

Brief description of Pull Request

Propagate exclude_schemas configuration to all MySQL collectors and append the user-defined schemas to the default excluded schemas.
Additionally, change the behaviour of explain_plans collector to skip queries from excluded schemas instead of processing them.

Pull Request Details

Issue(s) fixed by this Pull Request

Notes to the Reviewer

PR Checklist

  • Documentation added
  • Tests updated
  • Config converters updated

@cristiangreco cristiangreco changed the title feat: database_observability.mysql: support excluding schemas in all … feat: database_observability.mysql: support excluding schemas in all collectors Jan 26, 2026
@cristiangreco cristiangreco changed the title feat: database_observability.mysql: support excluding schemas in all collectors feat: Database_observability.mysql: support excluding schemas in all collectors Jan 26, 2026
@cristiangreco cristiangreco force-pushed the cristian/dbo11y-exclude-schemas-all-collectors branch 2 times, most recently from 6018db1 to 198112f Compare January 26, 2026 14:19
Comment on lines -1908 to -1912
"some_schema",
"some_digest1",
"select * from some_table where id = 1",
lastSeen,
).AddRow(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed because now with exclude_schemas we don't expect to fetch queries that belong to some_schema.

…collectors

Propagate `exclude_schemas` configuration to all MySQL collectors and
append the user-defined schemas to the default excluded schemas.
Additionally, change the behaviour of explain_plans collector to skip
queries from excluded schemas instead of processing them.
@cristiangreco cristiangreco force-pushed the cristian/dbo11y-exclude-schemas-all-collectors branch from 198112f to 69df4a0 Compare January 26, 2026 14:47
@cristiangreco cristiangreco marked this pull request as ready for review January 26, 2026 14:48
@cristiangreco cristiangreco requested a review from a team as a code owner January 26, 2026 14:48
Copy link
Contributor

@rgeyer rgeyer left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@cristiangreco cristiangreco merged commit d3542bc into main Jan 27, 2026
48 checks passed
@cristiangreco cristiangreco deleted the cristian/dbo11y-exclude-schemas-all-collectors branch January 27, 2026 10:43
@grafana-alloybot grafana-alloybot bot mentioned this pull request Jan 27, 2026
cristiangreco added a commit that referenced this pull request Jan 27, 2026
…ll collectors

Propagate `exclude_databases` configuration to all PostgreSQL collectors
and append the user-defined databases to the default excluded databases.

Additionally, change the behaviour of `explain_plans` collector to skip
queries from excluded databases instead of processing them.

Followup of #5350
cristiangreco added a commit that referenced this pull request Jan 27, 2026
…ll collectors

Propagate `exclude_databases` configuration to all PostgreSQL collectors
and append the user-defined databases to the default excluded databases.

Additionally, change the behaviour of `explain_plans` collector to skip
queries from excluded databases instead of processing them.

Followup of #5350
cristiangreco added a commit that referenced this pull request Jan 27, 2026
…ll collectors

Propagate `exclude_databases` configuration to all PostgreSQL collectors
and append the user-defined databases to the default excluded databases.

Additionally, change the behaviour of `explain_plans` collector to skip
queries from excluded databases instead of processing them.

Followup of #5350
cristiangreco added a commit that referenced this pull request Jan 27, 2026
…ll collectors

Propagate `exclude_databases` configuration to all PostgreSQL collectors
and append the user-defined databases to the default excluded databases.

Additionally, change the behaviour of `explain_plans` collector to skip
queries from excluded databases instead of processing them.

Followup of #5350
cristiangreco added a commit that referenced this pull request Jan 28, 2026
…ollectors

Propagate `exclude_databases` configuration to all PostgreSQL collectors
and append the user-defined databases to the default excluded databases.

Additionally, change the behaviour of `explain_plans` collector to skip
queries from excluded databases instead of processing them.

Followup of #5350
cristiangreco added a commit that referenced this pull request Jan 28, 2026
…ollectors

Propagate `exclude_databases` configuration to all PostgreSQL collectors
and append the user-defined databases to the default excluded databases.

Additionally, change the behaviour of `explain_plans` collector to skip
queries from excluded databases instead of processing them.

Followup of #5350
cristiangreco added a commit that referenced this pull request Jan 28, 2026
…ollectors

Propagate `exclude_databases` configuration to all PostgreSQL collectors
and append the user-defined databases to the default excluded databases.

Additionally, change the behaviour of `explain_plans` collector to skip
queries from excluded databases instead of processing them.

Followup of #5350
cristiangreco added a commit that referenced this pull request Jan 28, 2026
…ollectors

Propagate `exclude_databases` configuration to all PostgreSQL collectors
and append the user-defined databases to the default excluded databases.

Additionally, change the behaviour of `explain_plans` collector to skip
queries from excluded databases instead of processing them.

Followup of #5350
cristiangreco added a commit that referenced this pull request Jan 28, 2026
…ollectors (#5366)

### Brief description of Pull Request
Propagate `exclude_databases` configuration to all PostgreSQL collectors
and append the user-defined databases to the default excluded databases.

Additionally, change the behaviour of `explain_plans` collector to skip
queries from excluded databases instead of processing them.

Followup of #5350

### Pull Request Details

<!-- Add a more detailed descripion of the Pull Request here, if needed.
-->

### Issue(s) fixed by this Pull Request

<!--
Uncomment the following line and fill in an issue number if you want a
GitHub
  issue to be closed automatically when this PR gets merged.
-->

<!-- Fixes #issue_id -->

### Notes to the Reviewer

<!-- Add any relevant notes for the reviewers and testers of this PR.
-->

### PR Checklist

<!-- Remove items that do not apply. For completed items, change [ ] to
[x]. -->

- [ ] Documentation added
- [x] Tests updated
- [ ] Config converters updated
@ptodev ptodev added the backport/v1.13 Backport to release/v1.13 label Jan 28, 2026
grafana-alloybot bot pushed a commit that referenced this pull request Jan 28, 2026
…collectors (#5350)

### Brief description of Pull Request
Propagate `exclude_schemas` configuration to all MySQL collectors and
append the user-defined schemas to the default excluded schemas.
Additionally, change the behaviour of explain_plans collector to skip
queries from excluded schemas instead of processing them.

### Pull Request Details

<!-- Add a more detailed descripion of the Pull Request here, if needed.
-->

### Issue(s) fixed by this Pull Request

<!--
Uncomment the following line and fill in an issue number if you want a
GitHub
  issue to be closed automatically when this PR gets merged.
-->

<!-- Fixes #issue_id -->

### Notes to the Reviewer

<!-- Add any relevant notes for the reviewers and testers of this PR.
-->

### PR Checklist

<!-- Remove items that do not apply. For completed items, change [ ] to
[x]. -->

- [ ] Documentation added
- [x] Tests updated
- [ ] Config converters updated

(cherry picked from commit d3542bc)
ptodev pushed a commit that referenced this pull request Jan 28, 2026
…collectors [backport] (#5380)

## Backport of #5350

This PR backports #5350 to release/v1.13.

### Original PR Author
@cristiangreco

### Description
### Brief description of Pull Request
Propagate `exclude_schemas` configuration to all MySQL collectors and
append the user-defined schemas to the default excluded schemas.
Additionally, change the behaviour of explain_plans collector to skip
queries from excluded schemas instead of processing them.

### Pull Request Details

<!-- Add a more detailed descripion of the Pull Request here, if needed.
-->

### Issue(s) fixed by this Pull Request

<!--
Uncomment the following line and fill in an issue number if you want a
GitHub
  issue to be closed automatically when this PR gets merged.
-->

<!-- Fixes #issue_id -->

### Notes to the Reviewer

<!-- Add any relevant notes for the reviewers and testers of this PR.
-->

### PR Checklist

<!-- Remove items that do not apply. For completed items, change [ ] to
[x]. -->

- [ ] Documentation added
- [x] Tests updated
- [ ] Config converters updated


---
*This backport was created automatically.*

Co-authored-by: Cristian Greco <cristian@regolo.cc>
grafana-alloybot bot pushed a commit that referenced this pull request Jan 28, 2026
…ollectors (#5366)

### Brief description of Pull Request
Propagate `exclude_databases` configuration to all PostgreSQL collectors
and append the user-defined databases to the default excluded databases.

Additionally, change the behaviour of `explain_plans` collector to skip
queries from excluded databases instead of processing them.

Followup of #5350

### Pull Request Details

<!-- Add a more detailed descripion of the Pull Request here, if needed.
-->

### Issue(s) fixed by this Pull Request

<!--
Uncomment the following line and fill in an issue number if you want a
GitHub
  issue to be closed automatically when this PR gets merged.
-->

<!-- Fixes #issue_id -->

### Notes to the Reviewer

<!-- Add any relevant notes for the reviewers and testers of this PR.
-->

### PR Checklist

<!-- Remove items that do not apply. For completed items, change [ ] to
[x]. -->

- [ ] Documentation added
- [x] Tests updated
- [ ] Config converters updated

(cherry picked from commit 1fe42db)
ptodev pushed a commit that referenced this pull request Jan 28, 2026
…ollectors [backport] (#5383)

## Backport of #5366

This PR backports #5366 to release/v1.13.

### Original PR Author
@cristiangreco

### Description
### Brief description of Pull Request
Propagate `exclude_databases` configuration to all PostgreSQL collectors
and append the user-defined databases to the default excluded databases.

Additionally, change the behaviour of `explain_plans` collector to skip
queries from excluded databases instead of processing them.

Followup of #5350

### Pull Request Details

<!-- Add a more detailed descripion of the Pull Request here, if needed.
-->

### Issue(s) fixed by this Pull Request

<!--
Uncomment the following line and fill in an issue number if you want a
GitHub
  issue to be closed automatically when this PR gets merged.
-->

<!-- Fixes #issue_id -->

### Notes to the Reviewer

<!-- Add any relevant notes for the reviewers and testers of this PR.
-->

### PR Checklist

<!-- Remove items that do not apply. For completed items, change [ ] to
[x]. -->

- [ ] Documentation added
- [x] Tests updated
- [ ] Config converters updated


---
*This backport was created automatically.*

Co-authored-by: Cristian Greco <cristian@regolo.cc>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport/v1.13 Backport to release/v1.13 frozen-due-to-age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants