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

Add option to do terraform init without initializing backend #1044

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

Conversation

mimadrone
Copy link

I see this was raised before, as issue #318, but as it's a thing I would very much like to have, I've gone ahead and written the patch. This is simply an additional field for the Options structure that allows for passing "-backend=false" for terraform init.

Copy link
Contributor

@yorinasub17 yorinasub17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution! This LGTM! I'll kick off a regression build and if it passes, we can merge this in.

@morremeyer
Copy link

morremeyer commented Jan 24, 2022

@yorinasub17 Could you retrigger the tests if the failure is a flake or check what needs to be done?

I'd very much like to have this feature so that we can stop using terraform.RunTerraformCommandE everywhere.

If there's anything else that can be done to help out, please let me know.

Edit: I need to correct myself here, with the setup we use (terraform cloud remote backend), we need to generate an override file anyways as we want to override the whole backend config for all commands. The goal here is to use the local backend with terratest, but the remote backend in the configuration (for actual production use).

@iambalabharathi
Copy link

You need any help on this? @mimadrone
we need this as soon as possible. What we are doing now to workaround this issue. is bit ugly. :(

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

4 participants