Kornilios Kourtis
Kornilios Kourtis
We discussed this topic in yesterday's community call (May 11th). The general plan is as follows: 1. start with a PR from @olsajiri that refactors the way arguments are accessed...
@olsajiri @tdaudi I feel like we are discussing this in multiple places now (https://github.com/cilium/tetragon/pull/3725#discussion_r2087311836, https://github.com/cilium/tetragon/issues/3734#issuecomment-2878952880), so I think it makes sense to take a step back and see what the...
I believe this is fixed so closing. Relevant PRs: * https://github.com/cilium/tetragon/pull/3725 * https://github.com/cilium/tetragon/pull/3730
> I forgot to run `make crds`; not sure if that target was around last time? It was renamed from `make generate` to `make crds` recently: https://github.com/cilium/tetragon/commit/c26e483f7a3901eb3c5bc86671ba46f49c4b2481 > > Either...
Hi, > Thanks for the review @kkourt! I've responded to your comments and fixed the formatting issues, but I'm having trouble getting the test to work. > Thanks. I think...
> Hi @kkourt, thanks for the pointers! Your suggestion worked. > > @olsajiri I've split the PR into logical patches. Some CI is failing now but for unrelated reasons. Thanks,...
> > > Hi @kkourt, thanks for the pointers! Your suggestion worked. > > > @olsajiri I've split the PR into logical patches. Some CI is failing now but for...
Moving this to draft since there are many conflicts.
> Any progress on this? I believe what this PR is trying to do can be achieved via https://github.com/cilium/tetragon/pull/3143, which is a more generic approach.
I don't believe the tetragon operator runs as root: https://github.com/cilium/tetragon/blob/0d9b619cc8ebc79de196566ed1ebd92186597929/install/kubernetes/tetragon/values.yaml#L259-L266.