search

aurae-runtime / aurae

Distributed systems runtime daemon written in Rust.
https://aurae.io
Apache License 2.0
1.85k stars 91 forks source link

auraed ebpf refactoring #411

Closed future-highway closed 1 year ago

future-highway commented 1 year ago

So far, I have been unable to get the tests to run (they are ignored even when adding the --include-ignored flag).

I'll add additional comments inline

future-highway commented 1 year ago

My thought was to refactor some more in this PR, but considering the above comments re 2 more tracepoints, I think it's better to see what happens with those first, so I'm marking this as ready for review.

@JeroenSoeters Can you run the tests as part of reviewing this?

JeroenSoeters commented 1 year ago

@future-highway tests are passing here on my machine

JeroenSoeters commented 1 year ago

Just a few minor naming nit-picks. IMO we should merge this ASAP, as I'm about to add a few more eBPF probes.