Skip to content

fix: remove up metric#148

Merged
netanelC merged 1 commit intomasterfrom
netanelC-patch-1
Oct 23, 2025
Merged

fix: remove up metric#148
netanelC merged 1 commit intomasterfrom
netanelC-patch-1

Conversation

@netanelC
Copy link
Contributor

@netanelC netanelC commented Oct 23, 2025

This PR removes the "up" metric that causes the duplication of "up" metric in Prometheus

Question Answer
Bug fix
New feature
Breaking change
Deprecations
Documentation
Tests added
Chore

@netanelC netanelC requested a review from CptSchnitz October 23, 2025 08:19
@netanelC netanelC changed the title fix: change includeUp option to false in promBundleConfig fix: remove up metric Oct 23, 2025
@github-actions
Copy link

Hi it is me, your friendly bot helper! 👋
I've packed this commit for you 😊
You can install it like this:

{
  "dependencies": {
      "@map-colonies/telemetry": "https://ghatmpstorage.blob.core.windows.net/npm-packages/telemetry-fef5fa28367b333ba6ef00910b6f8e52acc8a413.tgz"
  }
}

The link will expire in a week ⌛

@netanelC netanelC merged commit 9e4bc25 into master Oct 23, 2025
3 checks passed
@netanelC netanelC deleted the netanelC-patch-1 branch October 23, 2025 08:26
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.

2 participants