element-hq / element-hq/dendrite

Prometheus stuff needs refactoring

Open
#2,056 0 comments 0 reactions 0 assignees View on GitHub
T-Task
Dominant language
Go
Stars
965
Forks
101
PR merge metrics
No merged PRs in 30d

Description

*This issue was originally created by [**@neilalexander**](https://github.com/neilalexander) at .*

We use `init()` a lot to register prometheus collectors which is bad for WASM, and dupe registrations caused problems for the server signing key tests too.

Contributor guide

Open the contributing guide

Research direction

Start by locating the init() functions that register Prometheus collectors and inspect how those registrations are used by the server signing key tests. Determine a registration approach that works under WASM and prevents duplicate collectors. Done means the refactoring is complete without duplicate-registration failures and the affected tests still pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, prometheus, wasm
Domain
backend, observability
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.