Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove experimental flag for use in latest terraform #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mvachhar
Copy link

@mvachhar mvachhar commented Nov 16, 2022

The latest terraform does not support the experiments = [module_variable_optional_attrs] flag and so this module cannot be used with the latest terraform. The default syntax for input variables in terraform 1.3.4 is adequate for this module so changes are minimal to fix it (mainly bumping the required tf version and removing the experiments line).

Also, fixes use of now deprecated of sensitive_context in local_file in favor of local_sensitive_file.

I have specifically not updated the README to reference a new version as that will only work once a new version has been published.

@redeux
Copy link
Owner

redeux commented Nov 17, 2022

Resolves #3

@bparentpichette
Copy link

bparentpichette commented Mar 7, 2023

This pull-request seems to be accepted by @redeux but it is not merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants