Repository navigation
Conversation
- 「2026-xx-xx に踏んだ」などの日付や経緯の語りを外し、罠の注意は 1 行で残す - InfisicalSecret の sync-options の説明を 1 行にそろえ、push-bridge の説明の重複を消す - Traefik / wg-easy / Zulip / Portainer / k3s の頃の比較を消す - routing-peer の hostNetwork の経緯、runner のネットワークの説明を短くする YAML の値・キー・埋め込みのスクリプトの処理は変えていない(変わったのはコメント行だけ)。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JbmWyUB9jcBcYCnNyiGBSV
|
Claude finished @5ym's task in 34s —— View job レビュー結果総評: 大きな問題はありません。マージして良いと思います。
指摘(軽微・任意)
確認できていないこと
テストは該当しない変更です。 |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
| # Dual-stack: without this the Service is IPv4-only, yet the CNI still | ||
| # installs hostPort DNAT rules for v6 :53 -- so IPv6 queries reach the | ||
| # svclb pod and are dropped there (klipper-lb only gets SRC_RANGES for v4). | ||
| # 無いと IPv4 だけの Service になる。 |
There was a problem hiding this comment.
軽微(任意): 「無いと IPv4 だけの Service になる」だけだと、なぜ dual-stack が要るのかが読み取れません。元のコメントにあった「CNI が v6 の :53 にも hostPort の DNAT を入れるので、Service が v4 のみだと IPv6 クエリが落ちる」という理由を 1 行残すと、罠の注意として意味が残ります(svclb の言及は不要)。ただし現状の hostPort 構成でも同じ理由が成り立つかは要確認です。
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JbmWyUB9jcBcYCnNyiGBSV
|
Claude finished @5ym's task in 13s —— View job レビュー結果(再レビュー)問題ありません。マージして良いと思います。
確認できていないこと
テストは該当しない変更です。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
apps/ の README 以外のマニフェストから、もう要らないコメントを削りました。38 ファイル、+92 / −221(差し引き −129 行)。
何を削ったか
$(の禁止、hostPort と Recreate の理由、/dns-queryに認証をかけない、hostNetwork にしない、WgPort は default.json が優先、Service links の名前の衝突など)変えていないもの
#か//で始まる行)を除いて比べたら、どのファイルも一致したCONFIGの本体と、renovate-post-upgrade.sh が探す- name: CONFIG/value: |の行には触れていない確かめたこと
この環境には PyYAML も kubectl も無いので、YAML のパーサーには通していない。代わりに、コメントを除いた中身が一致することを全ファイルで確かめ、ブロックスカラーの中に足したコメントの字下げが周りと同じことを目で見た。
🤖 Generated with Claude Code
https://claude.ai/code/session_01JbmWyUB9jcBcYCnNyiGBSV