Skip to content
This repository has been archived by the owner on Aug 24, 2023. It is now read-only.

fix: use collectDefaultMetrics option #7

Merged
merged 1 commit into from
Nov 22, 2022

Conversation

wemeetagain
Copy link
Member

For apps that make heavy use of prometheus metrics, it may be possible that the default metrics are already registered and managed and configured separately. This is the case in lodestar.

Feature to disable default metrics

src/index.ts Outdated Show resolved Hide resolved
@wemeetagain wemeetagain changed the title feat: allow default metrics to be disabled fix: use collectDefaultMetrics option Nov 22, 2022
@mpetrunic
Copy link
Member

cc @achingbrain to double check if our logic is actually right and we fixed a bug 😅

@achingbrain
Copy link
Member

Oops..

@achingbrain achingbrain merged commit 3e4f00c into libp2p:main Nov 22, 2022
github-actions bot pushed a commit that referenced this pull request Nov 22, 2022
## [1.1.2](v1.1.1...v1.1.2) (2022-11-22)

### Bug Fixes

* use collectDefaultMetrics option ([#7](#7)) ([3e4f00c](3e4f00c))
@github-actions
Copy link

🎉 This PR is included in version 1.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants