exein-io / pulsar

A modular and blazing fast runtime security tool for the IoT, powered by eBPF.
https://pulsar.sh
Other
888 stars 51 forks source link

ci: Use architest for running test suite across architectures #266

Closed vadorovsky closed 3 months ago

vadorovsky commented 6 months ago
banditopazzo commented 4 months ago

I think we should remove entirely the Cross as wrapper around cross because it's redundant: https://github.com/exein-io/pulsar/blob/018fd568c306916e7241cc89b94da53ece15a64a/xtask/src/main.rs#L25-L28

Then move and refactor the current xtask/cross as part of Test. As the command line option says, the xtask/test command is about the test-suite (Run eBPF test suite) and a good solution is to have that method taking cross arguments in case someone want to run the test-suite on non-host architecture

vadorovsky commented 4 months ago

I think we should remove entirely the Cross as wrapper around cross because it's redundant:

https://github.com/exein-io/pulsar/blob/018fd568c306916e7241cc89b94da53ece15a64a/xtask/src/main.rs#L25-L28

Then move and refactor the current xtask/cross as part of Test. As the command line option says, the xtask/test command is about the test-suite (Run eBPF test suite) and a good solution is to have that method taking cross arguments in case someone want to run the test-suite on non-host architecture

Done.

I also added the --force-architest flag for running architest even for the native architecture. I needed it mostly for Github Actions, because of course BPF LSM is not enabled on Ubuntu. :roll_eyes: