[PR #4592] [MERGED] [Management/Client] Trigger debug bundle runs from API/Dashboard #20742

Open
opened 2026-08-05 05:07:39 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/4592
Author: @aliamerj
Created: 10/6/2025
Status: Merged
Merged: 11/21/2025
Merged by: @pappz

Base: feature/remote-debug-releaseHead: feature/remote-debug-clean


📝 Commits (10+)

📊 Changes

56 files changed (+3449 additions, -1257 deletions)

View changed files

📝 .github/workflows/wasm-build-validation.yml (+2 -2)
📝 client/cmd/debug.go (+1 -28)
📝 client/cmd/status.go (+1 -1)
📝 client/cmd/testutil_test.go (+4 -2)
📝 client/cmd/up.go (+1 -1)
📝 client/embed/embed.go (+2 -1)
📝 client/internal/connect.go (+12 -10)
📝 client/internal/debug/debug.go (+26 -19)
client/internal/debug/upload.go (+101 -0)
📝 client/internal/debug/upload_test.go (+2 -2)
📝 client/internal/engine.go (+108 -21)
📝 client/internal/engine_test.go (+16 -22)
client/jobexec/executor.go (+66 -0)
📝 client/proto/daemon.pb.go (+3 -12)
📝 client/proto/daemon.proto (+0 -1)
📝 client/server/debug.go (+2 -97)
📝 client/server/server.go (+4 -95)
📝 client/server/server_test.go (+4 -2)
📝 client/status/status.go (+95 -5)
📝 client/status/status_test.go (+1 -1)

...and 36 more files

📄 Description

Describe your changes

Stack

  • feature/remote-debug-release - ⚠️ No PR associated with branch

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)

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

Documentation

Select exactly one:

  • I added/updated documentation for this change
  • Documentation is not needed for this change (explain why)

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

https://github.com/netbirdio/docs/pull/__

Summary by CodeRabbit

  • New Features

    • Remote job management for peers: create, list, and fetch jobs (bundle workloads) via API and streaming.
    • Client-side job executor and shared debug-bundle uploader for generating and uploading bundles.
  • Improvements

    • End-to-end remote-triggered debug bundle flow (generate → upload) with optional wait and configurable log path.
    • Status overview now built from full-status data with explicit daemon version.
  • Chores

    • API, proto and client/server streaming additions to support jobs and bundle workflows.

🔄 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/4592 **Author:** [@aliamerj](https://github.com/aliamerj) **Created:** 10/6/2025 **Status:** ✅ Merged **Merged:** 11/21/2025 **Merged by:** [@pappz](https://github.com/pappz) **Base:** `feature/remote-debug-release` ← **Head:** `feature/remote-debug-clean` --- ### 📝 Commits (10+) - [`cc595da`](https://github.com/netbirdio/netbird/commit/cc595da1ad47d66f5591fe67162ce957b1092000) [management/client] create job channel between management and client (#4367) - [`17a2af9`](https://github.com/netbirdio/netbird/commit/17a2af96eaf729f7cc0319d5769fc79fa3fd6f2f) implement remote debug api (#4418) - [`d01c3d5`](https://github.com/netbirdio/netbird/commit/d01c3d501153654474b36ddb147ba81445022402) [management/client] Integrate Job API with Job Stream and Client Engine (#4428) - [`bdae55a`](https://github.com/netbirdio/netbird/commit/bdae55ab797b12b7b0e561f817131f1220622470) update yml for job (#4532) - [`5a12c5d`](https://github.com/netbirdio/netbird/commit/5a12c5d3455624b33960085d87c84074ad4446d2) fix everything - [`60c5782`](https://github.com/netbirdio/netbird/commit/60c5782905fffe5856f4d027fabf4ee008ae1fa7) fix other lint issue - [`7a95bf5`](https://github.com/netbirdio/netbird/commit/7a95bf5652e5093891966161fa51ed1bba44aa52) fix bug with missing logs file - [`23f9dd0`](https://github.com/netbirdio/netbird/commit/23f9dd04b8613e7ad65f49bced50e4919e8eabfd) clean up - [`e570570`](https://github.com/netbirdio/netbird/commit/e570570fe502054e1c90957b821b4190d61f776a) add some info level log - [`0e9438d`](https://github.com/netbirdio/netbird/commit/0e9438d6586507519994db588eec6c468253905a) fix lint ### 📊 Changes **56 files changed** (+3449 additions, -1257 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/wasm-build-validation.yml` (+2 -2) 📝 `client/cmd/debug.go` (+1 -28) 📝 `client/cmd/status.go` (+1 -1) 📝 `client/cmd/testutil_test.go` (+4 -2) 📝 `client/cmd/up.go` (+1 -1) 📝 `client/embed/embed.go` (+2 -1) 📝 `client/internal/connect.go` (+12 -10) 📝 `client/internal/debug/debug.go` (+26 -19) ➕ `client/internal/debug/upload.go` (+101 -0) 📝 `client/internal/debug/upload_test.go` (+2 -2) 📝 `client/internal/engine.go` (+108 -21) 📝 `client/internal/engine_test.go` (+16 -22) ➕ `client/jobexec/executor.go` (+66 -0) 📝 `client/proto/daemon.pb.go` (+3 -12) 📝 `client/proto/daemon.proto` (+0 -1) 📝 `client/server/debug.go` (+2 -97) 📝 `client/server/server.go` (+4 -95) 📝 `client/server/server_test.go` (+4 -2) 📝 `client/status/status.go` (+95 -5) 📝 `client/status/status_test.go` (+1 -1) _...and 36 more files_ </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link #4354 ## Stack - `feature/remote-debug-release` - :warning: No PR associated with branch <!-- branch-stack --> - \#4592 :point\_left: ### 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) > By submitting this pull request, you confirm that you have read and agree to the terms of the [Contributor License Agreement](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md). ## Documentation Select exactly one: - [ ] I added/updated documentation for this change - [x] Documentation is **not needed** for this change (explain why) ### Docs PR URL (required if "docs added" is checked) Paste the PR link from <https://github.com/netbirdio/docs> here: <https://github.com/netbirdio/docs/pull/>\_\_ <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Remote job management for peers: create, list, and fetch jobs (bundle workloads) via API and streaming. - Client-side job executor and shared debug-bundle uploader for generating and uploading bundles. - **Improvements** - End-to-end remote-triggered debug bundle flow (generate → upload) with optional wait and configurable log path. - Status overview now built from full-status data with explicit daemon version. - **Chores** - API, proto and client/server streaming additions to support jobs and bundle workflows. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <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 05:07:39 -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#20742