search

grafana / beyla

eBPF-based autoinstrumentation of web applications and network metrics
https://grafana.com/oss/beyla-ebpf/
Apache License 2.0
1.45k stars 102 forks source link

Update helm charts to reflect latest map pinning changes #1261

Closed rafaelroquetto closed 18 hours ago

rafaelroquetto commented 1 month ago

https://github.com/grafana/beyla/pull/1251 removed the requirement for map pinning using the bpf filesystem. We need to update our helm charts to reflect that change, in particular, removing the parts that set up the bpf filesystem mounts.

marevers commented 1 month ago

@rafaelroquetto I can take a look at doing this - I assume from /charts/beyla/templates/daemon-set.yaml we can remove the entire init container, the bpffs volume mount/volume and the BEYLA_BPF_FS_PATH / BEYLA_BPF_FS_BASE_DIR environment variables?

grcevski commented 1 month ago

That would be awesome @marevers. You are correct, the BPF mount is not needed at all and it was the only reason for the init container.

marevers commented 1 month ago

Thanks. I've linked the PR with the removal.