-
Notifications
You must be signed in to change notification settings - Fork 174
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
auto-approve parameter is overwritten. #84
Comments
yes for me while using executing terraform from windows when i use terraform apply functioni am getting " flag provided but not defined: -auto-approve " error. |
Yes this does happen, I will look at a fix. |
I will address this in the rewrite I am going to do soon. If you'd like to help with the rewrite, or even planning for the rewrite let me know. |
yes definitely I can help. Please let me know how we can collaborate. |
It seems like providing the variable It doesn't actually seem to skip the plan, but instead sets the auto-approve variable accordingly. |
First of all, thanks for this tool. It seems I've been bitten by this bug too: Code snippet:
...results in (as per ps -ef): OTOH, if I don't provide the auto_approve flag, that is:
then ps shows: It seems NOTE:
this turned into Testing right with terraform, it results that both
Also, this seems related to #45. |
The apply function in init.py seems to overwrite auto-approve. I have made a local change
in my copy of the code and it works by first checking if auto-approve has been set
I've added the if statement before the
if not default['auto-approve']:
default['auto-approve'] = (skip_plan == True)
I would like to contribute this fix if possible but starting with opening the issue - already mentioned
in an issue opened in 2018 "Terraform ignore -auto-approve=True"
The text was updated successfully, but these errors were encountered: