Use beyla 1.3.3 with k8s discovery

GoogleCloudPlatform / opentelemetry-operator-sample

Toolbox of recipes, samples, and guides for using the OpenTelemetry Operator on Google Cloud

Apache License 2.0

32 stars 16 forks source link

Use beyla 1.3.3 with k8s discovery #90

Closed aabmass closed 8 months ago

aabmass commented 8 months ago

Similar to #86, but for the servicegraph recipe.

Upgrade beyla
Move to YAML based config, selecting only pods
Add rbac config for Beyla to watch pods
~Move k8s attributes detection into beyla, including updating RBAC~

Tested the graphgen script is still working and the CPU/Memory usage of beyla/otel-collector are the same or better:

aabmass commented 8 months ago

@dashpole I left in the k8sattributes processor since I will use it to lookup both server and client pod names in the next PR, which Beyla doesn't do right now.

dashpole commented 8 months ago

do we still need the rbac changes and the k8s detection config?

    attributes:
      kubernetes:
        enable: true

aabmass commented 8 months ago

@dashpole see https://github.com/grafana/beyla/issues/673

I could go back to BEYLA_SYSTEM_WIDE since we didn't see cardinality issues with the spanmetrics receiver as an alternative.