pwillis-els / terraformsh

A wrapper for Terraform in Bash
MIT License
66 stars 12 forks source link

terraform validate command error #10

Closed AMKamel closed 2 years ago

AMKamel commented 2 years ago

Hello @pwillis-els ,

I think it's a syntax error to use terraform validate command with -var-file option as it produced syntax errors with me. it only seem to support two options as in screenshot, right?

╷
│ Error: Failed to parse command-line flags
│ 
│ flag provided but not defined: -var-file
╵

image

pwillis-els commented 2 years ago

Yep!

I merged some code recently (https://github.com/pwillis-els/terraformsh/pull/7) (https://github.com/pwillis-els/terraformsh/pull/8) which should prevent passing those args to validate unless it's a version of Terraform earlier than 0.12, when those args used to be accepted.

Which version of terraformsh are you running, and which version of terraform? (I haven't made a new release of terraformsh with the changes yet, so you'll have to get the latest copy here: https://raw.githubusercontent.com/pwillis-els/terraformsh/main/terraformsh)

AMKamel commented 2 years ago

At the time was using 0.10 and terraform 1.0.10

AMKamel commented 2 years ago

It looks resolved by latest update but I got another problem which is that it seems terraformsh don't use workspaces and there is no way I found applicable to do the execution against some workspace

See in screenshot how it has ignore that we are in different workspace and tried to alter the current state of default workspace instead of use the current space? image

pwillis-els commented 2 years ago

That's interesting, I haven't used workspaces with the tool so I'm not sure what's going on. I'll need to do some testing.

I've created a new issue to track your workspaces issue: https://github.com/pwillis-els/terraformsh/issues/11

I have a suggestion there for something to try in the mean time