Skip to content
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

[TFPF] Upstream provider is now a mixed SDK/TFPF provider #1051

Closed
kpitzen opened this issue Apr 5, 2023 · 0 comments · Fixed by #1075
Closed

[TFPF] Upstream provider is now a mixed SDK/TFPF provider #1051

kpitzen opened this issue Apr 5, 2023 · 0 comments · Fixed by #1075
Assignees
Labels
area/providers awaiting-upstream The issue cannot be resolved without action in another repository (may be owned by Pulumi). blocked The issue cannot be resolved without 3rd party action. impact/usability Something that impacts users' ability to use the product easily and intuitively kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed

Comments

@kpitzen
Copy link
Contributor

kpitzen commented Apr 5, 2023

What happened?

As of the most recent release, the google-beta provider has begun to include TFPF resources alongside existing TF SDK resources. To support bridging these resources out of the box, we will require the tfbridge muxer work to be applied to this provider as well.

Expected Behavior

Resources like this: https://github.com/pulumi/terraform-provider-google-beta/blob/patched-v4.60.0/google-beta/data_source_dns_keys.go

Should be bridged into the Pulumi provider as expected, rather than backported as a reverted resource: https://github.com/pulumi/terraform-provider-google-beta/blob/patched-v4.60.0/google-beta/data_source_dns_keys_reverted.go

Steps to reproduce

Run make tfgen on a new tag of the upstream provider. Notice that new TFPF resources are missing from the output.

Output of pulumi about

N/A

Additional context

Existing epic tracking TFPF work in the bridge as well as the Muxer work mentioned above:
pulumi/pulumi-terraform-bridge#744

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@kpitzen kpitzen added kind/bug Some behavior is incorrect or out of spec needs-triage Needs attention from the triage team impact/usability Something that impacts users' ability to use the product easily and intuitively area/providers awaiting-upstream The issue cannot be resolved without action in another repository (may be owned by Pulumi). blocked The issue cannot be resolved without 3rd party action. and removed needs-triage Needs attention from the triage team labels Apr 5, 2023
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/providers awaiting-upstream The issue cannot be resolved without action in another repository (may be owned by Pulumi). blocked The issue cannot be resolved without 3rd party action. impact/usability Something that impacts users' ability to use the product easily and intuitively kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants