Skip to content

routing peer: WireGuard のポートは PVC の default.json が優先することを書く - #442

Merged
5ym merged 1 commit into
mainfrom
netbird-wgport-note
Oct 18, 2026
Merged

5ym merged 1 commit into
mainfrom
netbird-wgport-note

Conversation

@5ym

@5ym 5ym commented Oct 18, 2026

Copy link
Copy Markdown
Collaborator

#438 の続き。NetBird は WireGuard のポートを PVC の default.json に保存していて、NB_WIREGUARD_PORT は最初の起動にしか効かなかった(env だけ 51820 にして、実際は 51822 のまま hostPort とずれていた)。本番は default.json を直して作り直し済みで、スマホと IPv4 の P2P(51820)で繋がることを確かめた。そのことをコメントに書く。

🤖 Generated with Claude Code

https://claude.ai/code/session_01JbmWyUB9jcBcYCnNyiGBSV

@claude

claude Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Claude finished @5ym's task in 9s —— View job


レビュー結果

問題なしです。

apps/netbird/routing-peer.yaml にコメントを 2 行足すだけの変更で、動作は変わりません。

  • NB_WIREGUARD_PORT の値(51820)は、同じファイルの containerPort と hostPort(どちらも 51820)と合っています。
  • 追加したコメントは、直前の既存コメント(2026-10-18 に 51822 から 51820 へ変えた経緯)と内容が続いていて、書き方も揃っています。
  • default.json の "WgPort" が優先されるという記述は、PR 本文の検証結果(PVC の default.json を直して作り直し、IPv4 の P2P で接続できた)と矛盾しません。ただし、NetBird 本体の実装は確認していません。

@5ym
5ym merged commit 88a0ee6 into main Oct 18, 2026
1 check passed
@5ym
5ym deleted the netbird-wgport-note branch October 18, 2026 05:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant