Skip to content

chore: oauth2-proxy を v7.15.5 に固定し、Pulumi の CLI の版も Renovate で上げる - #639

Merged
5ym merged 3 commits into
mainfrom
pin-oauth2-proxy
Oct 30, 2026
Merged

5ym merged 3 commits into
mainfrom
pin-oauth2-proxy

Conversation

@5ym

@5ym 5ym commented Oct 30, 2026 •

Copy link
Copy Markdown
Collaborator

全体の見直し (#637・#638) で見つかった、デプロイに効く 2 件。

  • oauth2-proxy: apps/auth の 4 つの Deployment が quay.io/oauth2-proxy/oauth2-proxy:latest で、Renovate は拾えず (skip=invalid-value)、再起動のたびに黙って版が変わりえた。いま 4 つとも動いているイメージ (sha256:8498b0d0…) は quay で v7.15.5 (最新のリリース) と同じなので、v7.15.5 に固定する。中身は同じ (Pod は作り直される)

  • Pulumi の CLI: pulumi/.pulumi-version (pulumi.yml が使う CLI) が 3.267.0 のまま、SDK (@pulumi/pulumi) と tools の 3.273.0 から遅れていた。3.273.0 に揃え、regex の customManager で Renovate に拾わせる (depName は同じ pulumi/pulumi なので揃って上がる)。手元の dry-run で pulumi/.pulumi-version から 3.273.0 を拾うのを確かめた。pulumi/README の表も直した

  • ROADMAP: inline に入るコメント (infisical の Secret・apiserver-rbac・local-path) は直すと machine config が変わるので、次に当てるときに直す項目として足した

  • apps/forgejo/README: 17.0.1 でも forgejo-bot の認可済み連携は CLI に create-authorized-integration しか無く (一覧・削除なし。本番の forgejo admin user --help で確かめた)、見送りの理由はそのまま。版を書き足した

🤖 Generated with Claude Code

https://claude.ai/code/session_01T9PbUfqG4pPxus99UeLiJs

- apps/auth の 4 つの oauth2-proxy が :latest で、Renovate が拾えず (skip=invalid-value) 再起動で黙って版が変わりえた。
  いま動いているもの (sha256:8498b0d0…) と同じ v7.15.5 にする
- pulumi/.pulumi-version (CI の CLI) が 3.267.0 のまま SDK と tools の 3.273.0 から遅れていた。揃え、customManager で Renovate に拾わせる

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

claude Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

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


レビュー結果

結論: 問題なし。 マージして差し支えないと考えます。

確認したこと

  • oauth2-proxy: apps/auth の 4 ファイル (deployment.yaml・adguard-proxy.yaml・hubble-proxy.yaml・sub-proxy.yaml) をすべて v7.15.5 に固定しています。latest の残りはありません。
  • リリースノート (gh release view v7.15.5 -R oauth2-proxy/oauth2-proxy):
    • Go 1.26.8 への更新、依存の更新、CVE 4 件の修正が入っています。
    • 注意書きは Bitbucket provider の --bitbucket-team の非推奨だけです。このリポジトリの 4 つの Deployment は Bitbucket を使っていないので影響はありません (Bitbucket 関連の引数があるかは grep していないので、PR 本文の「いま動いているイメージと同じ」を信じた判断です)。
    • PR 本文の説明どおり、動いているイメージの digest が v7.15.5 と同じなら、Pod が作り直されるだけで挙動は変わりません。
  • Pulumi の CLI:
    • pulumi/.pulumi-version の 3.273.0 は package.json の @pulumi/pulumi と tools/Dockerfile の PULUMI_VERSION=v3.273.0 に揃っています。古い 3.267.0 の参照はリポジトリに残っていません。
    • 新しい regex の customManager は、3.273.0 だけの 1 行ファイルに合います。depName が pulumi/pulumi、extractVersionTemplate が v 付きタグを外す形で、Dockerfile や SDK と同じ依存として扱われます。
    • README の表も実態に合っています。

要確認 (任意・ブロッカーではありません)

  • tools/Dockerfile の ARG PULUMI_VERSION=v3.273.0 は既に Renovate が拾っているはずですが、このリポジトリでは確認できていません。そうであれば 3 か所の版が同じ PR で上がります。マージ後の最初の Renovate PR で、3 か所が揃って上がっているかを見てください。
  • pulumi グループ (groupName: pulumi) の matchFileNames に pulumi/.pulumi-version が入っていません。同じ PR にまとまるかは、グループ設定ではなく depName が同じであることに頼っています。別の PR に分かれても害はありませんが、「揃って上がる」と書いた description とずれる可能性があります。

テストに当たるものは無い変更です (設定とマニフェストのみ)。
・ ブランチ: pin-oauth2-proxy

@claude

claude Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

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


レビュー結果

  • 差分を読む
  • 観点ごとに確認する

所感: oauth2-proxy の v7.15.5 への固定 (4 ファイルとも同じ変更)、ROADMAP・forgejo README・pulumi/README の文言は問題ありません。固定するタグは PR 本文のとおり現行のイメージと同じ中身とのことで、latest からの変更による挙動の差は出ません。リリースノートの確認は版を上げる PR ではなく固定だけなので省きました。

要確認 (1 件、インラインに書きました): renovate.json の新しい customManager について、「SDK・tools と同じ pulumi/pulumi なので同じ PR で揃って上がる」とありますが、SDK は npm の @pulumi/pulumi で depName が違い、pulumi グループの packageRule も .pulumi-version を含みません。別 PR になる可能性が高く、その間は CLI と SDK の版がずれます。実際の Renovate の動きを dry-run で確かめるか、グループに入れるか、文言を直すかをご検討ください。手元の dry-run は .pulumi-version の検出だけを確かめたものと読みました。

細かい点: forgejo README の「2026-10-31 に確かめた」はコミット日時 (JST) と合っています。問題ありません。

テストは該当するものがありません (設定とマニフェストのみ)。
· ブランチ: pin-oauth2-proxy

Comment thread renovate.json
"^(?<currentValue>\\d+\\.\\d+\\.\\d+)\\s*$"
],
"datasourceTemplate": "github-releases",
"depNameTemplate": "pulumi/pulumi",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

要確認: 「同じ pulumi/pulumi なので、同じ PR で揃って上がる」は成り立たないかもしれません。

  • SDK は npm の @pulumi/pulumi (depName が別、pulumi グループの packageRule は package.json・bun.lock・Pulumi.yaml だけが対象)。
  • この customManager は github-releases の pulumi/pulumi で、.pulumi-version は上のグループに入らない。

なので CLI と SDK は別の枝・別の PR になりえて、その間は .pulumi-version と package.json の版がずれます (README の「SDK と揃える」が崩れる)。揃えたいなら matchFileNames に pulumi/.pulumi-version を足すか、matchDepNames で @pulumi/pulumi と pulumi/pulumi を同じ groupName に入れてください。ただしグループに入れると postUpgradeTasks (bun.lock 更新) も走るので、その副作用も見てください。ずれてよいなら、description と README の文言を直すだけで足ります。

CLI 3.273 の pulumi install で sdks/netbird の行が変わり、preview の差分チェックで落ちた。
.pulumi-version を pulumi のまとめ (postUpgradeTasks で pulumi install) に入れ、SDK と同じ PR で上がるようにする。

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

claude Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

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


レビュー結果

結論: 問題なし。 前回の指摘 (.pulumi-version が pulumi グループに入らず、SDK と別 PR になりうる) は、最新のコミットで直っています。

確認したこと

  • グループ化: pulumi/.pulumi-version を pulumi グループの matchFileNames に足し、customManager の description も「SDK と同じ PR」に直っています。CLI と SDK は同じ PR で上がるはずです。
  • postUpgradeTasks: グループに入ったので pulumi/renovate-post-upgrade.sh が走ります。このスクリプトは .pulumi-version の CLI を取ってチェックサムと照らし、pulumi install --no-plugins を流します。CLI を上げたときの bun.lock の変更もここで入ります。fileFilters は bun.lock と package.json です。
  • bun.lock の差分: @pulumi/netbird の行が ^3.238.0 → ^3.269.0 に変わり、hasInstallScript が付いています。CLI を 3.273.0 に上げて pulumi install を流した結果として説明がつきます。README と description の「bun.lock も変わる」とも合っています。
  • customManager: ^\d+\.\d+\.\d+\s*$ は 1 行ファイルの 3.273.0 に合います。extractVersionTemplate で v を外す形も、他の github-releases の規則と同じです。
  • oauth2-proxy: 4 ファイルとも v7.15.5 に固定されています。latest の残りはありません。PR 本文によると、動いているイメージの digest と同じ中身なので、Pod が作り直されるだけで挙動は変わりません。
  • ROADMAP・forgejo README・pulumi/README: 文言は実態に合っています。

要確認 (任意・ブロッカーではありません)

  • skipArtifactsUpdate: true の規則に .pulumi-version を足したので、.pulumi-version の書き換えは Renovate 本体が行い、bun.lock は postUpgradeTasks が更新します。マージ後の最初の pulumi グループの PR で、3 つ (.pulumi-version・package.json・bun.lock) が揃って変わっているかを見てください。
  • tools/Dockerfile の PULUMI_VERSION はこのグループに入っていません。別の PR で上がる可能性があります。ずれても害はありません。

テストに当たるものは無い変更です (設定とマニフェストのみ)。
· ブランチ: pin-oauth2-proxy

@github-actions

Copy link
Copy Markdown

🍹 preview on doany-cloudflare/prod

Pulumi report
Previewing update (prod):

@ previewing update.....
pulumi:pulumi:Stack doany-cloudflare-prod  
Resources:
61 unchanged

@5ym
5ym merged commit b718dc9 into main Oct 30, 2026
8 checks passed
@5ym
5ym deleted the pin-oauth2-proxy branch October 30, 2026 16:10
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