Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

T3202: Enable wireguard debug messages (backport #3679) #3706

Merged
merged 2 commits into from
Jun 25, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Jun 22, 2024

Change Summary

Added ability to enable or disable dynamic debugging for Wireguard on demand

set system option kernel debug wireguard

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

Related PR(s)

Component(s) name

system option

Proposed changes

How to test

set system option kernel debug wireguard
commit

# configure wireguard interface
set interfaces wireguard wg01 address '10.1.0.1/30'
set interfaces wireguard wg01 description 'VPN-to-wg02'
set interfaces wireguard wg01 peer to-wg02 address '192.0.2.1'
set interfaces wireguard wg01 peer to-wg02 allowed-ips '192.168.2.0/24'
set interfaces wireguard wg01 peer to-wg02 port '51820'
set interfaces wireguard wg01 peer to-wg02 public-key 'XMrlPykaxhdAAiSjhtPlvi30NVkvLQliQuKP7AI7CyI='
set interfaces wireguard wg01 per-client-thread
set interfaces wireguard wg01 port '51820'
set interfaces wireguard wg01 private-key 'cNcQKm4KTmtOFyUp+THWKWs50rAh/3il9YAUvSHzInE='
set protocols static route 192.168.2.0/24 interface wg01
commit

now dynamic debugging messages can be viewed by running journalctl

Smoketest result

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • I have run the components SMOKETESTS if applicable
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

This is an automatic backport of pull request #3679 done by [Mergify](https://mergify.com).

@mergify mergify bot added the conflicts label Jun 22, 2024
@mergify mergify bot requested a review from a team as a code owner June 22, 2024 17:03
@mergify mergify bot mentioned this pull request Jun 22, 2024
12 tasks
@github-actions github-actions bot added the sagitta VyOS 1.4 LTS label Jun 22, 2024
Copy link

github-actions bot commented Jun 22, 2024

👍
No issues in PR Title / Commit Title

@c-po
Copy link
Member

c-po commented Jun 22, 2024

Needs to be rebased after #3709 is merged

Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@c-po c-po force-pushed the mergify/bp/sagitta/pr-3679 branch from 4204894 to 5b0faee Compare June 24, 2024 12:10
@c-po c-po self-assigned this Jun 24, 2024
@vyos vyos deleted a comment from mergify bot Jun 24, 2024
@c-po c-po enabled auto-merge June 24, 2024 12:11
@c-po c-po merged commit 621dd08 into sagitta Jun 25, 2024
8 checks passed
@mergify mergify bot deleted the mergify/bp/sagitta/pr-3679 branch June 25, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sagitta VyOS 1.4 LTS
Development

Successfully merging this pull request may close these issues.

4 participants