lang/funcs: lookup() can work with maps of lists, maps and objects#22269
lang/funcs: lookup() can work with maps of lists, maps and objects#22269mildwonkey merged 3 commits intomasterfrom
Conversation
lookup() can already handle aribtrary objects of (whatever) and should handle maps of (whatever) similarly.
types match or can be converted.
| if len(args) == 3 { | ||
| _, err = convert.Convert(args[2], ty.ElementType()) | ||
| if err != nil { | ||
| return cty.NilType, function.NewArgErrorf(0, "the default value and map elements must have the same type") |
There was a problem hiding this comment.
The 0 here indicates that the problem is with the zeroth argument; perhaps it would be clearer to indicate 2 so that Terraform will indicate the default value as the subject of the error when it prints out diagnostics, as a subtle additional indication that it's probably the default value that ought to change to match the map, rather than the other way around.
|
Looking good, thanks! I left some minor notes inline. |
|
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. |
ghost
locked and limited conversation to collaborators
2 participants
lookup() can already handle arbitrary objects of (whatever) and should
handle maps of (whatever) similarly.
The documentation does not mention any limitation to the types returned, which was inconsistent with the error message when a map of objects, lists or maps was used.
Fixes #22267