Skip to content

Commit d05ed10

Browse files
committed
chore: validate S3 import and iam_roles new association resource
1 parent 63b7482 commit d05ed10

File tree

3 files changed

+8
-4
lines changed

3 files changed

+8
-4
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -307,7 +307,7 @@ No modules.
307307
| <a name="input_iam_role_path"></a> [iam\_role\_path](#input\_iam\_role\_path) | Path for the monitoring role | `string` | `null` | no |
308308
| <a name="input_iam_role_permissions_boundary"></a> [iam\_role\_permissions\_boundary](#input\_iam\_role\_permissions\_boundary) | The ARN of the policy that is used to set the permissions boundary for the monitoring role | `string` | `null` | no |
309309
| <a name="input_iam_role_use_name_prefix"></a> [iam\_role\_use\_name\_prefix](#input\_iam\_role\_use\_name\_prefix) | Determines whether to use `iam_role_name` as is or create a unique name beginning with the `iam_role_name` as the prefix | `bool` | `false` | no |
310-
| <a name="input_iam_roles"></a> [iam\_roles](#input\_iam\_roles) | Map of IAM roles and supported feature names to associate with the cluster | `map(string)` | `{}` | no |
310+
| <a name="input_iam_roles"></a> [iam\_roles](#input\_iam\_roles) | Map of IAM roles and supported feature names to associate with the cluster | `map(map(string))` | `{}` | no |
311311
| <a name="input_instance_class"></a> [instance\_class](#input\_instance\_class) | Instance type to use at master instance. Note: if `autoscaling_enabled` is `true`, this will be the same instance class used on instances created by autoscaling | `string` | `""` | no |
312312
| <a name="input_instance_timeouts"></a> [instance\_timeouts](#input\_instance\_timeouts) | Create, update, and delete timeout configurations for the cluster instance(s) | `map(string)` | `{}` | no |
313313
| <a name="input_instances"></a> [instances](#input\_instances) | Map of cluster instances and any specific/overriding attributes to be created | `any` | `{}` | no |

examples/s3_import/main.tf

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,12 @@ module "aurora" {
125125
create_security_group = true
126126
allowed_cidr_blocks = module.vpc.private_subnets_cidr_blocks
127127

128-
iam_database_authentication_enabled = true
128+
iam_roles = {
129+
s3_import = {
130+
role_arn = aws_iam_role.s3_import.arn
131+
feature_name = "s3Import"
132+
}
133+
}
129134

130135
# S3 import https://docs.aws.amazon.com/AmazonRDS/latest/AuroraUserGuide/AuroraMySQL.Integrating.LoadFromS3.html
131136
s3_import = {
@@ -134,7 +139,6 @@ module "aurora" {
134139
ingestion_role = aws_iam_role.s3_import.arn
135140
}
136141

137-
apply_immediately = true
138142
skip_final_snapshot = true
139143

140144
db_parameter_group_name = aws_db_parameter_group.example.id

variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -347,7 +347,7 @@ variable "endpoints" {
347347
# aws_rds_cluster_role_association
348348
variable "iam_roles" {
349349
description = "Map of IAM roles and supported feature names to associate with the cluster"
350-
type = map(string)
350+
type = map(map(string))
351351
default = {}
352352
}
353353

0 commit comments

Comments
 (0)