coinbase / terraform-landscape

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

When using interactive input, landscape stalls #18

Closed joscha closed 7 years ago

joscha commented 7 years ago

When using terraform plans that expect input variables, e.g.

$ terraform apply

var.ami
  Enter a value: ami-XXXXX

var.count
  Enter a value: 5

var.metadata
  Enter a value: role=yyyy

then running this output through landscape hides the fact that the plan/apply is waiting for such input. I am not sure if there is an easy way for landscape to detect this, but if there is, printing at least a warning would be great.

sds commented 7 years ago

Hey @joscha,

This isn't a workflow we deal with at Coinbase, but we'd be happy to accept a pull request that detects this case or allows you to provide input. Thanks!

joscha commented 7 years ago

Understood @sds - not sure my shell-fu is strong enough, but will check :) I'll close this in the meantime.