Skip to content

boot CI の ingress の chain の既定が drop かの検査も外す - #654

Merged
5ym merged 1 commit into
mainfrom
boot-no-firewall
Oct 31, 2026
Merged

5ym merged 1 commit into
mainfrom
boot-no-firewall

Conversation

@5ym

@5ym 5ym commented Oct 31, 2026

Copy link
Copy Markdown
Collaborator

#652 で受信の防火壁を外したので、nftables の ingress の chain が無くなった。#653 で「外から 10250 が通らない」検査は外したが、「ディスクとネットワーク」の段の「ingress の chain の既定が drop か」が残っていて boot が落ちていた(#652・#653 とも boot=fail のままマージされている)。戻すときに戻す旨をコメントに残す。

🤖 Generated with Claude Code

https://claude.ai/code/session_01T9PbUfqG4pPxus99UeLiJs

#652 で受信の防火壁を外したので、nftables の ingress の chain は無くなった。
#653 で外から 10250 の検査は外したが、こちらが残っていて boot が落ちていた。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T9PbUfqG4pPxus99UeLiJs
@claude

claude Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

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


レビュー結果

問題無しです。

  • 変更は .github/workflows/talos-boot.yml の ingress chain の既定が drop かの検査(/tmp/nft.yaml の生成と yq での判定)を削除するだけです。
  • /tmp/nft.yaml と $P は、削除した範囲より後では参照されていません(grep で確認)。壊れる参照はありません。
  • 456 行目の失敗時のログ出力にある talosctl get nftableschains は単なる表示で、検査ではありません。残して問題ありません。
  • 284 行目の既存コメントと同じ書き方で、戻すときの手掛かり(#646)もコメントに残してあります。bootstrap/talos.md の 68 行目にも防火壁を外した旨の記載があり、整合しています。
  • テスト: CI の検査自体を外す変更なので、追加すべきテストはありません。この PR の boot ジョブが通るかで確認できます。

@5ym
5ym merged commit 1a3cb28 into main Oct 31, 2026
5 checks passed
@5ym
5ym deleted the boot-no-firewall branch October 31, 2026 14:40
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