Skip to content

cover an edge case in kafka grok patterns - #13310

Merged
stefans-elastic merged 4 commits into
elastic:mainfrom
stefans-elastic:kafka-grok-pattern
Mar 27, 2025
Merged

cover an edge case in kafka grok patterns#13310
stefans-elastic merged 4 commits into
elastic:mainfrom
stefans-elastic:kafka-grok-pattern

Conversation

@stefans-elastic

@stefans-elastic stefans-elastic commented Mar 26, 2025

Copy link
Copy Markdown
Contributor

Proposed commit message

Cover the case of double spaces after log level (see screenshot below) in kafka logs.
Screenshot 2025-03-26 at 10 35 45 AM

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@stefans-elastic stefans-elastic added the bug Something isn't working, use only for issues label Mar 26, 2025
@stefans-elastic stefans-elastic self-assigned this Mar 26, 2025
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @stefans-elastic

@stefans-elastic
stefans-elastic marked this pull request as ready for review March 26, 2025 13:48
@stefans-elastic
stefans-elastic requested a review from a team as a code owner March 26, 2025 13:48
Comment thread packages/kafka/data_stream/log/_dev/test/pipeline/test-authlog.log
@andrewkroh andrewkroh added the Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] label Mar 26, 2025

@agithomas agithomas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@stefans-elastic
stefans-elastic merged commit 15b9cb5 into elastic:main Mar 27, 2025
@stefans-elastic
stefans-elastic deleted the kafka-grok-pattern branch March 27, 2025 07:50
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package kafka - 1.18.3 containing this change is available at https://epr.elastic.co/package/kafka/1.18.3/

flexitrev pushed a commit that referenced this pull request Mar 28, 2025
* cover an edge case in kafka grok patterns

* add changelog entry

* fix typos in test file names

* [kafka] fix first_line_pattern to cover auth logs format
flexitrev pushed a commit that referenced this pull request Mar 28, 2025
* cover an edge case in kafka grok patterns

* add changelog entry

* fix typos in test file names

* [kafka] fix first_line_pattern to cover auth logs format
flexitrev pushed a commit that referenced this pull request Mar 28, 2025
* cover an edge case in kafka grok patterns

* add changelog entry

* fix typos in test file names

* [kafka] fix first_line_pattern to cover auth logs format
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working, use only for issues Integration:kafka Kafka Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]

4 participants