Skip to content

Add hs-opentelemetry - #1519

Open
isovector wants to merge 2 commits into
haskell:masterfrom
tweag:hs-opentelemetry
Open

Add hs-opentelemetry#1519
isovector wants to merge 2 commits into
haskell:masterfrom
tweag:hs-opentelemetry

Conversation

@isovector

Copy link
Copy Markdown
Contributor

This PR adds an --enable-opentelemetry flag which, when enabled, turns on telemetry around hackage-server's router.

The purpose here is to start collecting traces and get real data on which routes get used in production, and how expensive each is.

@mergify

mergify Bot commented Aug 24, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@phadej

phadej commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

while telemetry could be useful, I think that hs-opentelemetry-sdk is a liability (iand675/hs-opentelemetry#315 - no bounds), especially for a project with effectively zero active maintainers.

@ysangkok

ysangkok commented Aug 25, 2026

Copy link
Copy Markdown
Member

I think logging/monitoring would be great to have, even if the project is under-maintained. But since I do not work on deployments, I will defer to @gbaz on whether this is the right direction.

@isovector

Copy link
Copy Markdown
Contributor Author

@phadej: very reasonable point. Would hiding it behind a default-off flag assuage some fears here? The reason I ask is that I'm in the middle of chicken-and-egg situation here; I'm trying to get enough data in order to determine if hackage-server-v3 can meaningfully move the needle. Which if it does, is a good argument to continue moving forwards on it.

Maybe we could gather this information from logs on the existing load balancer / reverse proxy? Perhaps @gbaz would know?

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.

3 participants