-
Notifications
You must be signed in to change notification settings - Fork 37
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
Unable to delete an azure-native.dbforpostgresql.Configuration
resource: PUT vs PATCH
#3476
Comments
Thank you for reporting this issue! I can reproduce it. This is the Azure HTTP request and response.
|
The Azure docs state
The Pulumi provider does that (not sending a value on reset of the property), but it made me wonder whether the provider maybe shouldn't send I tried both sending At this point I think there might be an Azure server-side issue here. The only thing I haven't tried yet is following the cli instructions I linked above and checking whether that works and, if yes, what request it sends. |
Thank you for checking and investigating, @thomas11! Much appreciated. Based on what you are writing it seems strange. If it is an Azure server-side issue, how do you typically proceed with a fix? |
I took a second, fresh look and compared our provider's request with what the Azure CLI does, via We'll have to look into adding an exception for updating this resource to the provider. |
azure-native.dbforpostgresql.Configuration
resourceazure-native.dbforpostgresql.Configuration
resource: PUT vs PATCH
Also filed [BUG] Cannot reset a dbforpostgresql Configuration via PUT #30143 against Azure. |
Belated thanks for following up on this and filing the bug, @thomas11 |
Hi, is there a workaround for this issue? I tried it with |
Unfortunately, there's no known workaround currently. We're still discussing with the Azure team on the upstream issue. |
@thomas11, could you explain to me why |
Hi @penenkel, sorry I didn't respond to that part of your message. I don't see at first glance why import {
Configuration,
Server,
SkuTier,
} from "@pulumi/azure-native/dbforpostgresql";
import { ResourceGroup } from "@pulumi/azure-native/resources";
const resourceGroup = new ResourceGroup(`deleteme`);
const server = new Server(
`deleteme`,
{
resourceGroupName: resourceGroup.name,
administratorLogin: "NotForProduction",
administratorLoginPassword: "NotForProduction",
sku: {
name: "Standard_B1ms",
tier: SkuTier.Burstable,
},
version: "16",
storage: {
storageSizeGB: 32,
},
},
{ parent: resourceGroup },
);
new Configuration(
"deleteme",
{
resourceGroupName: resourceGroup.name,
serverName: server.name,
configurationName: "azure.extensions",
value: "vector",
source: "user-override",
},
{ parent: server, deletedWith:server },
); Are you using |
I'm using the dotnet variant, but otherwise, my code looks similar. Though I'm not using parent, is that required? |
I don't think |
Hi @penenkel, I've just tried to reproduce the issue but the deletion worked fine with using Pulumi.AzureNative.Resources;
using Pulumi.AzureNative.DBforPostgreSQL;
using Pulumi.AzureNative.DBforPostgreSQL.Inputs;
using System.Collections.Generic;
return await Pulumi.Deployment.RunAsync(() =>
{
var resourceGroup = new ResourceGroup("resourceGroup");
var server = new Server("server", new ServerArgs
{
ResourceGroupName = resourceGroup.Name,
Location = resourceGroup.Location,
Version = "14",
Sku = new SkuArgs
{
Name = "Standard_B1ms",
Tier = SkuTier.Burstable
},
Storage = new StorageArgs
{
StorageSizeGB = 32
},
AdministratorLogin = "pulumiadminuser",
AdministratorLoginPassword = "pulumiadminpassword"
});
var conf = new Configuration("deleteme", new ConfigurationArgs
{
ResourceGroupName = resourceGroup.Name,
ServerName = server.Name,
ConfigurationName = "azure.extensions",
Value = "vector",
Source = "user-override",
},
new Pulumi.CustomResourceOptions { DeletedWith = server }
);
return new Dictionary<string, object?>
{
["serverAddress"] = server.FullyQualifiedDomainName
};
}); I ran |
hm, perhaps my stack had devolved into a broken state by that point. I will try again and report back. |
This issue has been addressed in PR #3752 and shipped in release v2.76.0. |
What happened?
I seem not to be able to delete an
azure-native.dbforpostgresql.Configuration
resource. As a consequence I can also not destroy a stack without manual intervention. As you can see in the reproducible example below I am only doing a minor configuration change.Thank you for your great work on this project.
Example
I have verified that the configuration change has happened in Azure.
After commenting out the
Configuration
code block and runningpulumi up
I get the following error:The same also happens when running
pulumi destroy
.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: