Anton Tayanovskyy

Luke noted NewValueKnown() bool API in https://github.com/hashicorp/terraform-plugin-sdk/blob/dd6c1007928eb4058ce836752fd9ec06d304941e/helper/schema/resource_diff.go#L423 that might possibly be part of fixing upstream here.

Might need to land it on master first and iterate on this a bit, since I'm not sure we can test a workflow otherwise before it is merged.

Oh never mind, this is somehow running already. SO I can experiment without merging. Will do.

ResourceProvider.Read we're trying to modify here is very overloaded in Pulumi unfortunately, so there's a matrix of case here to consider. #### Case 1: pulumi import on the CLI ```...

The additional matrix dimension is that resources can be PF based or SDKv2 based and the underlying TF behavior can be different .

TransformOutputs is a blunt tool since it will not only modify Read results, but also modify Create and Update results, and it can't yet understand which kind of Read context...

I gave this a quick read this is looking pretty good. - Lifecycle testing imports this can add significant peace of mind around the system behavior, excellent 👏 - The...

This reverts https://github.com/hashicorp/terraform-provider-aws/pull/35671 There is an open claim that this PR introduced https://github.com/hashicorp/terraform-provider-aws/issues/36459 which is still open upstream.

https://github.com/pulumi/pulumi-aws/pull/3426 introduced this

I'm revisiting this today because @corymhall is running into this patch creating friction in the routine update (https://github.com/pulumi/pulumi-aws/pull/3959). It looks like we are not yet ready to remove it, @VenelinMartinov...