bpftime icon indicating copy to clipboard operation
bpftime copied to clipboard

[FEATURE] Add CI to test bpftrace

Open tdaudi opened this issue 1 year ago • 3 comments

Description

This PR is base on the requested feature to add bpftrace GitHub actions.

Fixes # (issue) : github.com/eunomia-bpf/bpftime/issues/228

Type of change

  • [x] New feature (non-breaking change which adds functionality)

Checklist

  • [x] My code follows the style guidelines of this project
  • [x] I have performed a self-review of my own code
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [ ] My changes generate no new warnings
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [ ] New and existing unit tests pass locally with my changes
  • [ ] Any dependent changes have been merged and published in downstream modules
  • [ ] I have checked my code and corrected any misspellings

tdaudi avatar Apr 10 '24 10:04 tdaudi

Several problems remain with the draft.

  • [ ] I use apt install bpftrace to install bpftrace. Problem is, on ubuntu the package is not up-to-date. So next step will be to clone bpftrace and install it. But the downside will be the image will be incredibly heavy, and this test will take ages to perform ...
  • [ ] I need to fix the error [error] [agent.cpp:105] Unable to initialize shared memory: No such file or directory when using the script inside Docker. I still don't know why this happened

tdaudi avatar Apr 10 '24 10:04 tdaudi

If you want to work on this later on, you have to notice two things :

  • To test bpftrace you would like to build bpftrace from source inside the GitHub action to use the latest version, which is not trivial and may slow down a lot the running task.
  • GitHub does not yet have Ubuntu 24 available for actions, so if you want to use the built-in package, you have to test bpftime on an older kernel or old bpftrace package. If breaking change occurs, you won't know

tdaudi avatar Aug 27 '24 09:08 tdaudi

Thanks @ScriptSathi!

yunwei37 avatar Aug 29 '24 16:08 yunwei37