Skip to content

Conversation

@wilskong
Copy link

What does this PR do?

PR removes a log sample from the first Grok Parser of the Postgresql Integration Pipeline.

Grok Parsers allow a maximum of 5 log samples but this Grok Parser currently contains 6 samples. If a user clones this pipeline and edits any processor within it, they will be unable to save changes to the pipeline.

The sample that is removed was listed as the second sample but falls under the same rule as the first sample (default_format)

Motivation

A customer cloned the Postgresql Integration Pipeline recently and was unable to save changes to their clone. The error that was returned read "Creation of processor could not be completed successfully" without additional context.

see https://datadoghq.atlassian.net/browse/LOGSS-9623 for more details.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Removing a log sample from the first Grok Parser of the Postgresql Integration Pipeline. 

Grok Parsers allow a maximum of 5 log samples but this Grok Parser currently contains 6 samples. If a user clones this pipeline and edits any processor within it, they will be unable to save changes to the pipeline.

The sample that is removed was listed as the second sample but falls under the same rule as the first sample (default_format)
@wilskong wilskong requested a review from a team as a code owner November 25, 2025 19:44
@wilskong wilskong changed the title Update postgresql.yaml Update postgresql.yaml to reduce number of Grok Parser Samples Nov 25, 2025
@github-actions
Copy link

⚠️ Recommendation: Add qa/skip-qa label

This PR does not modify any files shipped with the agent.

To help streamline the release process, please consider adding the qa/skip-qa label if these changes do not require QA testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants