別アカウントの Bedrock を利用するためのクロスアカウント設定を追加

[tomachi1201]

目的

Bedrock サービスのみ、別の AWS アカウントを指定して構成する

利用条件

• Bedrock 用 AWS アカウント で IAM Role を手動で作成しておく
• 作成した IAM Role を cdk.json に手動で書く

修正内容

• packages/cdk/cdk.json

  • roleArn と sessionName を追加した。
  • roleArn に "arn:aws:iam::アカウントID:role/ロール名" を指定することにより、別の AWS アカウント上の Bedrock サービスを利用できるようになる。
  • sessionName にはデフォルト値を設定済みである。

• packages/cdk/lib/construct/api.ts

  • 上記を Lambda 環境変数に設定する
  • roleArn が指定されている場合に AssumeRole 用のポリシーを設定

• packages/cdk/lambda/utils/bedrockApi.ts

  • roleArn が設定されている場合に AssumeRole する

動作確認

• cdk デプロイできることを確認した
• ユースケース一覧が動作することを確認した
• roleArn 設定時に Bedrock アカウントのみ、トークン数などが増えていることを CloudWatch Metrics から確認した
• roleArn 非設定時に アプリケーションデプロイ用アカウントのみ、トークン数などが増えていることを CloudWatch Metrics から確認した

[tbrand]

いくつかコメントさせていただきました！

別のAWSアカウントでRoleを作成して cdk.json に設定するまでの手順を https://github.com/aws-samples/generative-ai-use-cases-jp/blob/main/docs/DEPLOY_OPTION.md に追加をお願いします！:pray: おそらく利用頻度としては低いオプションになると思うので、追加位置は最下部で問題ありません！また、手動で作成する手順を説明するにあたって、スクリーンショットなどは必要ありません！追加後に、DEPLOY_OPTION.md の他の項目と同様 README.md にアンカーリンクを追加してください！

また、念のためですが、ルートディレクトリにて npm run lint を実行していただけますでしょうか？:pray: (すでに実行されていたらすいません！)

[tomachi1201]

別のAWSアカウントでRoleを作成して cdk.json に設定するまでの手順を https://github.com/aws-samples/generative-ai-use-cases-jp/blob/main/docs/DEPLOY_OPTION.md に追加をお願いします！:pray: おそらく利用頻度としては低いオプションになると思うので、追加位置は最下部で問題ありません！また、手動で作成する手順を説明するにあたって、スクリーンショットなどは必要ありません！追加後に、DEPLOY_OPTION.md の他の項目と同様 README.md にアンカーリンクを追加してください！
→対応しましたのでご確認頂ければと思います。

また、念のためですが、ルートディレクトリにて npm run lint を実行していただけますでしょうか？:pray: (すでに実行されていたらすいません！)
→すみません、こちら実行していなかったので実行致しました。

コメント頂いた内容を修正したものをアップしておりますのでご確認をお願い致します。

[tbrand]

不要なコミットが入っています！

[tomachi1201]

すみません、修正します！

[tbrand]

今までの設定項目と統一して、利用しない場合のデフォルト値を null にしていただいても良いでしょうか？

[tomachi1201]

null に変更後に再デプロイしたところ、以下のエラーが出力されたのですが、Lambda の環境変数には null は設定できないと理解しました。（認識齟齬ございましたらご指摘ください）

nerativeAiUseCasesStack failed: Error [ValidationError]: [/Resources/APIPredict09E4E4FF/Type/Environment/Variables/CROSS_ACCOUNT_BEDROCK_ROLE_ARN] 'null' values are not allowed in templates

そのため、空文字列 "" をデフォルト値とできればと思いますが如何でしょうか。
もしくは、null 値の場合、環境変数自体を設定しないように修正した方が良いでしょうか。

[tbrand]

すみません改めて確認したら全て null というわけではなく空文字もちょくちょく使っていたので、空文字で大丈夫です！:pray:

[tbrand]

LGTM！ありがとうございます！