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

RDSに対してMigrationを実行する為のリソースを作成する #40

Merged
merged 6 commits into from
Aug 8, 2021

Conversation

kobayashi-m42
Copy link
Member

@kobayashi-m42 kobayashi-m42 commented Aug 4, 2021

issueURL

#32

Doneの定義

#32 の完了の定義を満たしていること

変更点概要

RDS に対して Migration を実行するために下記のリソースを構築した。

  • ECR、ECS Cluster
    ECS タスクの定義は Migrationを実行できる環境を構築 lgtm-cat-migration#2 で作成する。
    ECS タスク実行に必要となる下記のリソースについても作成。

    • ECSに設定するSecurityGroup (RDS のインバウンドルールを変更し、ECS -> RDS の接続ができるようにした)
    • タスク実行用 IAMロール
    • ECS タスクの定義に設定するためのSSMパラメータストア
  • 22-migration を追加したのでディレクトリ名を変更
    RDSの構築より前に migration のリソースを構築するためディレクトリ名を下記の通り変更。
    22-rds -> 23-rds

  • RDS と RDS Proxy の独自ドメイン名を設定
    Route53 の CNAME レコードを追加し、下記のドメインでアクセスできることを確認。
    RDS: stg.lgtm-cat-rds
    RDS Proxy : stg.lgtm-cat-rds-proxy

補足情報

ECS タスクの実行方法については、nekochans/lgtm-cat-migration#2 に記載する。

@kobayashi-m42 kobayashi-m42 self-assigned this Aug 7, 2021
@kobayashi-m42 kobayashi-m42 requested a review from keitakn August 7, 2021 16:27
@kobayashi-m42 kobayashi-m42 marked this pull request as ready for review August 7, 2021 16:27
Copy link
Member

@keitakn keitakn left a comment

Choose a reason for hiding this comment

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

RDSのローカルドメインも設定してくれてありがとう🐱
問題なさそう👌

LGTMeow

@kobayashi-m42 kobayashi-m42 merged commit ce45bbc into main Aug 8, 2021
@kobayashi-m42 kobayashi-m42 deleted the feature/issue32/ecs-migration branch August 8, 2021 15:46
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