Skip to content

Initial commit of the OpenTelemetry trace exposition#226

Merged
rakyll merged 16 commits into
mainfrom
otel
Jul 1, 2026
Merged

Initial commit of the OpenTelemetry trace exposition#226
rakyll merged 16 commits into
mainfrom
otel

Conversation

@rakyll

@rakyll rakyll commented Jul 1, 2026

Copy link
Copy Markdown
Member

Refactor telemetry configuration and resolve deprecated GCP metadata calls:

  • Replace deprecated metadata.ProjectID() and metadata.OnGCE() with context-aware versions.
  • Update telemetry.SetTraceProvider to accept variadic otlptracegrpc.Options and remove the hardcoded endpoint parameter.
  • Add telemetry.WithGCPCredentials() helper to encapsulate GCP-specific credentials.
  • Isolate GCP-specific server telemetry configuration into a new serve_gcp.go file.

Fix premature gRPC stream cancellation in the harness execution clients:

  • Update the client receive loop to drain the stream until io.EOF after receiving the HarnessEnd frame, preventing CANCELLED errors in telemetry.
  • Encapsulate the stream draining and message dispatching logic into a shared drainStream helper in a new stream.go file.
  • Clean up unused io imports in antigravity.go and substrate.go.

Follow-up:

  • End-to-end trace context propagation to HarnessService deployments
  • End-to-end trace context propagation to Substrate calls

@rakyll rakyll marked this pull request as draft July 1, 2026 04:05
@rakyll rakyll requested review from joycel-github and wjjclaud July 1, 2026 04:20
@rakyll rakyll marked this pull request as ready for review July 1, 2026 04:22
@rakyll rakyll merged commit 9c87844 into main Jul 1, 2026
8 checks passed
@rakyll rakyll deleted the otel branch July 1, 2026 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants