[PR #999] [MERGED] FIx error on ip6tables not available #13214

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/999
Author: @pascal-fischer
Created: 7/3/2023
Status: Merged
Merged: 7/14/2023
Merged by: @mlsmaycon

Base: mainHead: fix/error_on_ip6tables_not_available


📝 Commits (3)

📊 Changes

3 files changed (+124 additions, -86 deletions)

View changed files

📝 client/firewall/iptables/manager_linux.go (+11 -2)
📝 client/internal/routemanager/firewall_linux.go (+13 -0)
📝 client/internal/routemanager/iptables_linux.go (+100 -84)

📄 Description

Describe your changes

On machines where ip6tables was not available we would fail and throw an error without adding ip v4 routes. Whit this change the client can continue even though ip6tables is not available.

https://github.com/netbirdio/netbird/issues/917

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/999 **Author:** [@pascal-fischer](https://github.com/pascal-fischer) **Created:** 7/3/2023 **Status:** ✅ Merged **Merged:** 7/14/2023 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `fix/error_on_ip6tables_not_available` --- ### 📝 Commits (3) - [`76023a4`](https://github.com/netbirdio/netbird/commit/76023a497c31069ed35bb1dd0f496e7855b66363) adding check operation to confirm if ip*tables is available - [`925e138`](https://github.com/netbirdio/netbird/commit/925e138dde42fc5c79836758df2b2ba6dfc63046) linter - [`e6b8d96`](https://github.com/netbirdio/netbird/commit/e6b8d962a28ddcc74e71d33aa0203ec8a99f7b4b) linter ### 📊 Changes **3 files changed** (+124 additions, -86 deletions) <details> <summary>View changed files</summary> 📝 `client/firewall/iptables/manager_linux.go` (+11 -2) 📝 `client/internal/routemanager/firewall_linux.go` (+13 -0) 📝 `client/internal/routemanager/iptables_linux.go` (+100 -84) </details> ### 📄 Description ## Describe your changes On machines where ip6tables was not available we would fail and throw an error without adding ip v4 routes. Whit this change the client can continue even though ip6tables is not available. ## Issue ticket number and link https://github.com/netbirdio/netbird/issues/917 ### 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 02:07:47 -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#13214