Skip to content
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

test gha runner #5

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

test gha runner #5

wants to merge 9 commits into from

Conversation

hbrueckner
Copy link
Owner

No description provided.

Signed-off-by: Hendrik Brueckner <[email protected]>
Signed-off-by: Hendrik Brueckner <[email protected]>
Signed-off-by: Hendrik Brueckner <[email protected]>
Signed-off-by: Hendrik Brueckner <[email protected]>
@hbrueckner
Copy link
Owner Author

mkdir -p build
  cd build && cmake -DUSE_BUNDLED_DEPS=OFF -DENABLE_DRIVERS_TESTS=ON -DBUILD_LIBSCAP_MODERN_BPF=ON -DMODERN_BPF_DEBUG_MODE=ON -DBUILD_BPF=True -DBUILD_LIBSCAP_GVISOR=OFF ../

Invest cmake option as those are causing socketcall tests to fail.

Signed-off-by: Hendrik Brueckner <[email protected]>
@hbrueckner
Copy link
Owner Author

cmake -DUSE_BUNDLED_DEPS=OFF -DENABLE_DRIVERS_TESTS=ON -DBUILD_LIBSCAP_MODERN_BPF=ON -DMODERN_BPF_DEBUG_MODE=ON -DBUILD_BPF=True -DBUILD_LIBSCAP_GVISOR=OFF ../

-DMODERN_PROBE_INCLUDE="-I/usr/include/s390x-linux-gnu" was missing

Commit a094db8 "cleanup: align modern bpf behavior with other drivers"
changed the drivers test in the system exit events.  This commit also
ignores `socketcall`s with wrong call IDs in the system enter event
tests.

Signed-off-by: Hendrik Brueckner <[email protected]>
The modern BPF driver maps the SYS_ACCEPT to SYS_ACCEPT4 which is being
tested in the socketcall SYS_ACCEPT tests.

The kmod and bpf drivers do not switch directly but rather ensure to
use the PPM event instead of syscall.  The socketcall SYS_ACCEPT tests
contain a specific handling for kmod.  Enable this handling to also for
the bpf driver.

Signed-off-by: Hendrik Brueckner <[email protected]>
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.

1 participant