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

IaC - RDS Aurora #15

Closed
wants to merge 6 commits into from
Closed

Conversation

starkmatt
Copy link
Contributor

@starkmatt starkmatt commented Apr 14, 2020

RDS Aurora

Here we have a Code that creates an Aurora DB Cluster in AWS. I've checked from my end and it's worked without any issues. Please give it a try from your end and let me know what you think

terraform init
terraform apply

PS
You may run into some issues when running terraform destroy after.
If this is the case, you'll need to edit your tf.state file and set the following argument to true
skip_final_snapshot = true

hashicorp/terraform-provider-aws#4910

@starkmatt
Copy link
Contributor Author

Next step is seeing if we can connect our WP-Instance to this DB>

@starkmatt starkmatt mentioned this pull request Apr 14, 2020
Comment on lines +1 to +4
provider "aws" {
version = "~> 2.0"
region = "ap-southeast-2"
}
Copy link
Contributor

Choose a reason for hiding this comment

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

not required. there should be a single provider for all resources

Comment on lines +1 to +11
resource "aws_rds_cluster" "wp-db" {
cluster_identifier = "wp-db"
engine = "aurora-mysql"
engine_version = "5.7.mysql_aurora.2.03.2"
availability_zones = ["ap-southeast-2a", "ap-southeast-2b", "ap-southeast-2c"]
database_name = "wpdb"
master_username = "wpadmin"
master_password = "iammradmin"
backup_retention_period = 5
preferred_backup_window = "07:00-09:00"
}
Copy link
Contributor

Choose a reason for hiding this comment

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

File is not in the right folder.
Missing several things related to the requirements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants