Skip to content

feat(database_observability.postgres): Add exclude databases/users for logs collector#5569

Merged
cristiangreco merged 1 commit intomainfrom
cristian/dbo11y-pglogs-exclude-users-databases
Feb 19, 2026
Merged

feat(database_observability.postgres): Add exclude databases/users for logs collector#5569
cristiangreco merged 1 commit intomainfrom
cristian/dbo11y-pglogs-exclude-users-databases

Conversation

@cristiangreco
Copy link
Contributor

Brief description of Pull Request

Like we do in other collectors, propagate exclude_databases and exclude_users config options to the logs collector, allowing to specify databases and/or users that should be excluded from log parsing and metrics generation.

Pull Request Details

Issue(s) fixed by this Pull Request

Notes to the Reviewer

PR Checklist

  • Documentation added
  • Tests updated
  • Config converters updated

Comment on lines -40 to -45
type ParsedError struct {
ErrorSeverity string
SQLStateCode string
SQLStateClass string
User string
DatabaseName string
Copy link
Contributor Author

@cristiangreco cristiangreco Feb 18, 2026

Choose a reason for hiding this comment

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

Removed struct in favour of updating the metric directly.

…r logs collector

Like we do in other collectors, propagate `exclude_databases` and
`exclude_users` config options to the logs collector, allowing to
specify databases and/or users that should be excluded from log parsing
and metrics generation.
@cristiangreco cristiangreco force-pushed the cristian/dbo11y-pglogs-exclude-users-databases branch from 96ef086 to f66bd1a Compare February 18, 2026 11:02
@cristiangreco cristiangreco marked this pull request as ready for review February 18, 2026 11:09
@cristiangreco cristiangreco requested a review from a team as a code owner February 18, 2026 11:09
Copy link
Contributor

@gaantunes gaantunes 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 bff5060 into main Feb 19, 2026
47 of 49 checks passed
@cristiangreco cristiangreco deleted the cristian/dbo11y-pglogs-exclude-users-databases branch February 19, 2026 07:09
@grafana-alloybot grafana-alloybot bot mentioned this pull request Feb 19, 2026
jharvey10 pushed a commit that referenced this pull request Feb 26, 2026
…r `logs` collector (#5569)

### Brief description of Pull Request
Like we do in other collectors, propagate `exclude_databases` and
`exclude_users` config options to the logs collector, allowing to
specify databases and/or users that should be excluded from log parsing
and metrics generation.

### 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
- [ ] Tests updated
- [ ] Config converters updated
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants