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

support for terraform show #88

Open AlexMichaelJonesNC opened 5 years ago

AlexMichaelJonesNC commented 5 years ago

I'm looking to potentially open a PR for outputting terraform show on a plan file to a another file, so i can use drone-github-comment to post the expected terraform execution back to the originating PR so that our team can review the expected terraform change from there.

I wanted to see what your thoughts were on how it should be implemented.

1 - There options related to source file and color v -no-color that need to be set as part of running the command, I assume the correct way to do this config would be a new yaml field similar to fmt_options.

hieven commented 4 years ago

@jmccann What do you think about this issue? Do you think what @AlexMichaelJonesNC suggests here is a correct direction to support the feature?

I'd like to contribute to it since I have similar requirements in my job recently. Thanks!

jmccann commented 4 years ago

So I am in favor of such a workflow and wouldn't mind trying to support it. I personally haven't used terraform show much and not sure what all would be needed to support it ... but guessing it wouldn't be much to add another supported command/action.

I will throw out there though that as you get into "more complicated" automated workflows for terraform you may want to instead use a different tool like https://www.runatlantis.io/guide/. I've seen it in action and think it's pretty nice.