-
Notifications
You must be signed in to change notification settings - Fork 7
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
TestRegress170 fails under latest bridge changes #528
Comments
Reproes with bridge master |
P1 because it's blocking the next bridge release. |
I have confirmed that the regression is genuine (and not overfitting from the test). |
This looks like a bug in upstream: hetznercloud/terraform-provider-hcloud#650. I have confirmed that Pulumi can use the same workaround as TF suggests. I'll bake the work-around into a transform so we don't expose the change to our users. I believe that we are safe to release the bridge. |
Fixes #528 I have removed the diff replay test since it doesn't capture the issue anymore. I have replaced this with an integration test since we need to ensure that "Check" behaves such that diff doesn't present an issue.
Fixes #528 I have removed the diff replay test since it doesn't capture the issue anymore. I have replaced this with an integration test since we need to ensure that "Check" behaves such that diff doesn't present an issue.
Cannot close issue:
Please fix these problems and try again. |
Describe what happened
https://github.com/pulumi/pulumi-hcloud/actions/runs/9451894942/job/26034396438?pr=525
Seems to expect no diff but gets a diff.
Looks like a regression on #170
Sample program
.
Log output
Affected Resource(s)
No response
Output of
pulumi about
.
Additional context
No response
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).
The text was updated successfully, but these errors were encountered: