cal-itp / benefits

Transit benefits enrollment, minus the paperwork.
https://docs.calitp.org/benefits
GNU Affero General Public License v3.0
28 stars 9 forks source link

Delete all data migration files that are no longer needed #1244

Closed angela-tran closed 1 year ago

angela-tran commented 1 year ago

Once #1243 is done, let's clean up all the files and Azure resources that are no longer being used.

Tasks

Acceptance criteria

thekaveman commented 1 year ago

@angela-tran once we're fully swapped over to Key Vault in all environments, assuming we don't have the necessary permissions (I don't think we do) let's ask DevSecOps to help with the deletes. I think even the buckets themselves could go away so as not to confuse anyone.

angela-tran commented 1 year ago

Let's send an email to DevSecOps. This ticket can be closed since our side is done

angela-tran commented 1 year ago

I poked around the storage accounts in Azure today and found that I was able to delete files and containers. In the dev resource group, I had to delete all files within a container before I could delete the container. In test and prod resource groups though, I was able to delete at the container level even if there were files within them.

I cleaned up all the containers that used to hold fixture / data migration files. I left alone the containers that I think were used by Application Insights and of course the container that holds our Terraform state.