-
Notifications
You must be signed in to change notification settings - Fork 18
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
Create a package for built-in functions #267
Comments
Should we close out pulumi/pulumi#1552 as a dupe of this? |
Good progress on this one, moving out of M19 but keeping in M20 as I'm sure we'll keep chipping away. |
@jen20 Should we make available the library as it currently stands and close this out and then track any remaining things to add as part of the new repo? |
Yes, I think that's the best solution here - I'll do that today. The remaining functions either have very unusual semantics in Terraform, or are related to IPv6. I'll publish what we have, and open issues for each remaining function. |
Is this still an active feature request? Really hoping for |
@jen20 Do we still have the work for this available somewhere? Would love to just publish as an NPM package in current form. |
Is this still a thing ? |
We haven't heard many requests/needs for this in the last 18 months, but in principle it's still "a good idea". There was a PR that made a lot of progress on this at #268, but it was never finalized. Two things:
|
👋 I'll add one vote to the "interested" tally! We are using C# - if this were to happen I'm guessing it would need to be implemented separately in each language right? Since this isn't resource provider code there isn't any centralized golang implementation behind grpc or anything? |
Ah I was just looking for the equivalent CIDR helpers. Sad to see they aren't available still! |
We decided to create a package,
@pulumi/terraform
, which contains all of the Terraform built-in functions in raw, unopinionated form (same names, same semantics). This will help significantly with conversion tooling and efforts (for instance,tf2pulumi
can use it).The text was updated successfully, but these errors were encountered: