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

Parsing error #116

Closed owensj8 closed 4 years ago

owensj8 commented 4 years ago

When I try to run landscape I get a parsing error. The terraform plan on its' own is successful so not sure what this could be. ''' ❯ tf12 plan | landscape Terraform Landscape: a parsing error occured. Falling back to original Terraform output... ''' On another issue that seems related i saw an option to grab a trace which I did.

''' ❯ tf12 plan | landscape --trace Traceback (most recent call last): 13: from /usr/local/bin/landscape:23:in <main>' 12: from /usr/local/bin/landscape:23:inload' 11: from /var/lib/gems/2.5.0/gems/terraform_landscape-0.3.3/bin/landscape:7:in <top (required)>' 10: from /var/lib/gems/2.5.0/gems/terraform_landscape-0.3.3/lib/terraform_landscape/cli.rb:25:inrun' 9: from /var/lib/gems/2.5.0/gems/commander-4.5.2/lib/commander/delegates.rb:15:in run!' 8: from /var/lib/gems/2.5.0/gems/commander-4.5.2/lib/commander/runner.rb:68:inrun!' 7: from /var/lib/gems/2.5.0/gems/commander-4.5.2/lib/commander/runner.rb:452:in run_active_command' 6: from /var/lib/gems/2.5.0/gems/commander-4.5.2/lib/commander/command.rb:155:inrun' 5: from /var/lib/gems/2.5.0/gems/commander-4.5.2/lib/commander/command.rb:184:in call' 4: from /var/lib/gems/2.5.0/gems/terraform_landscape-0.3.3/lib/terraform_landscape/cli.rb:34:inblock (2 levels) in define_commands' 3: from /var/lib/gems/2.5.0/gems/terraform_landscape-0.3.3/lib/terraform_landscape/cli.rb:53:in print' 2: from /var/lib/gems/2.5.0/gems/terraform_landscape-0.3.3/lib/terraform_landscape/printer.rb:42:inprocess_stream' 1: from /var/lib/gems/2.5.0/gems/terraform_landscape-0.3.3/lib/terraform_landscape/printer.rb:99:in process_string' /var/lib/gems/2.5.0/gems/terraform_landscape-0.3.3/lib/terraform_landscape/terraform_plan.rb:40:infrom_output': Expected one of [\s], [ ], '~', '-/+', '-', '+', '<=' at line 3, column 3 (byte 5) after (TerraformLandscape::TerraformPlan::ParseError) '''

sds commented 4 years ago

Hey @owensj8, as mentioned in the README, Landscape does not support Terraform 0.12 at this time. Would welcome a pull request adding support!