search

jmccann / drone-terraform

Drone plugin for triggering Terraform deployments
http://plugins.drone.io/jmccann/terraform/
Apache License 2.0
86 stars 93 forks source link

Update to terraform version 0.9.4 #39

Closed msuterski closed 7 years ago

msuterski commented 7 years ago

Updates TF to version 0.9.4.

TF 0.9.x introduced breaking changes (mostly around remote state).

msuterski commented 7 years ago

To support the new backend types, the init command needs to run on each machine, which require running it on each run in case of the plugin.

I'm going to file another PR that will add this functionality.

Closing this for now.

jmccann commented 7 years ago

Seems #40 may be what you are looking for. I'm going to try and review it today.

msuterski commented 7 years ago

Thanks. #40 is missing a couple of options necessary for the init to work as expected. I'll file a PR that adds them and we can think about combining them.

jmccann commented 7 years ago

@msuterski May be easier to add comments to #40 to address it there?

msuterski commented 7 years ago

40 is a mix of a couple of things, not necessarily only adding support for the new init command. The part for the init is just calling it without any options. It would take a couple of comments to request what's done in #41.

Maybe we could point to #41 or simply merge it to #40?