Hello everyone :wave: please forgive if this is th...
# typescript
s
Hello everyone 👋 please forgive if this is the wrong place for it but wanted to discuss it before blindly opening a bug ticket on the repo. We found an interesting (possible) bug in the
package.json
in the
@pulumi/aws
package: Currently it references
@pulumi/pulumi: 3.42.0
https://github.com/pulumi/pulumi-aws/blob/master/sdk/nodejs/package.json#L16 However it contains calling code to a function (
invokeOutput
) only introduced in
v3.133.0
https://github.com/pulumi/pulumi/releases/tag/v3.133.0 / https://github.com/pulumi/pulumi/pull/17237 Calling function ref: https://github.com/pulumi/pulumi-aws/blob/master/sdk/nodejs/route53/getZone.ts#L158 (there could be more, this is the one we found). We have encountered (due to installing specific versions of
@pulumi/pulumi
which are
<3.133.0
but still satisfies
^3.42.0
) the following error:
Copy code
error: Running program '/builds/...../index.ts' failed with an unhandled exception:
    <ref *1> TypeError: pulumi.runtime.invokeOutput is not a function
        at Object.getZoneOutput (/builds/.../node_modules/@pulumi/route53/getZone.ts:158:27)
        at /builds/....../resource.ts:153:39
Is this considered an issue I should be raising a bug over on github for? Or is it more a "sort your own deps out" 😂
f
s
Thanks for the reply, certainly looks like that could be it! I'll have a play around with our versions again today and see if it is. 🙂
Looking at the code in 17353, this only updates the
invokeOutput
function itself though. I think this would not help as our error is:
TypeError: pulumi.runtime.invokeOutput is not a function
I believe... it's all down to the aws-sdk not having a dependency on a version of the core that has this function in it. I agree updating is the preferred solution but updating the dependency in the aws-sdk for a version of pulumi which contains the code that wants to be called could protect others from this error.
f
oh no, you're absolutely correct; we just updated the dep a few hours ago 😄 https://github.com/pulumi/pulumi-aws/blob/master/sdk/nodejs/package.json#L16
relevant issue for tracking if you'd like: https://github.com/pulumi/pulumi-aws/issues/4641