-
Notifications
You must be signed in to change notification settings - Fork 9.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bad output value silently ignored #5334
Comments
I also got bit by this one. Out of curiosity, why does your second example fail? I've successfully used the following, the variable is a list:
|
The second one fails because it's not specifying a field to return. Should be something like |
Ah, that's clear now, thanks! |
Thanks for reporting this - I agree this should give an error rather than silently dropping the output. |
Currently outputs with bad interpolation paths are silently ignored at plan and apply, rather than giving an error message. This (currently failing) test reproduces that.
This also happens for bad interpolation function calls, e.g. passing the wrong type or number of args to a interpolation function, e.g. compact("x", "y") |
This is actually tracked as "core+bug" issue #4975. I'm centralizing there! |
@mitchellh you centralized #4975 to this issue, forming a frustrating cycle. Is there a more canonical issue that could be linked? |
Oh Hah. I made a mistake, #9080 is the open one. |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Naïvely one might expect to be able to use a list as an output (instead of the required string). Doing this produces no error:
Nor this:
Nor this:
Basically, you end up stumbling around confused why your output isn't showing up. It would be great to get some sort of error message instead of silently dropping the output.
Appears related to #4369.
The text was updated successfully, but these errors were encountered: