This message was deleted.
# typescripts
sparse-intern-71089
10/27/2021, 8:09 PMThis message was deleted.
l
little-cartoon-10569
10/27/2021, 8:20 PMIs it one of the get functions that returns a Promise, rather than an Output?
little-cartoon-10569
10/27/2021, 8:24 PMI have a fair few resources like that. Here's a snippet from the constructor of one.
b
billowy-army-68599
10/27/2021, 8:24 PM
does this capture the issue?
https://github.com/pulumi/pulumi/issues/5758
l
little-cartoon-10569
10/27/2021, 8:25 PMThe important bit imo is that Promises returned from get functions like getAmi have to be wrapped in
pulumi.output()l
limited-rainbow-51650
10/27/2021, 8:41 PM@billowy-army-68599 that issue is indeed my mental tormentation. 🙂
limited-rainbow-51650
10/27/2021, 8:44 PM@little-cartoon-10569 my code looks very similar to yours regarding, except for the place where you use
pulumi.output(...)limited-rainbow-51650
10/27/2021, 8:45 PMLet me try it around the get function.
b
billowy-army-68599
10/27/2021, 8:46 PM
@limited-rainbow-51650 i believe your experience is going to get much better here pretty soon 😄
l
limited-rainbow-51650
10/27/2021, 8:47 PM@billowy-army-68599 you mean by applying the suggestion from @little-cartoon-10569, or by you and your colleagues fixing that issue? 😄
b
billowy-army-68599
10/27/2021, 8:48 PM
by making the experience better, accepting outputs in functions
🎉 1
billowy-army-68599
10/27/2021, 8:48 PM
but any time tenwit is involved your life gets better too 😄
💯 1
😊 1
billowy-army-68599
10/27/2021, 8:56 PM
@limited-rainbow-51650 - @enough-garden-22763 pointed me to this:
https://github.com/pulumi/examples/pull/1096/files
take a look at the experience and let us know what you think
🚀 1
🍸 1
🎉 1
e
enough-garden-22763
10/27/2021, 9:00 PMThe functions now have $fnOutput forms that return an Output, and accept Input/Output in arguments; these forms behave similarly to what you would do by mechanically wrapping promise-returning functions with an apply block. This should be available in TypeScript already, we are just waiting on languages like C# to land support before making an official blog post about it.
l
limited-rainbow-51650
10/27/2021, 9:01 PM@enough-garden-22763 which minimal version of Pulumi has this for NodeJS?
e
enough-garden-22763
10/27/2021, 9:02 PMThe relevant version here is version of providers.
👍🏼 1
enough-garden-22763
10/27/2021, 9:02 PME.g. https://github.com/pulumi/pulumi-aws/blob/v4.25.0/CHANGELOG.md v4.25.0 definitely has it, it’s not obvious which older version had it, one sec
enough-garden-22763
10/27/2021, 9:05 PMSo sounds like we’re looking at providers built against 3.15.0 (2021-10-14) or higher of pulumi/pulumi; I can’t seem to easily correlate this through pulumi-aws versions, apologies.
l
limited-rainbow-51650
10/27/2021, 9:06 PMIn my case, I will have to search for versions of the GCP (classic) and Kubernetes providers to have this.
e
enough-garden-22763
10/27/2021, 9:11 PMenough-garden-22763
10/27/2021, 9:13 PMSo it sound like since https://github.com/pulumi/pulumi-gcp/blob/master/CHANGELOG.md#5230-2021-10-08 5.23.0
enough-garden-22763
10/27/2021, 9:19 PMWith Kubernetes I don’t seem to find any functions in https://www.pulumi.com/registry/packages/kubernetes/api-docs/ so it does not seem it would be affected; sorry this is the provider I’m a bit less familiar with, so if you give me an example piece of code you are thinking of updating that can help me find it quickly.
l
limited-rainbow-51650
10/27/2021, 9:25 PM@enough-garden-22763 with all the help I got so far, I was able to fix it. I will definitely start to use the new function Output functionality ASAP. It simplifies the coding a lot!
❤️ 1
3 Views