#terraform-aws-modules (2024-01)
Terraform Modules
Discussions related to https://github.com/terraform-aws-modules
Archive: https://archive.sweetops.com/terraform-aws-modules/
2024-01-04
![Tommi Jensen avatar](https://secure.gravatar.com/avatar/c49dc4c662d564d0ca04ad7721ea5892.jpg?s=72&d=https%3A%2F%2Fa.slack-edge.com%2Fdf10d%2Fimg%2Favatars%2Fava_0023-72.png)
hey there.
I created a PR in terraform-aws-vpc-peering-multi-account (https://github.com/cloudposse/terraform-aws-vpc-peering-multi-account/pull/82) - december 4’th, with a proposed fix for https://github.com/cloudposse/terraform-aws-vpc-peering-multi-account/issues/81 - teardown/enabled = false - not working because of an enabled check on the provider config, specifically for assumerole usage.
not sure if the codeowners are swamped, or no longer there, but it seems other PR’s are also stranded. I couldn’t find guidelines regarding where/what/who to poke to try and alleviate the issue. currently we have to use the proposed fix PR as module, or we cannot do cross account vpc peering teardown. I don’t mind if it’s not deemed an acceptable fix, but some feedback would be good
pointers? ( was directed here from #terraform )
![Andriy Knysh (Cloud Posse) avatar](https://avatars.slack-edge.com/2018-06-13/382332470551_54ed1a5d986e2068fd9c_72.jpg)
@Tommi Jensen thanks for the PR, it’s approved, but please run the commands in the comment to update README and GH actions
![Tommi Jensen avatar](https://secure.gravatar.com/avatar/c49dc4c662d564d0ca04ad7721ea5892.jpg?s=72&d=https%3A%2F%2Fa.slack-edge.com%2Fdf10d%2Fimg%2Favatars%2Fava_0023-72.png)
done! and thanks.
![Tommi Jensen avatar](https://secure.gravatar.com/avatar/c49dc4c662d564d0ca04ad7721ea5892.jpg?s=72&d=https%3A%2F%2Fa.slack-edge.com%2Fdf10d%2Fimg%2Favatars%2Fava_0023-72.png)
@Andriy Knysh (Cloud Posse) just a friendly bump to notify you that the requested changes has been made.
![Tommi Jensen avatar](https://secure.gravatar.com/avatar/c49dc4c662d564d0ca04ad7721ea5892.jpg?s=72&d=https%3A%2F%2Fa.slack-edge.com%2Fdf10d%2Fimg%2Favatars%2Fava_0023-72.png)
@Andriy Knysh (Cloud Posse) bump.
![Andriy Knysh (Cloud Posse) avatar](https://avatars.slack-edge.com/2018-06-13/382332470551_54ed1a5d986e2068fd9c_72.jpg)
i’ll review it again in a few. Need to understand the comment @RB made about the PR reverting a previous PR
![Andriy Knysh (Cloud Posse) avatar](https://avatars.slack-edge.com/2018-06-13/382332470551_54ed1a5d986e2068fd9c_72.jpg)
@Tommi Jensen please address one comment and we’ll approve the PR, thank you
![Tommi Jensen avatar](https://secure.gravatar.com/avatar/c49dc4c662d564d0ca04ad7721ea5892.jpg?s=72&d=https%3A%2F%2Fa.slack-edge.com%2Fdf10d%2Fimg%2Favatars%2Fava_0023-72.png)
will do!
![Tommi Jensen avatar](https://secure.gravatar.com/avatar/c49dc4c662d564d0ca04ad7721ea5892.jpg?s=72&d=https%3A%2F%2Fa.slack-edge.com%2Fdf10d%2Fimg%2Favatars%2Fava_0023-72.png)
done. fwiw, I did that earlier, it just drifted meanwhile
2024-01-08
![Jason Kusar avatar](https://avatars.slack-edge.com/2024-01-08/6442627802932_461955f3f3d353e1cabb_72.jpg)
Hi all, I submitted a PR against terraform-aws-utils. Any chance someone could take a look at it? Thanks! https://github.com/cloudposse/terraform-aws-utils/pull/31
what
Add new Tel Aviv (il-central-1) region
why
Provide full coverage
references
![Andriy Knysh (Cloud Posse) avatar](https://avatars.slack-edge.com/2018-06-13/382332470551_54ed1a5d986e2068fd9c_72.jpg)
the PR looks good, can you please address the comments
what
Add new Tel Aviv (il-central-1) region
why
Provide full coverage
references
![Jason Kusar avatar](https://avatars.slack-edge.com/2024-01-08/6442627802932_461955f3f3d353e1cabb_72.jpg)
Done
![Andriy Knysh (Cloud Posse) avatar](https://avatars.slack-edge.com/2018-06-13/382332470551_54ed1a5d986e2068fd9c_72.jpg)
thanks, please also format all TF code
![Jason Kusar avatar](https://avatars.slack-edge.com/2024-01-08/6442627802932_461955f3f3d353e1cabb_72.jpg)
Was just doing that now
![Jason Kusar avatar](https://avatars.slack-edge.com/2024-01-08/6442627802932_461955f3f3d353e1cabb_72.jpg)
I usually have that as a pre-commit hook. Never think to run it manually. It’s done now
![Andriy Knysh (Cloud Posse) avatar](https://avatars.slack-edge.com/2018-06-13/382332470551_54ed1a5d986e2068fd9c_72.jpg)
@Jason Kusar one more thing please, see comments
![Jason Kusar avatar](https://avatars.slack-edge.com/2024-01-08/6442627802932_461955f3f3d353e1cabb_72.jpg)
Do I need to re-run make or anything after making that change?
![Andriy Knysh (Cloud Posse) avatar](https://avatars.slack-edge.com/2018-06-13/382332470551_54ed1a5d986e2068fd9c_72.jpg)
no, thanks
![Andriy Knysh (Cloud Posse) avatar](https://avatars.slack-edge.com/2018-06-13/382332470551_54ed1a5d986e2068fd9c_72.jpg)
@Jason Kusar i have to ask you to add all the config for ca-west-1
region as well (it was added after the module was last tested), same as you did for the Israel region Please see comments. Thanks a lot
![Jason Kusar avatar](https://avatars.slack-edge.com/2024-01-08/6442627802932_461955f3f3d353e1cabb_72.jpg)
Done
![Andriy Knysh (Cloud Posse) avatar](https://avatars.slack-edge.com/2018-06-13/382332470551_54ed1a5d986e2068fd9c_72.jpg)
![Andriy Knysh (Cloud Posse) avatar](https://avatars.slack-edge.com/2018-06-13/382332470551_54ed1a5d986e2068fd9c_72.jpg)
thanks a lot @Jason Kusar for all the updates
2024-01-09
![Vesa avatar](https://secure.gravatar.com/avatar/402bb3d86e02e731309018f1e0a54fc4.jpg?s=72&d=https%3A%2F%2Fa.slack-edge.com%2Fdf10d%2Fimg%2Favatars%2Fava_0017-72.png)
I encountered an issue with the AWS Terraform state backend module, and fortunately, someone has already created a fix for it. It just requires reviewers; could anyone please take a look? https://github.com/cloudposse/terraform-aws-tfstate-backend/pull/151
![Gabriela Campana (Cloud Posse) avatar](https://avatars.slack-edge.com/2023-05-17/5281506983315_fbbf3b358313efef4647_72.jpg)
2024-01-12
2024-01-23
![Karim avatar](https://avatars.slack-edge.com/2022-03-14/3231168253749_7ce30cfa074f578eaaff_72.jpg)
Hello all we’re using cloudposse terraform module terraform-aws-rds module which is using terraform-null-label v.0.25.0 & terraform-aws-route53-cluster-hostname v0.13.0 modules under-hood. Suddenly, terraform fails downloading the modules saying the branch not exist as below:
│ Error: Failed to download module
│ Could not download module "this" (context.tf:23) source code from
│ "git::<https://github.com/cloudposse/terraform-null-label?ref=488ab91e34a24a86957e397d9f7262ec5925586a>":
│ error downloading
│ '<https://github.com/cloudposse/terraform-null-label?ref=488ab91e34a24a86957e397d9f7262ec5925586a>':
│ /opt/homebrew/bin/git exited with 128: Cloning into
│ '.terraform/modules/this'...
│ fatal: Remote branch 488ab91e34a24a86957e397d9f7262ec5925586a not found in
│ upstream origin
However the under-hood modules are referenced by their tags not commit hashes. Any insights what could be the reason?
![Tommi Jensen avatar](https://secure.gravatar.com/avatar/c49dc4c662d564d0ca04ad7721ea5892.jpg?s=72&d=https%3A%2F%2Fa.slack-edge.com%2Fdf10d%2Fimg%2Favatars%2Fava_0023-72.png)
tags dereference to commits. git rev-parse tagname (take one of your tagged versions e.g.)
2024-01-24
2024-01-25
2024-01-30
![Alex S avatar](https://secure.gravatar.com/avatar/ab2a08af6924598b11d2a1f004b4524c.jpg?s=72&d=https%3A%2F%2Fa.slack-edge.com%2Fdf10d%2Fimg%2Favatars%2Fava_0021-72.png)
Hey folks, anyone able to review a small change for me? Just changes to variables (and readme update). Thank you https://github.com/cloudposse/terraform-aws-ecs-container-definition/pull/178
what
Allow specifying containerPortRange in portMappings
why
To support containerPortRange
references
![Andriy Knysh (Cloud Posse) avatar](https://avatars.slack-edge.com/2018-06-13/382332470551_54ed1a5d986e2068fd9c_72.jpg)
@Alex S thanks, the PR looks good, there is an issue with the terratest, please see comments
what
Allow specifying containerPortRange in portMappings
why
To support containerPortRange
references