Skip to content

[Checkpoint] Added parsing and a field to indicate sample logs - #17040

Merged
robester0403 merged 3 commits into
elastic:mainfrom
robester0403:fix-checkpoint-sample-only-parsing-error
Feb 3, 2026
Merged

[Checkpoint] Added parsing and a field to indicate sample logs#17040
robester0403 merged 3 commits into
elastic:mainfrom
robester0403:fix-checkpoint-sample-only-parsing-error

Conversation

@robester0403

@robester0403 robester0403 commented Jan 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
image

Proposed commit message

Added parsing and a field for processing packet drop sample logs.

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
@robester0403 robester0403 self-assigned this Jan 22, 2026
@robester0403
robester0403 requested a review from a team as a code owner January 22, 2026 21:17
@robester0403 robester0403 added bugfix Pull request that fixes a bug issue Integration:checkpoint Check Point Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience] labels Jan 22, 2026
@elasticmachine

elasticmachine commented Jan 22, 2026

Copy link
Copy Markdown

Pinging @elastic/integration-experience (Team:Integration-Experience)
@elasticmachine

elasticmachine commented Jan 22, 2026

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @robester0403
bhapas
bhapas previously requested changes Jan 23, 2026
View reviewed changes

@bhapas bhapas 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.

https://support.checkpoint.com/results/sk/sk144192 does not mention about sample only to be present in packets field. Would like to see if the original event from firewall contains this.
@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Jan 23, 2026
@bhapas
bhapas dismissed their stale review February 3, 2026 06:31

Dismiss

@robester0403
robester0403 merged commit 46185f3 into elastic:main Feb 3, 2026
8 checks passed
@robester0403
robester0403 deleted the fix-checkpoint-sample-only-parsing-error branch February 3, 2026 17:08
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

Package checkpoint - 1.45.2 containing this change is available at https://epr.elastic.co/package/checkpoint/1.45.2/
jakubgalecki0 pushed a commit to jakubgalecki0/integrations that referenced this pull request Feb 19, 2026
@robester0403 @jakubgalecki0
[Checkpoint] Added parsing and a field to indicate sample logs (elast…
623562e 
…ic#17040)

Added parsing and a field for processing packet drop sample logs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes a bug issue documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:checkpoint Check Point Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience]

4 participants

@robester0403 @elasticmachine @bhapas @andrewkroh