Skip to content

chore(deploy): cloudflare-proxied "true" の注釈を外す - #209

Merged
5ym merged 1 commit into
mainfrom
chore/drop-cloudflare-true
Oct 22, 2026
Merged

5ym merged 1 commit into
mainfrom
chore/drop-cloudflare-true

Conversation

@5ym

@5ym 5ym commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

gitops が Cloudflare を通すのを基本にした(*.doany.io を proxied に。danything/gitops#501 / #502)ので、Cloudflare を通すための個別の注釈は要らない。

ExternalDNS はもう "true" を拾わず(annotationFilter は false だけ)、この名前はワイルドカードで Cloudflare 経由のまま。外しても経路は変わらない。

🤖 Generated with Claude Code

https://claude.ai/code/session_01NN4TmSXHrMP8dkTbtkjfz1

gitops が Cloudflare を通すのを基本にした(*.doany.io を proxied に、danything/gitops#501 / #502)ので、
Cloudflare を通すための個別の注釈は要らない。ExternalDNS はもう "true" を拾わず、外しても経路は変わらない
(ワイルドカードで Cloudflare 経由のまま)。

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

claude Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

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


レビュー結果

問題なしです。

  • 変更は deploy/httproute.yaml から external-dns.kubernetes.io/cloudflare-proxied: "true" の注釈とそのコメントを削除するだけです。YAML の構造も壊れていません(annotations: ごと削除されており、metadata は name と namespace だけが残ります)。
  • ホスト名 x.doany.io は *.doany.io のワイルドカードに含まれます。PR 説明どおり、gitops 側でワイルドカードが proxied になっていれば、経路は変わりません。
  • deploy/service.yaml にある Cloudflare への言及はキャッシュに関するコメントだけで、この注釈とは無関係です。
  • シークレットの混入や、認証・認可への影響はありません。テストの対象になる変更でもありません。

要確認(マージ前): リポジトリ内では確認できないので、danything/gitops#501 / #502 が本番に反映済みで、*.doany.io が実際に proxied になっていることを先に確認してください。未反映のままマージすると、ExternalDNS が既存の個別レコードを削除し、x.doany.io が一時的に Cloudflare を経由しなくなる可能性があります。

@5ym
5ym merged commit f7287a4 into main Oct 22, 2026
2 checks passed
@5ym
5ym deleted the chore/drop-cloudflare-true branch October 22, 2026 06:08
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