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

terraform planの差分確認&対応 #47

Merged
merged 2 commits into from
Nov 7, 2021
Merged

terraform planの差分確認&対応 #47

merged 2 commits into from
Nov 7, 2021

Conversation

keitakn
Copy link
Member

@keitakn keitakn commented Nov 6, 2021

issueURL

#45

Doneの定義

変更点概要

terraform plan で差分がある箇所をチェックして必要な箇所で terraform apply -refresh-only を実行。

以下のディレクトリ配下で実行を行った。

  • /data/providers/aws/environments/prod/12-images
  • /data/providers/aws/environments/stg/12-images
  • /data/providers/aws/environments/prod/22-migration
  • /data/providers/aws/environments/prod/23-rds

terraform apply -refresh-only を実行しても良いケースをREADMEに追加。

@keitakn keitakn self-assigned this Nov 6, 2021
@keitakn keitakn requested a review from kobayashi-m42 November 6, 2021 08:16
@keitakn keitakn marked this pull request as ready for review November 6, 2021 08:16
Copy link
Member

@kobayashi-m42 kobayashi-m42 left a comment

Choose a reason for hiding this comment

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

READMEの更新ありがとう!
LGTMeow

@keitakn keitakn merged commit 3b4a313 into main Nov 7, 2021
@keitakn keitakn deleted the feature/issue45 branch November 7, 2021 04:59
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