Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

デプロイ時に Custom::CDKBucketDeployment がタイムアウトするバグの修正 #694

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

tbrand
Copy link
Contributor

@tbrand tbrand commented Oct 25, 2024

変更内容の説明

  • Custom::CDKBucketDeployment がタイムアウトするのは RAG の docs 用の bucket に大量のアクセスログが存在しているから
  • 大量のアクセスログが存在してしまっている件は サーバーアクセスのログ記録の設定元と設定先のS3バケットが同一になっている #689 で修正済み (アクセスログは別の bucket に吐くように修正された)
  • ただし、既存ユーザーは RAG の docs 用 bucket に大量のアクセスログがまだ存在するため、デプロイ時にそれらのファイルを削除しようとしてタイムアウトすることがある
  • v1.0.0 #672 の対応で exclude: ['AccessLogs/'] オプションを追加したが、この対応は不完全で、ログの出力先を AccessLogs に変更した対応の前から存在する logs ファイルが排除されていなかった。
    (これが今回の対応) exclude の対象に logs* を追加することで、全てのログファイルを対象外に

チェック項目

  • npm run lint を実行した
  • 関連するドキュメントを修正した
  • 手元の環境で動作確認済み

関連する Issue

関連する Issue を可能な限り挙げてください。

Copy link
Contributor

@wadabee wadabee left a comment

Choose a reason for hiding this comment

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

LGTM!

@wadabee wadabee merged commit c20c70e into main Oct 25, 2024
2 checks passed
@wadabee wadabee deleted the fix-deploy-timeout branch October 25, 2024 04:43
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.

3 participants