Results 8 issues of marcin

https://github.com/cloudposse/terraform-root-modules/blob/ec95a8960f28298c9dcde346c0b231ef3a06835d/aws/elasticsearch/variables.tf#L73-L78

Hey We use workspaces dynamically. Removing them should be normal behaviour and all resources created by such workspace should be scheduled for deletion with confirmation or auto destroyed. Currently before...

## Describe the Bug ES was created without `iam_role_arns`. After adding it and applying it failed with: ``` module.elasticsearch.aws_iam_role.elasticsearch_user[0]: Creating... module.elasticsearch.aws_iam_role.elasticsearch_user[0]: Creation complete after 1s [id=xxx-user] module.elasticsearch.data.aws_iam_policy_document.default[0]: Refreshing state... module.elasticsearch.aws_elasticsearch_domain_policy.default[0]:...

bug

Hey For lock file in network attached `afpfs` lock was created for every python process I ran and didn't prevent parallel execution. I expected to receive a warning or error...

help-wanted
feature

Hey, when I add: ``` recipient { type = "team" name = "${each.value}-${var.opsgenie_escalation_name}" } ``` I get error: ``` Error: Unsupported argument on escalation.tf line 63, in resource "opsgenie_escalation" "this":...

## Describe the Feature Current code for adding routes is based on `count` https://github.com/cloudposse/terraform-aws-vpc-peering/blob/47c6e51bf449de36cc7f487317bdab2d0b5aadaa/main.tf#L51-L67 Changing tags can possible result in changing count order and removal of actually using peering routing....

### Summary by CodeRabbit - Refactor: Updated the conditional compilation directives across multiple files. This change enhances the modularity of the code by allowing specific runtime modules to be included...

dependencies :deciduous_tree:
rust

all of them get myID = 1 ``` puppet class { '::zookeeper': servers => ['x', 'y', 'z''], client_ip => $::ipaddress_eth0, election_port => 2889, leader_port => 3889, require => Class['java'], }...