Feature request: plan.replace

[jasonmk]

From TF 0.15.2 and onward, the replace option to plan/apply is preferred to using the taint-apply combo. It doesn't appear that ruby-terraform supports this option. How difficult would it be to add?

More details here: https://www.terraform.io/cli/commands/plan#replace-address

Thanks!

[tobyclemson]

Hey @jasonmk,

This shouldn't be too hard to add and we'd happily accept a PR. Alternatively, if you are happy to wait a little while, we'll try to get to it in the next week or so.

Thanks, Toby

[tobyclemson]

As a bit of a pointer as to where to add this in a PR, you would need to:

• Add a test in spec/ruby_terraform/commands/plan_spec.rb - you can probably use the shared behaviour
• Add a new string option in lib/ruby_terraform/options/definitions.rb
• Add the option to the plan command in lib/ruby_terraform/commands/plan.rb
• Update the docs in lib/ruby_terraform/commands/plan.rb and lib/ruby_terraform.rb

[jasonmk]

Taking a look. Will adding to plan make it work with apply directly as well or would it have to be added to each?

[tobyclemson]

Ah! Apologies, missed that you had said apply too.

You'll need to add to apply as well.

[jasonmk]

Took a stab at it. It's an option that you can specify multiple times. I don't really like using the word 'replaces' as the parameter, but it looks like maybe the pluralization is necessary for the command line builder (at least based on how targets is implemented). Let me know if there's a better way.

[tobyclemson]

Looks good to me. I think even though 'replaces' sounds a little strange, it's consistent with the other pluralised parameters so I think it makes sense to stick with that.

[tobyclemson]

1.5.0 includes this change.

Thanks for taking the time!

[jasonmk]

Thank you for pushing it out so fast!