Skip to content

Conversation

@maekawataiki
Copy link
Collaborator

@maekawataiki maekawataiki commented Dec 20, 2025

Description of Changes

CDK および CloudFormation ではクロススタック参照は公式では強い参照のみを実装している。
スタック間の不整合を伴う可能性のある変更を避け、そのような変更はユーザー側での多段階デプロイによる安全なデプロイもしくは自己責任での弱い参照導入を想定している。

弱い参照の方がいいスタック(デプロイパラメータパラメータの変更にて有効化無効化などで依存関係を一時的に解消することなく内容の変更が発生する依存関係)
ApplicaitonInferenceProfile - モデル変更の際に参照変更を伴う
AgentCoreStack - すでに導入済み。GenericAgent と AgentBuilder を同じスタックに入っているため片方のみを無効化した際にデプロイエラーになる
AgentStack - Search Agent と Code Interpreter Agent が同じスタックに入っているため片方のみを無効化した際にデプロイエラーになる

強い参照で問題ないスタック(デプロイパラメータの変更など通常のオペレーションでは強い参照が課題にならないため)
Knowledgebase - KB の作り直しが発生したりする変更は止めた方が良いので強い参照維持
ClosedNetwork - VPC を変更したり等はあまり発生しない想定のため強い参照維持
Dashboard - メインスタックの UserPool は変更されない想定のため強い参照維持
VideoBucket - 基本的にスタック内での変更ではなくメインスタックが参照するのが別リージョンのスタックになるため強い参照で問題ない
Guardrail - 今の所パラメータで設定変更されず有効化無効化のみのため。今後ルールをパラメータで変更する可能性がある場合弱い参照に切り替え
CloudFrontWAF - ルールが変更されてもWAF 自体の変更はないため。過去 #942 ExportName を変更するような誤った破壊的変更があったようなことがない限りは問題ない

Checklist

  • Modified relevant documentation
  • Verified operation in local environment
  • Executed npm run cdk:test and if there are snapshot differences, execute npm run cdk:test:update-snapshot to update snapshots

Related Issues

#942
#1285
#1385

Copy link
Collaborator

@daisuke-awaji daisuke-awaji left a comment

Choose a reason for hiding this comment

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

LGTM 😄

@maekawataiki maekawataiki merged commit d23355a into main Dec 23, 2025
6 checks passed
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.

2 participants