Improve Pyroscope Profiling Integration

[SeamusGrafana]

Add Support for the Go Profiles GitHub Integration in Grafana Cloud.

Add Support for the Span Profiles to Traces Feature

Some Minor updates to README.md to fix some misc Lint Issues, and improve Docs.

[federicotdn]

@SeamusGrafana would you mind adapting this to main? I had to fix some middleware-related things, see comment in #143.

[SeamusGrafana]

Thanks @federicotdn

Have done that and tested locally, I can see the pyroscope.profile.id in some Spans.

But not all, mostly the newer ones;

Have I missed it somewhere?

Thanks

[federicotdn]

I've changed the service name attribute for those endpoints, could you merge main again?
Also, I assume you are building a new QP image locally and running that one. The one in Dockerhub will always be slightly behind.

[SeamusGrafana]

Also, I assume you are building a new QP image locally and running that one. The one in Dockerhub will always be slightly behind.

Yeah, using make to build it and running the Binary directly.

Will test shortly, thanks!

[SeamusGrafana]

Thanks @federicotdn

I tried again but got the same results as before.

[federicotdn]

Aah ok. Could you share what set of commands you are using locally to test this? I can try reproducing it locally.

[SeamusGrafana]

Thanks @federicotdn

From the root of the branch;

docker buildx build .

Copy the SHA256 for the gernated image, and it add it into here;

version: '3'

services:
  quickpizza:
    image: <IMAGE-SHA256>
    container_name: quickpizza
    environment:
      OTEL_RESOURCE_ATTRIBUTES: service.namespace=quickpizza
      QUICKPIZZA_CONF_FARO_URL: ${QUICKPIZZA_FARO_RECEIVER:-http://alloy:12347/collect}
      QUICKPIZZA_LOG_LEVEL: ${QUICKPIZZA_LOG_LEVEL:-debug}
      QUICKPIZZA_OTLP_ENDPOINT: ${QUICKPIZZA_OTLP_ENDPOINT:-http://alloy:4318}
      QUICKPIZZA_PYROSCOPE_ENDPOINT: ${QUICKPIZZA_PYROSCOPE_ENDPOINT:-http://pyroscope:4040}
      QUICKPIZZA_PYROSCOPE_NAME: ${QUICKPIZZA_PYROSCOPE_NAME:-quickpizza}
      QUICKPIZZA_TRUST_CLIENT_TRACEID: 1
      VIRTUAL_HOST: ${QUICKPIZZA_VIRTUAL_HOST:-quickpizza.lab}
    ports:
      - ${QUICKPIZZA_PORT:-3333}:3333
      - ${QUICKPIZZA_PORT_GRPC_1:-3334}:3334
      - ${QUICKPIZZA_PORT_GRPC_2:-3335}:3335
    restart: on-failure
    labels:
      namespace: quickpizza

Run docker compose up

Navigate to QP site in browser, genertae few pizzas, then test new delay endpoint.

I have an Alloy, Mimir, Loki, Tempo and Pyroscope already configured where QP sends data too.

[SeamusGrafana]

Hey @federicotdn, I managed to find and fix the issue with the pyroscope.profile.id not showing for the new endpoints and pushed the changed for this

I also bumped the Pyrocope Version too, and Merged main into this.

You now see a pyroscope.profile.id for most spans now.

Are you able to have a look at this and review it when you can please (not urgent).

Thanks!