-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Filebeat] Update checkpoint.source_object
mapping
#25145
[Filebeat] Update checkpoint.source_object
mapping
#25145
Conversation
4ea682c
to
5a5acb4
Compare
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪💚 Flaky test reportTests succeeded. Expand to view the summary
Test stats 🧪
|
@andrewkroh @andrewstucki could you run the CI tests? |
run tests |
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
@andrewkroh I think we're good to merge unless you have any comments. |
run tests |
Looks like all the tests passed |
I suspected that this fail after it merged to master because in master I pinned the geoip database version which affected the contents of the generated files. So I updated the branch and regenerated the test output files. |
run tests |
What does this PR do?
Changes
checkpoint.source_object
from Long to Keyword per https://supportcenter.checkpoint.com/supportcenter/portal?eventSubmit_doGoviewsolutiondetails=&solutionid=sk144192Why is it important?
Field should be a string, not number
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs