Skip to content

Instrument services and 2 more modules - #1003

Open
o11y-one-temp-app[bot] wants to merge 1 commit into
o11y/instrument/01a17e466db9797480bf4d0d61f20657from
o11y/instrument/01a17e466dbe72c594a42e8b58915202
Open

o11y-one-temp-app[bot] wants to merge 1 commit into
o11y/instrument/01a17e466db9797480bf4d0d61f20657from
o11y/instrument/01a17e466dbe72c594a42e8b58915202

Conversation

@o11y-one-temp-app

Copy link
Copy Markdown

o11y.one found these telemetry gaps in containerish/OpenRegistry at 4a8d3d2. Fix run 01a17e46-6dbe-72c5-94a4-2e8b58915202, agent run 01a17e4a-1d8c-7f62-9eff-10897d939fb8.

Part of instrumenting this repository: merge #1001 first. This touches only services, store, vcs's files, so it merges in any order with the other modules' pull requests.

Gaps fixed

  • http.client at services/kon/github_actions/v1/server/build_logs.go:174
  • http.client at services/kon/github_actions/v1/server/build_logs.go:180
  • db.client at store/v1/store.go:28
  • db.client at store/v1/store.go:31
  • db.client at store/v1/store.go:42
  • db.client at store/v1/store.go:43
  • http.client at vcs/github/github.go:174
  • http.client at vcs/github/github.go:180

Dependencies

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

  • github.com/XSAM/otelsql v0.44.0
  • go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.71.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 go.mod, services/kon/github_actions/v1/server/build_logs.go, store/v1/store.go, vcs/github/github.go.

o11y.one does not push to this branch again.

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