Skip to content

Commit

Permalink
Fix TF cycle error (#10)
Browse files Browse the repository at this point in the history
* Remove `provider "aws"`. It causes cycles in `terraform plan` and `terraform apply`

* Update `README.md`
  • Loading branch information
aknysh authored Oct 5, 2017
1 parent 03a667b commit ec83b44
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 14 deletions.
12 changes: 6 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -60,12 +60,12 @@ output "efs_backup_security_group" {

### `datapipeline_config` variables

| Name | Default | Description | Required |
|:-----------------------------------|:--------------:|:------------------------------------------------------------|:--------:|
| instance_type | `t2.micro` | Instance type to use | Yes |
| email | `` | Email to use in `SNS` | Yes |
| period | `24 hours` | Frequency of pipeline execution (frequency of backups) | Yes |
| timeout | `60 Minutes` | Pipeline execution timeout | Yes |
| Name | Default | Description | Required |
|:-----------------------------------|:--------------:|:-----------------------------------------------------------------------------|:--------:|
| instance_type | `t2.micro` | Instance type to use | Yes |
| email | `` | Email to use in `SNS`. Needs to be provided, otherwise the module will fail | Yes |
| period | `24 hours` | Frequency of pipeline execution (frequency of backups) | Yes |
| timeout | `60 Minutes` | Pipeline execution timeout | Yes |



Expand Down
8 changes: 0 additions & 8 deletions main.tf
Original file line number Diff line number Diff line change
@@ -1,11 +1,3 @@
terraform {
required_version = ">= 0.9.1"
}

provider "aws" {
region = "${signum(length(var.region)) == 1 ? var.region : data.aws_region.default.name}"
}

data "aws_region" "default" {
current = true
}
Expand Down

0 comments on commit ec83b44

Please sign in to comment.