Skip to content

Prometheus Sink: Fix setting DLQ pipeline, add NOISY marker for logs#6388

Merged
kkondaka merged 1 commit into
opensearch-project:mainfrom
kkondaka:prom-sink-fixes
Jan 5, 2026
Merged

Prometheus Sink: Fix setting DLQ pipeline, add NOISY marker for logs#6388
kkondaka merged 1 commit into
opensearch-project:mainfrom
kkondaka:prom-sink-fixes

Conversation

@kkondaka

@kkondaka kkondaka commented Jan 5, 2026

Copy link
Copy Markdown
Collaborator

Description

Fix Prometheus Sink -

  1. Add NOISY markers to LOG messages
  2. Fix setting of DLQ pipeline by setting it during sink initialization and not during sink object construction

Issues Resolved

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • [ X] New functionality includes testing.
  • New functionality has a documentation issue. Please link to it in this PR.
    • New functionality has javadoc added
  • [X ] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Krishna Kondaka <krishkdk@amazon.com>

@dlvenable dlvenable left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @kkondaka !

@kkondaka kkondaka merged commit 502f601 into opensearch-project:main Jan 5, 2026
45 of 51 checks passed
wandna-amazon pushed a commit to wandna-amazon/data-prepper that referenced this pull request Jan 8, 2026
simonelbaz pushed a commit to simonelbaz/data-prepper that referenced this pull request Jan 31, 2026
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.

3 participants