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

Update libbpf #393

Closed
wants to merge 4 commits into from
Closed

Update libbpf #393

wants to merge 4 commits into from

Conversation

thiagoftsm
Copy link
Contributor

Summary

This PR is updating LIBBPF and also adding binaries for CentOS 7 while we do not address LSF issues.

Test Plan
  1. Get binaries according to your C library from [this](ADD ACTIONS LINK HERE) link and extract them inside a directory, for example: ../artifacts.
    You can also get everything for glibc [here](UPLOAD FILE WITH ALL BINARIES TO SIMPLIFY REVIEWERS).

  2. Extract them running:

    $ for i in `ls *.zip`; do unzip $i; rm .gitkeep ; rm $i; done
    $ for i in `ls *.xz`; do tar -xf $i; rm $i* ; done
  3. Compile branch an run the following tests:

    # make clean; make tester
    # for i in `seq 0 3`; do ./kernel/legacy_test --netdata-path ../artifacts --content --iteration 1 --pid $i --log-path file_pid$i.txt; done
  4. Every test should ends with Success, unless you do not have a specific target (function) available.

Additional information

This PR was tested on:

Linux Distribution Environment Kernel Version Real Parent Parent All PIDs Without PIDs
LINUX DISTRIBUION Bare metal/VM uname -r

@thiagoftsm thiagoftsm requested a review from Ferroin as a code owner July 23, 2024 02:31
@thiagoftsm
Copy link
Contributor Author

Hello @Ferroin,

I have to merge this PR to update libbpf and proceed with a bug fix (netdata/netdata#18199), please, take a look on it.

Best regards!

@thiagoftsm
Copy link
Contributor Author

I am closing in favor of #392

@thiagoftsm thiagoftsm closed this Jul 24, 2024
@thiagoftsm thiagoftsm deleted the update_libbpf2 branch July 24, 2024 14:10
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