Skip to content

Set up OpenTelemetry: SDK, exporter and the deployed commit - #1001

Open
o11y-one-temp-app[bot] wants to merge 2 commits into
mainfrom
o11y/instrument/01a17e466db9797480bf4d0d61f20657
Open

o11y-one-temp-app[bot] wants to merge 2 commits into
mainfrom
o11y/instrument/01a17e466db9797480bf4d0d61f20657

Conversation

@o11y-one-temp-app

Copy link
Copy Markdown

o11y.one found these telemetry gaps in containerish/OpenRegistry at 0f24d5b. Fix run 01a17e46-6db9-7974-80bf-4d0d61f20657, agent run 01a17e46-6dc2-7801-9d03-a402b0a3a050.

Each module's pull request targets this branch: merge this one first, and o11y.one then points them at the default branch.

Gaps fixed

  • otel.sdk at dfs/mock/memMappedSystem.go:258
  • otel.exporter at dfs/mock/memMappedSystem.go:258
  • service.name at dfs/mock/memMappedSystem.go:258
  • service.version.commit at dfs/mock/memMappedSystem.go:258
  • otel.sdk at router/github.go:49
  • otel.exporter at router/github.go:49
  • service.name at router/github.go:49
  • service.version.commit at router/github.go:49

Dependencies

Pinned by o11y.one's recipes; no model chose a version:

  • go.opentelemetry.io/contrib/instrumentation/github.com/labstack/echo/otelecho v0.71.0
  • go.opentelemetry.io/otel v1.46.0
  • go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.46.0
  • go.opentelemetry.io/otel/sdk v1.46.0

Lockfiles

Dependency lockfiles (package-lock.json, pnpm-lock.yaml, go.sum, Python lock files) are denylisted, so this pull request leaves them as they were. Refresh them before CI can pass: npm install or pnpm install, go mod tidy, or your Python locker (uv lock, poetry lock, pip-compile).

Source sent to a model vendor

openai read cmd/registry/registry.go, config/config.go, dfs/mock/memMappedSystem.go, go.mod, main.go, router/github.go, router/router.go, telemetry/otel/otel.go.

o11y.one does not push to this branch again.

@o11y-one-temp-app

Copy link
Copy Markdown
Author

o11y.one telemetry

No new telemetry gaps in this pull request.

1 gap this pull request did not add
  • otel.sdk: No OpenTelemetry SDK is started for this service. Start one before the app is built: NodeSDK or registerOTel (TypeScript), a TracerProvider passed to trace.set_tracer_provider (Python), sdktrace.NewTracerProvider with otel.SetTracerProvider (Go).
    • router/github.go line 55

Not scanned, language not covered: go.mod.

Create stacked PR on the o11y.one telemetry check opens a pull request into this branch that adds the instrumentation; the author or a repository admin may click it.

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.

0 participants