Skip to content

fix: ArgoCD が見る Forgejo の組織名を doa にする - #178

Merged
5ym merged 1 commit into
mainfrom
forgejo-org-rename
Sep 16, 2026
Merged

5ym merged 1 commit into
mainfrom
forgejo-org-rename

Conversation

@5ym

@5ym 5ym commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

組織を danythingdoa に改名したあと、ArgoCD が Forgejo の 3 アプリを同期できなくなっていた(worklog / todoroku / tamasagashi が SYNC STATUS = Unknown)。

ComparisonError: failed to list refs: unexpected requesting
"https://fj.doany.io/danything/worklog-cloud/info/refs?service=git-upload-pack" status code: 307

Forgejo は旧パスに 307 を返すが、ArgoCD の git クライアントはリダイレクトを追わない。

  • bootstrap/argocd/repos.yaml の gitea scmProvider の ownerdoa
  • 改名時に直す場所(ここと Infisical の url)をコメントに残した
  • apps/forgejo/README.mdargocd-creds.yaml のコメントも追従

マージ後にやること:

  1. Infisical /argocd/forgejo-repo-credsurlhttps://fj.doany.io/doa に(repo-creds は URL の接頭辞で照合するので、ずれると資格情報が付かない) ← 私からは値を書き換えられないのでお願いします
  2. bootstrap/ は ArgoCD の管理外(手で適用)なので kubectl apply -f bootstrap/argocd/repos.yaml

🤖 Generated with Claude Code

https://claude.ai/code/session_01JgGYf5qbjDXYhtzcpLsGvv

組織を danything → doa に改名したあと、ArgoCD が旧名の URL を叩いて 307 を
受け、worklog / todoroku / tamasagashi の 3 つが Unknown で同期停止していた
(ArgoCD の git クライアントはリダイレクトを追わない)。scmProvider の owner を
doa にして、改名時に直す場所をコメントに書いた。

bootstrap/ は手で適用するので、マージ後に kubectl apply が要る。
Infisical /argocd/forgejo-repo-creds の url も doa に直すこと(repo-creds は
URL の接頭辞で照合するため)。

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JgGYf5qbjDXYhtzcpLsGvv
@claude

claude Bot commented Sep 16, 2026

Copy link
Copy Markdown

Claude finished @5ym's task in 1m 23s —— View job


レビュー結果

  • PR の内容を把握する
  • 変更ファイルを確認する
  • レビューコメントを投稿する

bootstrap/argocd/repos.yamlgitea scmProvider の ownerapps/forgejo/argocd-creds.yamlapps/forgejo/README.md の一部を danythingdoa に更新する内容で、307 リダイレクトで ApplicationSet が同期できなくなる原因(gitea scmProvider の owner が旧組織名のまま)に対する修正として妥当です。ロジック・セキュリティ面の問題はありません。

1点、ドキュメントの一貫性について気になった点があり、apps/forgejo/README.md の 46 行目にインラインコメントを付けました:

  • この PR で 48 行目・58 行目の danythingdoa に直っていますが、同じファイル内の 25 行目(組織 danything を作る手順)、46 行目(「先に 1 回だけ」の組織作成手順)、86 行目(見出し「組織 danything は…」)、112 行目(curl のサンプル URL)が danything のまま残っています。
  • 組織は既に doa に改名済みなので、このまま残すと将来この手順書を読んで作業する人が誤った組織名で作業してしまう可能性があります(46 行目の手順をそのまま実行すると danything という組織を新規作成してしまう、112 行目の curl はそのままでは失敗する、など)。
  • 意図的に旧名のまま残しているのでなければ、他の箇所と合わせて doa に更新することをおすすめします(要確認)。

なお docs/decisions.md 内の danything/tamasagashi 等の記述は決定ログ(過去の記録)なので、これは書き換え不要と考えます。

Comment thread apps/forgejo/README.md
@@ -45,7 +45,7 @@

1. Forgejo に組織 `danything` を作る

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

この PR で argocd-creds.yaml への url や README 内の他の参照(48 行目、58 行目)は doa に直していますが、ここ(46 行目)・25 行目・86 行目の見出し・112 行目の curl 例は danything のまま残っています。

組織は既に doa に改名済みなので、このまま残すと「先に 1 回だけ」の手順(46 行目)で誤って danything という組織を新規作成してしまったり、112 行目の curl をそのまま実行すると存在しない danything/<name> に当たって失敗したりします。要確認(意図的に古い手順として残しているのでなければ、他の箇所と合わせて doa に更新した方が良さそうです)。

@5ym
5ym merged commit 6ce3ef8 into main Sep 16, 2026
1 check passed
@5ym
5ym deleted the forgejo-org-rename branch September 16, 2026 07:09
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