tetragon icon indicating copy to clipboard operation
tetragon copied to clipboard

Move some packages from standalone Hubble

Open tklauser opened this issue 2 years ago • 2 comments

Move some packages from standalone Hubble (github.com/cilium/hubble, branch v0.5) to local packages to reduce dependencies.

This is in support of #73 to eventually allow dropping the dependency on github.com/cilium/hubble and updating to a recent version of github.com/cilium/cilium.

Reviewed best by commit.

tklauser avatar May 25 '22 07:05 tklauser

Thanks @tklauser!

Regarding the test failure: https://github.com/cilium/tetragon/runs/6588057292:

{ "process_kprobe": { "process": { "pid": 11522, "start_time": "2022-05-25T08:03:04.685Z" }, "parent": {}, "function_name": "__x64_sys_writev", "args": [ { "int_arg": 1 }, { "bytes_arg": "aGVsbG8gaW92ZWMgd29ybGQ=" } ], "action": "KPROBE_ACTION_POST" }, "time": "2022-05-25T08:04:05.098Z"

We seem to be missing the process info, which is why the test fails. I'll rerun the test to see if it will happen again.

kkourt avatar May 25 '22 08:05 kkourt

@kkourt I've seen this similar flake a few times in various tests. Seems like a legit bug but hard to reproduce. We had an issue about it here #39 for TestMultipleMountsFiltered.

willfindlay avatar May 25 '22 13:05 willfindlay