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

feat(tests): new e2e tests [7/N] #1814

Merged
merged 27 commits into from
May 21, 2024
Merged
Show file tree
Hide file tree
Changes from 24 commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
6714ab9
fix(bpf): better timespec support for ia32
therealbobo May 8, 2024
9318654
new(libsinsp_e2e): new misc tests
therealbobo Apr 29, 2024
b4da9e9
fix(bpf,modern_bpf): better 32bit support preadv/pwritev family
therealbobo May 2, 2024
c9e025b
update(e2e_ci): bump actions
therealbobo May 2, 2024
1169541
fix(libsinsp_e2e): dump only necessary events
therealbobo May 3, 2024
958c2f0
fix(modern_bpf): avoid overflow reading sockaddr_storage
therealbobo May 3, 2024
8fb9c41
fix(libsinsp_e2e): add retry to subprocess class
therealbobo May 4, 2024
640d3b7
fix(ci): workaround to avoid random failures of asan
therealbobo May 5, 2024
159a3e3
fix(bpf): cast incompatibile pointer
therealbobo May 6, 2024
1ed4779
cleanup(bpf): remove useless ifdefs
therealbobo May 6, 2024
85f1c74
cleanup(bpf): hide ia32 login in helper function
therealbobo May 6, 2024
ffc0cf9
fix(bpf): fix bpf on aarch64
therealbobo May 7, 2024
6d67928
refactor(bpf): split ia32 bpf_parse_readv_writev_bufs helper
therealbobo May 8, 2024
9990067
refactor(bpf): split bpf_parse_readv_writev_bufs even more
therealbobo May 9, 2024
c01c519
fix(modern_bpf): read MAX_UNIX_SOCKET_PATH from sockaddr
therealbobo May 12, 2024
ad811d1
fix(libsinsp_e2e): remove flaky test
therealbobo May 12, 2024
3316ca9
cleanup(bpf): rename 32/64bit bpf helpers
therealbobo May 12, 2024
5b2edcf
cleanup(modern_bpf,bpf): use already defined struct
therealbobo May 12, 2024
6e31a35
refactor(modern_bpf): split iovec helper in 32/64 bit variants
therealbobo May 12, 2024
3a3de12
fix(bpf): use CONFIG_COMPAT to exclude ia32 calls
therealbobo May 16, 2024
f0596b3
fix(modern_bpf): define the compat timespec in struct flavors
therealbobo May 16, 2024
7d09814
fix(bpf): fix missing definitions
therealbobo May 17, 2024
46324e7
fix(libsinsp_e2e): increase tcp_client_server test stability
therealbobo May 17, 2024
d35a96f
fix(bpf): address review comments
therealbobo May 20, 2024
3ccf016
fix(modern_bpf): address review comments
therealbobo May 20, 2024
f47b808
fix(libsinsp_e2e): increase tcp_client_server* tests stability
therealbobo May 21, 2024
7101943
fix(test/e2e): pin python requests version
therealbobo May 21, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -293,6 +293,10 @@ jobs:
git config --global --add safe.directory $GITHUB_WORKSPACE

- name: Build and test 🏗️🧪
env:
# This avoids random failures on CI.
# (https://github.com/google/sanitizers/issues/1322#issuecomment-699946942)
ASAN_OPTIONS: intercept_tls_get_addr=0
run: |
mkdir -p build && cd build
cmake -DBUILD_BPF=ON \
Expand All @@ -302,8 +306,8 @@ jobs:
-DUSE_BUNDLED_LIBBPF=ON \
..
make -j$(nproc) sinsp-example driver bpf
sudo make e2e-install-deps
sudo ../test/e2e/scripts/run_tests.sh
sudo -E make e2e-install-deps
sudo -E ../test/e2e/scripts/run_tests.sh

- name: Archive test reports
uses: actions/upload-artifact@a8a3f3ad30e3422c9c7b888a15615d19a852ae32 # v3.1.3
Expand Down
16 changes: 10 additions & 6 deletions .github/workflows/e2e_ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ jobs:
sudo apt install -y --no-install-recommends linux-headers-$(uname -r) gcc-multilib g++-multilib

- name: Run sccache-cache
uses: mozilla-actions/sccache-action@v0.0.3
uses: mozilla-actions/sccache-action@v0.0.4

- name: Build e2e tests 🏗️
env:
Expand All @@ -96,10 +96,11 @@ jobs:
-DUSE_BUNDLED_GTEST=ON \
..
make -j6 libsinsp_e2e_tests
sudo rm -vfr test/libsinsp_e2e/resources/_proc
cd ..

- name: Cache build
uses: actions/cache/save@v3
uses: actions/cache/save@v4.0.2
if: always()
id: cache
with:
Expand All @@ -123,7 +124,7 @@ jobs:

- name: Restore build
id: cache
uses: actions/cache/restore@v3
uses: actions/cache/restore@v4.0.2
with:
path: build
key: build-e2e-${{ matrix.arch }}-${{ github.run_id }}
Expand All @@ -146,7 +147,7 @@ jobs:

- name: Install deps
run: |
sudo apt install -y --no-install-recommends clang gcc llvm build-essential cmake
sudo apt install -y --no-install-recommends clang gcc llvm build-essential cmake python2

- name: Install kernel headers (actuated)
uses: self-actuated/get-kernel-sources@201eed7d915ac0a6021fb402cde5be7a6b945b59
Expand All @@ -161,9 +162,12 @@ jobs:
# different workers, so we rebuild the drivers.
- name: Rebuild drivers
run: |
cd build
pushd build
make -B driver bpf
cd ..
pushd test/libsinsp_e2e/resources/
sudo tar xzf fake-proc.tar.gz
popd
popd

- name: Run e2e tests with ${{ matrix.driver.name }} 🏎️
if: matrix.arch == 'amd64'
Expand Down
2 changes: 1 addition & 1 deletion driver/SCHEMA_VERSION
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2.20.0
2.20.1
Loading
Loading