Chore: remove Azure file share and mount#3492
Conversation
|
There is one more resource that can be removed, moving this back to draft to add it... |
since for each environment we are not storing the Sqlite database file in an Azure File Share mount, we can remove this. For more context, these resources were originally created by #1886
angela-tran
left a comment
There was a problem hiding this comment.
This looks good to me. Glad to have this cleaned up 🙌
jgravois
left a comment
There was a problem hiding this comment.
nice cleanup!
we discussed nuking the sacalitpd001 storage account yesterday too, but i'm under the impression that terraform isn't managing that resource. 🤙
Yeah, I think it's fine to delete that one too. My memory of it is it was created before we started using Terraform. At this point, it's been sitting around unused for years. |
|
Thanks for the review everyone! I'll keep an eye out on this as it gets deployed. And that's right @jgravois, we won't move |
Part of #3444
This PR will help with #3444 and #2831 by having less resources to move to the new subscription and by freeing up the App Service from the storage account, simplifying the move.
Azure file share
Since for each environment we are storing the Sqlite database file in the Azure managed
/homedirectory, and not in an Azure file share mount, we can remove this. The file share was originally created to persist the Sqlite database in #1775 but after noticing some issues with the database being locked, the database was moved to the Azure managed/homedirectory in #2166.As an extra verification of the behavior described above, we confirmed that the database file in the file share of each environment (
dev,test, andprod) was last modified in June 2024, matching the date of #2166.Recovery services vault
A recovery services vault was initially configured in #1886, but since we are not using the Azure file share anymore, we can remove this.
As an extra verification that it is not being used, we confirmed in the Azure Portal that for each environment,
Protected items -> Backup itemsshows 0 backed up items for each category of backup type.Terraform plan
Terraform plan shows
as expected.