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.

Sign up for GitHub

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

Jump to bottom

Test: TF diff decision and pulumi diff decision match (PF) #2297

Closed
t0yv0 opened this issue Aug 9, 2024 · 1 comment · Fixed by #2592
Closed

Test: TF diff decision and pulumi diff decision match (PF) #2297

t0yv0 opened this issue Aug 9, 2024 · 1 comment · Fixed by #2592
Assignees
@VenelinMartinov
Labels
kind/task Work that's part of an ongoing epic resolution/fixed This issue was fixed
Milestone
0.113

Comments

@t0yv0
Copy link
Member

t0yv0 commented Aug 9, 2024 •
edited
Loading

Similar testing work to #2298 but for PF-based providers.
@VenelinMartinov VenelinMartinov removed their assignment Aug 20, 2024
@mjeffryes mjeffryes added the kind/task Work that's part of an ongoing epic label Aug 23, 2024
@mjeffryes mjeffryes added this to the 0.110 milestone Sep 12, 2024
@mjeffryes mjeffryes modified the milestones: 0.110, 0.111 Oct 2, 2024
@mjeffryes mjeffryes modified the milestones: 0.111, 0.112 Oct 30, 2024
This was referenced Nov 4, 2024
Merged
Merged
VenelinMartinov added a commit that referenced this issue Nov 4, 2024
@VenelinMartinov
Move cross test utilities to be shared between sdkv2 and pf (#2576)
ae9cd89 
This is a refactor which moves some of the SDKv2 cross-test utilities to
the crosstestimpl colder so that they can be reused in the PF
cross-tests.

related to #2297
VenelinMartinov added a commit that referenced this issue Nov 5, 2024
@VenelinMartinov
PF Diff cross tests (#2578)
b278128 
This change adds the framework code for running Diff cross-tests in PF.
It also adds a simple cross-test to show the framework works. I'll add
additional coverage in a follow-up PR.

I've also exposed the Stdout output of both TF Plan and Pulumi Preview,
so that they can be recorded in the cross-test. This will allow us to
use the cross-tests for testing accurate bridge previews.

related to #2297
@VenelinMartinov VenelinMartinov mentioned this issue Nov 6, 2024
Merged
@mjeffryes mjeffryes modified the milestones: 0.112, 0.113 Nov 13, 2024
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Nov 14, 2024
@pulumi-bot
Copy link
Contributor

This issue has been addressed in PR #2592 and shipped in release v3.96.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@VenelinMartinov VenelinMartinov

Labels
kind/task Work that's part of an ongoing epic resolution/fixed This issue was fixed
Projects
None yet
Milestone
0.113
Development

Successfully merging a pull request may close this issue.

Plugin Framework bridge cross tests pulumi/pulumi-terraform-bridge
4 participants
@t0yv0 @mjeffryes @VenelinMartinov @pulumi-bot