[WIP] Add interpolate uuid function

[gjohnson]

Mostly just an exposed version of the uuid function added by @phinze and @radeksimko for resources.

I thought about adding a size parameter to control the length of the generated id (would make it more of a uid than a legit uuid though), thoughts?

To Do

• discuss size limit
• cleanup test to use testFunctionCase
• docs

[gjohnson]

This doesn't solve everything in #2269 yet, mostly assumes you'll use ignore_changes for now.

[phinze]

Can you run gofmt on this? That will line up these declarations.

[phinze]

This is looking good! A few minor comments and needs a rebase, but then this should be good to go. 👍

[phinze]

I thought about adding a size parameter to control the length of the generated id (would make it more of a uid than a legit uuid though), thoughts?

I'd be down for a second optional parameter that simply trims the string to a given max length - provided we document that it removes the uniqueness properties of RFC 4122 and increases the chance of collision.

uuid() -> "terraform-{uuid}"
uuid("foo-") -> "foo-{uuid}"
uuid("foo-", 12) -> "foo-{uuid}"[:12]

[jason-riddle]

Curious. What is the status on this?

[bigkraig]

👍

[bigkraig]

hey @gjohnson any chance you can rebase this?

[phinze]

Hey folks, I ended up revisiting this over in #5575 and taking a slightly different approach on the implementation. If we want a second function that works more like Terraform's internal PrefixedUniqueId we can consider that, but for the function we call uuid() I think it's best for it to be more of a standard style. 👍

Thanks for the work you put into this @gjohnson!

[ghost]

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.