- Notifications
You must be signed in to change notification settings - Fork928
Closed
Description
OS Information
- OS: Linux
- Architecture: amd64
coder --version
: Coder v0.0.0-devel+c034e83 Wed May 18 23:15:19 UTC 2022
c034e83
Steps to Reproduce
- Observe that the
disk_size
variable in theaws-linux
example template has a default of20
- Edit the default to
""
- Try to create the template and get an error at the "detecting persistent resources" step:
template import provision for start: recv import provision: initialize terraform: configuration is invalid
- Edit the default to "asdf"
- Try to create the template and get the same error
- Edit the default value to any valid number e.g. 1000
- Try to create the template
- No error
Expected
If template authors set a default to an invalid value (e.g. they put letters in the default of a field withtype = number
) they should get a sensible error message.
Actual
Evil error message
Logs
Screenshot
Notes
Hopefully terraform is spitting out an error that we're just not displaying correctly, otherwise this will be hellish to fix.
Dean: Run terraform validate before the plan in the template import job.