Skip to content

Commit

Permalink
Instrument gRPC status code for client spans (#1044)
Browse files Browse the repository at this point in the history
* Add gRPC Status object to offsets

* Add ClientConn_Invoke_Returns ebpf probe

* Add status code to span event

* make docker-generate

* Set grpc.status.code attribute

* Generate status code fixture

* Add changelog entry

* Generate error span in grpc test

* Switch to int32 and change offset logic

* Check argument 2 and use bpf_probe_read_user

* Add error struct to pointer chain

* Update verify.bats

* lint

* Fix span status

* Skip error checks if resp==nil

* Use u32

* Update changelog

* Add call to stop_tracking_span

* Make docker-offsets

* Update changelog for new offsets

* make fixture-grpc

* Update bats test
  • Loading branch information
damemi authored Sep 11, 2024
1 parent 311584c commit dbd8e85
Show file tree
Hide file tree
Showing 10 changed files with 835 additions and 47 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,9 @@ OpenTelemetry Go Automatic Instrumentation adheres to [Semantic Versioning](http
- Support Go `1.23.1`. ([#1051](https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/1051))
- Log version information in the CLI. ([#1077](https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/1077))
- Support `google.golang.org/grpc` `1.66.1`. ([#1078](https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/1078))
- Add gRPC status code attribute for client spans (`rpc.grpc.status_code`). ([#1044](https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/1044))
- Support `google.golang.org/grpc` `1.68.0-dev`. ([#1044](https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/1044))
- Support `go.opentelemetry.io/otel@v1.30.0`. ([#1044](https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/1044))

### Changed

Expand Down
Loading

0 comments on commit dbd8e85

Please sign in to comment.