[PR #470] [MERGED] Improve module load #12655

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/470
Author: @mlsmaycon
Created: 9/14/2022
Status: Merged
Merged: 9/14/2022
Merged by: @mlsmaycon

Base: mainHead: improve-module-load


📝 Commits (10+)

  • 001393b Add additional check for needed kernel modules
  • 2e36763 Check if wireguard and tun modules are loaded
  • 4f8e1f6 fix state check
  • 565d20f Add module function tests
  • 7c629ab Add test execution in container
  • 1a6ee9f run client package tests on docker
  • c2f5cc7 add package comment to new file
  • 4c03285 force entrypoint
  • 5570aab add --privileged flag
  • e52b39f clean only if tables where created

📊 Changes

8 files changed (+637 additions, -32 deletions)

View changed files

📝 .github/workflows/golang-test-linux.yml (+52 -0)
📝 client/internal/connect.go (+1 -1)
📝 client/internal/routemanager/nftables_linux.go (+4 -2)
📝 iface/iface_darwin.go (+2 -2)
📝 iface/iface_linux.go (+6 -25)
📝 iface/iface_windows.go (+2 -2)
iface/module_linux.go (+349 -0)
iface/module_linux_test.go (+221 -0)

📄 Description

Check if Wireguard and Tun modules are loaded

If modules are loaded return true, otherwise attempt to load them

Run client packages tests on docker containters


🔄 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/470 **Author:** [@mlsmaycon](https://github.com/mlsmaycon) **Created:** 9/14/2022 **Status:** ✅ Merged **Merged:** 9/14/2022 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `improve-module-load` --- ### 📝 Commits (10+) - [`001393b`](https://github.com/netbirdio/netbird/commit/001393b9a66666bc2428ec3e052abc7245a406c5) Add additional check for needed kernel modules - [`2e36763`](https://github.com/netbirdio/netbird/commit/2e367635b8d3593856b53246b1aeea93f707bf1f) Check if wireguard and tun modules are loaded - [`4f8e1f6`](https://github.com/netbirdio/netbird/commit/4f8e1f6776b7313b249b3a17b4da5aa61c3c84c6) fix state check - [`565d20f`](https://github.com/netbirdio/netbird/commit/565d20f3a6733cf7568c0c6ce2f52a89f4f6f348) Add module function tests - [`7c629ab`](https://github.com/netbirdio/netbird/commit/7c629ab6ecbc6ced9d4b9db3981f10d1dfb16265) Add test execution in container - [`1a6ee9f`](https://github.com/netbirdio/netbird/commit/1a6ee9f272515b4a2ad458bafb5cca3c7668e9a0) run client package tests on docker - [`c2f5cc7`](https://github.com/netbirdio/netbird/commit/c2f5cc73a470b78bb7354ecabc3f244c3c57f160) add package comment to new file - [`4c03285`](https://github.com/netbirdio/netbird/commit/4c03285245dd4300e93cefd0fedcf114ee0e112b) force entrypoint - [`5570aab`](https://github.com/netbirdio/netbird/commit/5570aaba13e16aecdc3d5bd89400664bc7801dc1) add --privileged flag - [`e52b39f`](https://github.com/netbirdio/netbird/commit/e52b39f12300a8a68d0d30df25b82af0cd1601b5) clean only if tables where created ### 📊 Changes **8 files changed** (+637 additions, -32 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/golang-test-linux.yml` (+52 -0) 📝 `client/internal/connect.go` (+1 -1) 📝 `client/internal/routemanager/nftables_linux.go` (+4 -2) 📝 `iface/iface_darwin.go` (+2 -2) 📝 `iface/iface_linux.go` (+6 -25) 📝 `iface/iface_windows.go` (+2 -2) ➕ `iface/module_linux.go` (+349 -0) ➕ `iface/module_linux_test.go` (+221 -0) </details> ### 📄 Description Check if Wireguard and Tun modules are loaded If modules are loaded return true, otherwise attempt to load them Run client packages tests on docker containters --- <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:22 -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#12655