[PR #517] [MERGED] Add Open-telemetry support #12698

Closed
opened 2026-08-05 02:06:29 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/517
Author: @braginini
Created: 10/20/2022
Status: Merged
Merged: 10/21/2022
Merged by: @braginini

Base: mainHead: feature/open-telemetry


📝 Commits (9)

  • 11d0e81 First steps towards open-telemetry support
  • 6a34298 Fix metrics output
  • a37f6b9 Fix request/response counters prefixes
  • cb85a11 Refactor counter key
  • 45bd17c Extract metrics to higher level
  • bf19ec6 Rollback change made by mistake
  • 9fea558 Fix codacy
  • 357640c Fix review notes
  • cdb7eca Separate http listener for metrics

📊 Changes

7 files changed (+302 additions, -64 deletions)

View changed files

📝 go.mod (+15 -7)
📝 go.sum (+31 -12)
📝 management/cmd/management.go (+42 -1)
📝 management/cmd/root.go (+1 -0)
📝 management/server/http/handler.go (+73 -43)
📝 management/server/http/middleware/jwtmiddleware.go (+1 -1)
management/server/http/middleware/metrics.go (+139 -0)

📄 Description

This PR brings open-telemetry metrics to the
Management service.
The Management service exposes new HTTP endpoint
/metrics on 8081 port by default.
The port can be changed by specifying
--metrics-port PORT flag when starting the service.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netbirdio/netbird/pull/517 **Author:** [@braginini](https://github.com/braginini) **Created:** 10/20/2022 **Status:** ✅ Merged **Merged:** 10/21/2022 **Merged by:** [@braginini](https://github.com/braginini) **Base:** `main` ← **Head:** `feature/open-telemetry` --- ### 📝 Commits (9) - [`11d0e81`](https://github.com/netbirdio/netbird/commit/11d0e81b9c06ec78ba25044001ab40b84bfd1078) First steps towards open-telemetry support - [`6a34298`](https://github.com/netbirdio/netbird/commit/6a3429828a5b6d0816540aeab9f108385553bb59) Fix metrics output - [`a37f6b9`](https://github.com/netbirdio/netbird/commit/a37f6b919fe918d55ccfd71f0e8bbac2d7144262) Fix request/response counters prefixes - [`cb85a11`](https://github.com/netbirdio/netbird/commit/cb85a1108d7233745a1956730ae87e90a2b37163) Refactor counter key - [`45bd17c`](https://github.com/netbirdio/netbird/commit/45bd17c425e58a89b123578c01ec2430464cb1c8) Extract metrics to higher level - [`bf19ec6`](https://github.com/netbirdio/netbird/commit/bf19ec6ec1c95f87bd0b495f8b8d4594dfb782bb) Rollback change made by mistake - [`9fea558`](https://github.com/netbirdio/netbird/commit/9fea5587e4c1d275ae7146ac052ff747549961a7) Fix codacy - [`357640c`](https://github.com/netbirdio/netbird/commit/357640c49c0038d6f93df05d0c0a8fdda86da877) Fix review notes - [`cdb7eca`](https://github.com/netbirdio/netbird/commit/cdb7ecac7ad79a941c1841b7fa6c5927a4be2508) Separate http listener for metrics ### 📊 Changes **7 files changed** (+302 additions, -64 deletions) <details> <summary>View changed files</summary> 📝 `go.mod` (+15 -7) 📝 `go.sum` (+31 -12) 📝 `management/cmd/management.go` (+42 -1) 📝 `management/cmd/root.go` (+1 -0) 📝 `management/server/http/handler.go` (+73 -43) 📝 `management/server/http/middleware/jwtmiddleware.go` (+1 -1) ➕ `management/server/http/middleware/metrics.go` (+139 -0) </details> ### 📄 Description This PR brings open-telemetry metrics to the Management service. The Management service exposes new HTTP endpoint /metrics on 8081 port by default. The port can be changed by specifying --metrics-port PORT flag when starting the service. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2026-08-05 02:06:29 -04:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DYNR/netbird#12698