[PR #2599] [MERGED] [management] Add command flag to set metrics port for signal and relay service, and update management port #15386

Closed
opened 2026-08-05 03:07:32 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/2599
Author: @benniekiss
Created: 9/13/2024
Status: Merged
Merged: 9/14/2024
Merged by: @mlsmaycon

Base: mainHead: config_metrics


📝 Commits (2)

  • 7760f67 add flags to customize metrics port
  • b7a99a1 change management default metrics port to match other services

📊 Changes

3 files changed (+6 additions, -10 deletions)

View changed files

📝 management/cmd/root.go (+1 -1)
📝 relay/cmd/root.go (+3 -5)
📝 signal/cmd/run.go (+2 -4)

📄 Description

Describe your changes

Currently, the listen port for the metrics server is hardcoded for the signal and relay service.

Updated the default port for management from 8081 to 9090.

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

🔄 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/2599 **Author:** [@benniekiss](https://github.com/benniekiss) **Created:** 9/13/2024 **Status:** ✅ Merged **Merged:** 9/14/2024 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `config_metrics` --- ### 📝 Commits (2) - [`7760f67`](https://github.com/netbirdio/netbird/commit/7760f67670c3443bc2616ffa3c38d28a43b9ed96) add flags to customize metrics port - [`b7a99a1`](https://github.com/netbirdio/netbird/commit/b7a99a1023289c07be0711a6ea7c49e943d22c30) change management default metrics port to match other services ### 📊 Changes **3 files changed** (+6 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `management/cmd/root.go` (+1 -1) 📝 `relay/cmd/root.go` (+3 -5) 📝 `signal/cmd/run.go` (+2 -4) </details> ### 📄 Description ## Describe your changes Currently, the listen port for the metrics server is hardcoded for the signal and relay service. Updated the default port for management from 8081 to 9090. ## Issue ticket number and link ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [x] Is a feature enhancement - [x] It is a refactor - [ ] Created tests that fail without the change (if possible) - [ ] Extended the README / documentation, if necessary --- <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 03:07:32 -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#15386