Updates for Terraform 0.13 - Remove module-wide create/enable variable #88

cmassey-berico · 2020-10-27T13:06:25Z

No description provided.

variables.tf

modules/authorization/variables.tf

tests/config_authorization/main.tf

tests/config_aggregator/main.tf

kbalk

I've made some comments, but they may not require any action as I'm still learning how terraform works and the comments may not be appropriate.

lorengordon · 2020-11-16T21:09:12Z

@kbalk Thanks for the review! I made a few changes in response, and replied to all the comments. Let me know if you have anything else.

lorengordon force-pushed the master branch 2 times, most recently from 7645f49 to 3dbfb7e Compare November 11, 2020 16:04

Cody Massey and others added 9 commits November 12, 2020 07:34

remove create_... in variables.tf

fa2b0ea

removed any references to the create_... in main.tf

84a27b6

removed the join() action in outputs.tf

260ae22

removed tests/no_create directory

9b49578

updated remaining tests/ to remove references to the create_ variable

b15f996

removed any README.md or versions.tf files from the tests/ directories

6ae66ba

Uses modules to create aggregator and authorization resources

3903136

Patches aggregator output to avoid unnecessary diff

9af01ac

Removes terratest go packaging

d9c8029

lorengordon force-pushed the master branch from 3dbfb7e to d9c8029 Compare November 12, 2020 15:34

Bumps version to 2.0.0

731a289

lorengordon requested a review from a team November 14, 2020 16:10