-
Notifications
You must be signed in to change notification settings - Fork 176
feat: extend PPME_SYSCALL_INOTIFY_INIT_X with enter parameters
#2514
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
Conversation
Add `PPME_SYSCALL_INOTIFY_INIT_E` parameters to `PPME_SYSCALL_INOTIFY_INIT_X` event definition and align all 3 kernel drivers to it. Add new rules to scap file converter table to convert events in old scap files to the new layout. Add/update inotify_init-related drivers, scap converter and sinsp parser tests to account the new layout. Signed-off-by: Leonardo Di Giovanna <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2514 +/- ##
==========================================
+ Coverage 78.39% 78.40% +0.01%
==========================================
Files 284 285 +1
Lines 31740 31763 +23
Branches 4643 4643
==========================================
+ Hits 24881 24904 +23
Misses 6859 6859
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
FedeDP
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
|
LGTM label has been added. DetailsGit tree hash: c7c9b1561425fa9b136ac128e6d396be19c40e1d |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ekoops, FedeDP The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
X64 kernel testing matrix
ARM64 kernel testing matrix
|
Perf diff from master - unit testsHeap diff from master - unit testsHeap diff from master - scap fileBenchmarks diff from master |
What type of PR is this?
/kind test
/kind feature
Any specific area of the project related to this PR?
/area driver-kmod
/area driver-bpf
/area driver-modern-bpf
/area libscap-engine-gvisor
/area libscap-engine-savefile
/area libscap
/area libsinsp
/area tests
Does this PR require a change in the driver versions?
/version driver-SCHEMA-version-minor
What this PR does / why we need it:
This PR is part of #2427.
It:
PPME_SYSCALL_INOTIFY_INIT_Eparameters toPPME_SYSCALL_INOTIFY_INIT_Xevent definition and aligns all 3 kernel drivers to itWhich issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
/milestone 0.22.0
Does this PR introduce a user-facing change?: