search

coinbase / terraform-landscape

Improve Terraform's plan output to be easier to read and understand
Apache License 2.0
1.58k stars 111 forks source link

Support when Terraform prompts for values from standard input #103

Open ianomad opened 5 years ago

ianomad commented 5 years ago

cmd: terraform plan .. | landscape... cmd: , but it's stuck

i guess the workaround is using -var flags, but that's a bit inconvenient

sds commented 5 years ago

Would happily accept a pull request adding support for this!

xild commented 5 years ago

@sds any suggestion to implement? I can go over it if you have an idea how to do that in a good way 😄

ianomad commented 4 years ago

i am gonna guess the reason why it's stuck is because when doing a pipe, landscape input takes over the stuck terraform process waiting for its input. one way is let landscape read those vars (not sure how challenging that will be)

Dzhuneyt commented 4 years ago

This is a showstopper.

sds commented 4 years ago

Yes, @ianomad is correct about the situation with standard input and piping from terraform to landscape.

I'm going to lock this thread to prevent comments that add noise.

At this point, would welcome a pull request adding support for this!