[PR #2197] [CLOSED] Debounce network changes #18898

Open
opened 2026-08-05 04:09:03 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/2197
Author: @hurricanehrndz
Created: 6/25/2024
Status: Closed

Base: mainHead: chernand/debounce_network_change


📝 Commits (1)

📊 Changes

1 file changed (+42 additions, -21 deletions)

View changed files

📝 client/internal/engine.go (+42 -21)

📄 Description

Describe your changes

Block restarting netbird engine for 3s when network change detected. This will help prevent Netbird from attempting to create multiple wiregurad interfaces when restarting the engine in rapid succession

#2196
#2130

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/2197 **Author:** [@hurricanehrndz](https://github.com/hurricanehrndz) **Created:** 6/25/2024 **Status:** ❌ Closed **Base:** `main` ← **Head:** `chernand/debounce_network_change` --- ### 📝 Commits (1) - [`84959c8`](https://github.com/netbirdio/netbird/commit/84959c86a5c3f626a509880417d965b47affabac) Debounce network changes ### 📊 Changes **1 file changed** (+42 additions, -21 deletions) <details> <summary>View changed files</summary> 📝 `client/internal/engine.go` (+42 -21) </details> ### 📄 Description ## Describe your changes Block restarting netbird engine for 3s when network change detected. This will help prevent Netbird from attempting to create multiple wiregurad interfaces when restarting the engine in rapid succession ## Issue ticket number and link #2196 #2130 ### Checklist - [x] 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 --- <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 04:09:03 -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#18898