Justin Van Patten

Results 348 comments of Justin Van Patten

@wardVD, are you asking that we upgrade our `grpcio` dependency to the latest version? How are you being impacted by the above bullets?

Thanks for the additional details. Makes sense. We definitely want to stay up-to-date with the latest versions.

I believe this has been addressed with #17487 (shipped in `pulumi` 3.136.0 and 3.136.1), which now requires `grpcio~=1.66.2` (the latest version). We're still strict in the version range we support...

This looks like it may be specific to AWS. Moving to the aws repo for further triage.

> Having an issue when trying to dynamically assemble(concatenate) resource_name from another resources properties `resource_name` must be a plain string. You can't create it based on other resource output properties...

I'm not seeing any obvious changes in `@pulumi/pulumi` between 3.127.0 and 3.128.0 that could impact this. Could they share a minimal snippet of a unit test that demonstrates the issue?

Thanks for the details @ringods! One more question: What does the implementation of `getDefaults` do?

The `import type` _should_ be erased at runtime, so it'd be interesting if this was indeed a cycle at runtime. https://github.com/pulumi/pulumi/blob/4c0ccfa6ad6f4ce1191a6d649a561100f058d740/sdk/nodejs/runtime/state.ts#L22

Another question @ringods: What are they using for the unit test? Sounds like possibly jest? Are they using ts-jest? I'm trying to put together a repro project, but haven't been...

I still can't reproduce this. I've created https://github.com/justinvp/pulumi-16974. When running the test, it succeeds. This is on macOS with Node v20.9.0. @ringods, are you able to reproduce the regression?