alan-turing-institute / data-safe-haven

https://data-safe-haven.readthedocs.io
BSD 3-Clause "New" or "Revised" License
61 stars 15 forks source link

`dbforpostgresql.Configuration` can't be deleted #2072

Closed jemrobinson closed 3 months ago

jemrobinson commented 3 months ago

:white_check_mark: Checklist

:computer: System information

:package: Packages

List of packages ```none Paste list of packages here ```

:no_entry_sign: Describe the problem

Attempts to delete dbforpostgresql:Configuration resources fail. See this issue on the pulumi-azure-native repo.

:deciduous_tree: Log messages

Relevant log messages ```none Diagnostics: azure-native:dbforpostgresql:Configuration (sre_remote_desktop_db_guacamole_secure_transport_configuration): error: Code="InternalServerError" Message="An unexpected error occured while processing the request. Tracking ID: 'd746ae35-9ba2-4495-ad04-27e41998ac30'"```

:recycle: To reproduce

Deploy an SRE with a dbforpostgresql:Configuration then remove it and attempt to update.

JimMadge commented 3 months ago

I think this should have been fixed by #2059

https://github.com/alan-turing-institute/data-safe-haven/blob/63539c4bbcb3d3fff960dc64a3e6f787fda7ae38/data_safe_haven/infrastructure/components/composite/postgresql_database.py#L93-L96

jemrobinson commented 3 months ago

Cool - I'll take a look on latest develop. This ResourceOption is new to me!

jemrobinson commented 3 months ago

This seems to be working now.