-
Notifications
You must be signed in to change notification settings - Fork 8
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
Pulumi Refresh Unexpectedly Removing Datadog Resources #412
Comments
Hi @andrew-purdin. I'm sorry to hear that. To confirm, are you saying that May be related to DataDog/terraform-provider-datadog#2219. |
Yeah @iwahbe that's correct. We first saw the breaking behavior via refresh on December 7th and our pipeline ran last before that on November 6th without the refresh causing any removals. That run did not include any version changes of the provider nor any other dependencies. |
Thanks for confirming @andrew-purdin. |
Hi @iwahbe, we were doing some more digging on this and we think the problem is caused by some API changes that were made by DataDog. (See 2146 and 2200 on the TF provider). I'm not 100% certain, but I think we are unable to work around this until version 3.32.0 of the TF provider is integrated to the Pulumi provider because of this PR. I saw there is an open PR to jump to 3.33.0, but it's been stalled for a long time. Can we expect to see that released soon? We have some Datadog integration resources we need to provision very soon, so I'd appreciate your timely help with this. |
@iwahbe can you provide an update on this please? |
Hi @andrew-purdin. Sorry for the lack of update. We are working on this now. |
Thanks @iwahbe, I see your PR, do you think it will be merged soon? |
Hey @andrew-purdin. We're going to merge today. We were hesitant to merge before, since the change added about 1.5 million lines of generated code to the pulumi-datadog SDK. That makes certain libraries very heavy, which is not the experience we hope to give users. We were hoping that we would have time to reduce the size of the SDK (this issue: pulumi/pulumi-terraform-bridge#1468), but that has proven harder then expected. Given the breaking change introduced by Datadog, we have decided to merge now, and then follow up when we have time to reduce the SDK size. |
What happened?
Pulumi refresh is removing our
IntegrationLambdaArn
andIntegrationLogCollection
across all of our stacks. The resources exist and have not been altered via the Datadog UI. This wasn't happening as of about a month ago, though we can observe the behavior on@pulumi/datadog
4.23 as well as 4.21.Example
Output of
pulumi about
I'm not comfortable posting all the information from
pulumi about
, the issue is consistent between my local machine and a remote pipeline however. Below is relevant version and plugin information.pulumi plugin ls
from local:Snippets from
pulumi about
:Additional context
This was working consistently for some time, so I suspect something has changed either in the upstream Terraform provider, or in the Datadog API.
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: